Re: [yang-doctors] Yangdoctors last call review of draft-ietf-ospf-yang-09

Christian Hopps <chopps@chopps.org> Wed, 18 April 2018 12:33 UTC

Return-Path: <chopps@chopps.org>
X-Original-To: yang-doctors@ietfa.amsl.com
Delivered-To: yang-doctors@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 7A654129C53 for <yang-doctors@ietfa.amsl.com>; Wed, 18 Apr 2018 05:33:11 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.9
X-Spam-Level:
X-Spam-Status: No, score=-1.9 tagged_above=-999 required=5 tests=[BAYES_00=-1.9] autolearn=ham autolearn_force=no
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id X_CiVFhxgE9p for <yang-doctors@ietfa.amsl.com>; Wed, 18 Apr 2018 05:33:08 -0700 (PDT)
Received: from smtp.chopps.org (smtp.chopps.org [54.88.81.56]) by ietfa.amsl.com (Postfix) with ESMTP id 857E512D778 for <yang-doctors@ietf.org>; Wed, 18 Apr 2018 05:33:08 -0700 (PDT)
Received: from tops.chopps.org (47-50-69-38.static.klmz.mi.charter.com [47.50.69.38]) (using TLSv1.2 with cipher AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by smtp.chopps.org (Postfix) with ESMTPSA id 39BBE61FE4; Wed, 18 Apr 2018 12:33:07 +0000 (UTC)
References: <87po6fmlkn.fsf@nic.cz> <CABCOCHSFfgjVS58S0po-9yEUZw=cjzG6k_LY8V9NSdy+kHQvLg@mail.gmail.com> <1516006155.5426.3.camel@nic.cz> <20180115.095941.341257692373089671.mbj@tail-f.com> <1516009099.5426.13.camel@nic.cz> <20180115095306.3m657hzwjemogqah@elstar.local> <1516011013.5426.25.camel@nic.cz> <20180115102009.aai3jetqsx3tgevw@elstar.local> <1516012128.5426.28.camel@nic.cz>
User-agent: mu4e 1.0; emacs 25.3.1
From: Christian Hopps <chopps@chopps.org>
To: Ladislav Lhotka <lhotka@nic.cz>
Cc: Juergen Schoenwaelder <j.schoenwaelder@jacobs-university.de>, yang-doctors@ietf.org
In-reply-to: <1516012128.5426.28.camel@nic.cz>
Date: Wed, 18 Apr 2018 08:33:06 -0400
Message-ID: <87in8obsot.fsf@chopps.org>
MIME-Version: 1.0
Content-Type: text/plain; format=flowed
Archived-At: <https://mailarchive.ietf.org/arch/msg/yang-doctors/Szo7Ki8rY-Lq6_rvM2h2Dk7vexA>
Subject: Re: [yang-doctors] Yangdoctors last call review of draft-ietf-ospf-yang-09
X-BeenThere: yang-doctors@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: Email list of the yang-doctors directorate <yang-doctors.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/yang-doctors>, <mailto:yang-doctors-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/yang-doctors/>
List-Post: <mailto:yang-doctors@ietf.org>
List-Help: <mailto:yang-doctors-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/yang-doctors>, <mailto:yang-doctors-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 18 Apr 2018 12:33:11 -0000

Ladislav Lhotka <lhotka@nic.cz>; writes:
> On Mon, 2018-01-15 at 11:20 +0100, Juergen Schoenwaelder wrote:
>> > > > This is IMO not true. As I already wrote, for using a when expression
>> > > > like
>> > > >
>> > > >    when "../type = 'ospf:ospfv3'"
>>
>> Fine, so this construction should be avoided since it is ambiguous.
>
> Yes, so I think the erratum should say just this. New modules should by all
> means use the XPath functions.

Can we add this to tooling (pyang) as a warning or an error?

Thanks,
Chris.