Re: [bfcpbis] Draft new version: draft-ietf-bfcpbis-rfc4583bis-25

Paul Kyzivat <pkyzivat@alum.mit.edu> Mon, 01 October 2018 15:55 UTC

Return-Path: <pkyzivat@alum.mit.edu>
X-Original-To: bfcpbis@ietfa.amsl.com
Delivered-To: bfcpbis@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id A4705130E08 for <bfcpbis@ietfa.amsl.com>; Mon, 1 Oct 2018 08:55:01 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.201
X-Spam-Level:
X-Spam-Status: No, score=-4.201 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3, SPF_PASS=-0.001] 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 aBOvGjrGgMuc for <bfcpbis@ietfa.amsl.com>; Mon, 1 Oct 2018 08:55:00 -0700 (PDT)
Received: from alum-mailsec-scanner-7.mit.edu (alum-mailsec-scanner-7.mit.edu [18.7.68.19]) by ietfa.amsl.com (Postfix) with ESMTP id 0FB75130DFE for <bfcpbis@ietf.org>; Mon, 1 Oct 2018 08:54:59 -0700 (PDT)
X-AuditID: 12074413-125ff70000000b2e-cf-5bb24353812d
Received: from outgoing-alum.mit.edu (OUTGOING-ALUM.MIT.EDU [18.7.68.33]) (using TLS with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by alum-mailsec-scanner-7.mit.edu (Symantec Messaging Gateway) with SMTP id C2.7A.02862.35342BB5; Mon, 1 Oct 2018 11:54:59 -0400 (EDT)
Received: from PaulKyzivatsMBP.localdomain (c-24-62-227-142.hsd1.ma.comcast.net [24.62.227.142]) (authenticated bits=0) (User authenticated as pkyzivat@ALUM.MIT.EDU) by outgoing-alum.mit.edu (8.13.8/8.12.4) with ESMTP id w91FsvVJ016548 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES128-SHA bits=128 verify=NOT); Mon, 1 Oct 2018 11:54:58 -0400
To: Christer Holmberg <christer.holmberg@ericsson.com>, "bfcpbis@ietf.org" <bfcpbis@ietf.org>
References: <1B7CCF77-8173-44B3-8321-3A761C53AD43@ericsson.com> <3D819DDF-D384-4495-8C9E-3A784B335EA8@cisco.com> <CAHBDyN5CicKHunbgghrEwVnN1yxHzC+7X6fdVWTTdR1c5mxxQQ@mail.gmail.com> <66a5f9ad-7f8d-4655-1f27-5a6d6fd03088@alum.mit.edu> <DFE8413F-17F6-4331-99A1-687559B2698B@ericsson.com>
From: Paul Kyzivat <pkyzivat@alum.mit.edu>
Message-ID: <74b552e6-95ea-e081-bc67-1cce5f430c77@alum.mit.edu>
Date: Mon, 1 Oct 2018 11:54:57 -0400
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:52.0) Gecko/20100101 Thunderbird/52.9.1
MIME-Version: 1.0
In-Reply-To: <DFE8413F-17F6-4331-99A1-687559B2698B@ericsson.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFmplleLIzCtJLcpLzFFi42IRYndR1A123hRtsPEmn8W/dUeZLC7MPMzo wOTx6+tVNo8lS34yBTBFcdmkpOZklqUW6dslcGV8O/GJqeAVe8Wlc/fZGxhPsnYxcnJICJhI PLuxgrGLkYtDSGAHk8TJX9NYIJwHTBIzGs4xg1QJC3hJrG87C9YhIpAm8fHCfqiOpUwS25/+ ZARJsAloScw59J8FxOYVsJc4OP0TWJxFQEXizqs7TCC2KFDz384ljBA1ghInZz4Bqufg4BRw kLi9PBIkzCxgJjFv80NmCFtc4taT+UwQtrzE9rdzmCcw8s9C0j0LScssJC2zkLQsYGRZxSiX mFOaq5ubmJlTnJqsW5ycmJeXWqRrrpebWaKXmlK6iRESqsI7GHedlDvEKMDBqMTDyyC/KVqI NbGsuDL3EKMkB5OSKK+eGFCILyk/pTIjsTgjvqg0J7X4EKMEB7OSCO9OkHLelMTKqtSifJiU NAeLkjgvs8neKCGB9MSS1OzU1ILUIpisDAeHkgRvgRNQo2BRanpqRVpmTglCmomDE2Q4D9Dw aSA1vMUFibnFmekQ+VOMlhx7vjbNYOZ4MaMDSLY9vT6DWYglLz8vVUqc1wCkQQCkIaM0D24m LPW8YhQHelGYlxGkigeYtuCmvgJayAS0sLF0A8jCkkSElFQD41rx//GTf757ZX/jn9IWmbgZ x5wlThz4YqrAHWH65EPespAXmafZrCxvNn59YKnbncJ141vdY45EObFjZldsTujwKtV2bstu lDNbXvH+77beK/L6giWzZnXel1daJ5+xNPe06tbr94qSxMIXrPebse/thBn2sYI/J+70u/x0 VsW0nXWi99m9rJVYijMSDbWYi4oTAZib7PQYAwAA
Archived-At: <https://mailarchive.ietf.org/arch/msg/bfcpbis/GPePgfg7u9-aFwFZbsklyCl3ij4>
Subject: Re: [bfcpbis] Draft new version: draft-ietf-bfcpbis-rfc4583bis-25
X-BeenThere: bfcpbis@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: BFCPBIS working group discussion list <bfcpbis.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/bfcpbis>, <mailto:bfcpbis-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/bfcpbis/>
List-Post: <mailto:bfcpbis@ietf.org>
List-Help: <mailto:bfcpbis-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/bfcpbis>, <mailto:bfcpbis-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 01 Oct 2018 15:55:02 -0000

On 10/1/18 4:15 AM, Christer Holmberg wrote:
> Hi Paul,
> 
> Thanks for your comments! Please see inline.
> 
> ---
>      
>>     * General comment:
>>     
>>     I have a slight preference for doing references to rules defined in
>>     other documents differently. E.g.:
>>     
>>         DIGIT = <DIGIT defined in [RFC5234]>
>>     
>>     rather than:
>>     
>>         ;DIGIT is defined in [RFC5234]
>>     
>>     The advantage is that when mechanically verifying the SDP, the comment
>>     form results in an undefined symbol while the <> form doesn't. (No, I
>>     didn't mechanically verify the SDP this time.)
>    
> Will fix as suggested. I did get some xml2rfc problems with the '<' and '>' characters, though, so for now I did:
>   
>    DIGIT = DIGIT defined in [RFC5234]

The <> is explicit abnf syntax, and is the important part.
You can fix this using &lt &gt, or using <![CDATA[...]]>.

Personally I always use CDATA for abnf.

	Thanks,
	Paul