Hi,
I do not agree these changes should be made at this late date.
It seems to me that in order to support a feature you have to implement it,
and therefore if any features are set then the module is implemented, not
imported.
All features should be set to false in an import-only module.
IMO this interpretation holds for typedef modules like iana-crypt-hash.
We list that as implemented (because it is) and the features that are
supported are set.
Andy
On Wed, Jan 30, 2019 at 11:03 AM Martin Bjorklund <m...@tail-f.com> wrote:
> Hi,
>
> Ladislav Lhotka <lho...@nic.cz> wrote:
> > Hi,
> >
> > unlike RFC 7895, 7895bis doesn't provide the "feature" leaf list for
> > import-only modules. But is it really so that features have no use in
> > such modules?
> >
> > For example, an enum can depend on a feature, and if it is inside a
> > typedef, it can also be in an import-only module. What if that feature
> > is defined in the same module?
>
> I think you're right, and that this is an unfortunate omission.
>
> The fix is simple though; we would have to add the leaf-list features
> to import-only. Probably refactor the "feature" leaf-list into a
> grouping so it works like the grouping location-leaf-list:
>
> grouping feature-leaf-list {
> leaf-list feature {
> type yang:yang-identifier;
> description
> "List of all YANG feature names from this module that are
> supported by the server, regardless whether they are defined
> in the module or any included submodule.";
> }
> }
>
> And then "uses feature-leaf-list":
>
> OLD:
>
> grouping module-implementation-parameters {
> description
> "Parameters for describing the implementation of a module.";
>
> leaf-list feature {
> type yang:yang-identifier;
> description
> "List of all YANG feature names from this module that are
> supported by the server, regardless whether they are defined
> in the module or any included submodule.";
> }
>
> NEW:
>
> grouping module-implementation-parameters {
> description
> "Parameters for describing the implementation of a module.";
>
> uses feature-leaf-list;
>
>
> And in the list "import-only":
>
> OLD:
>
> uses location-leaf-list;
>
> uses feature-leaf-list;
>
>
>
> /martin
>
> _______________________________________________
> netmod mailing list
> netmod@ietf.org
> https://www.ietf.org/mailman/listinfo/netmod
>
_______________________________________________
netmod mailing list
netmod@ietf.org
https://www.ietf.org/mailman/listinfo/netmod
