[MDSAL-836] Move generated YangModelBindingProvider and ResourceYangModuleInfo Created: 01/Aug/23 Updated: 26/Jan/24 Resolved: 18/Jan/24 |
|
| Status: | Resolved |
| Project: | mdsal |
| Component/s: | Binding codegen |
| Affects Version/s: | None |
| Fix Version/s: | 13.0.0 |
| Type: | Improvement | Priority: | Medium |
| Reporter: | Robert Varga | Assignee: | Robert Varga |
| Resolution: | Done | Votes: | 0 |
| Labels: | pt | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Description |
|
We currently generate $YangModuleInfoImpl and $YangModelBindingProvider into org.opendaylight.yang.gen.v1.NAMESPACE package. The naming of these two classes is rather unfortunate, as it starts with $ due to the requirement not to conflict with other generated classes. At the end of the day, though, these are separate constructs. Allocate a new top-level package name to host these: org.opendaylight.yang.svc.v1, appended with the namespace, and name these YangModuleInfoImpl and YangModelBindingProviderImpl in that package. At the end of the day:
We already define the leading namespaces in yang.binding.contract.Naming, this needs to be captured there. This also will throw a wrench in assumptions around binding class -> provider/info mapping, so those need to be fixed up, too. |