Juergen Schoenwaelder <[email protected]> wrote:
> Hi,
> 
> the final check of the YANG 1.1 WG last call edits has passed and so
> we are getting ready to wrap things up. While working on the document
> writeup that goes to the IESG, I found a quirks that I think we should
> fix before sending the document to Benoit:
> 
> 1) ID nits identified two instances of 'MUST not' that should be
>    'MUST NOT'.

Fixed.

> 2) The capability URN registration requires to define a capability
>    name. Right now we only provide a URL. The IANA considerations
>    need to define the capability name, e.g. :yang-library.

Fixed.

> 3) I checked the ABNF against http://tools.ietf.org/tools/bap/abnf.cgi.
>    Note that I had to replace '%s' with '' since this tool does not yet
>    support RFC 7405 (case sensitive strings). Anyway, the tool does a
>    good job and suggested these two changes:
> 
>    - yang-char = %x9 / %xA / %xD / %x20-D7FF /
>    + yang-char = %x09 / %x0A / %x0D / %x20-D7FF /
> 
>    - relative-path       = 1*(".." "/") descendant-path
>    + relative-path       = 1*"../" descendant-path

Fixed.

>    In addition, the tool suggested to add a couple of parenthesis so
>    that the reader does not have to remember operator precedences by
>    heart. I think Martin should take a look at this and decide which
>    ones to incorporate (or if there is even an error lingering
>    somewhere).

I checked this, but I prefer to keep the current rules.

> In addition, there was the suggestion on the mailing list to mention
> the change of the non-config leaf-lists uniqueness in the 'Summary of
> Changes from RFC 6020' which I think should be implemented.

Fixed.

> Martin, can you look at these issues and post a new I-D? Once
> available, I will check the diffs, finish my writeup and then submit
> the document to Benoit.

There is one more open issue; see the thread "Multiple deviations with
same target".


/martin

_______________________________________________
netmod mailing list
[email protected]
https://www.ietf.org/mailman/listinfo/netmod

Reply via email to