Re: [netmod] WG Last Call: draft-ietf-netmod-artwork-folding-02

"Rob Wilton (rwilton)" <rwilton@cisco.com> Tue, 28 May 2019 17:31 UTC

Return-Path: <rwilton@cisco.com>
X-Original-To: netmod@ietfa.amsl.com
Delivered-To: netmod@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id B755D120198; Tue, 28 May 2019 10:31:46 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -14.499
X-Spam-Level:
X-Spam-Status: No, score=-14.499 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_HI=-5, SPF_PASS=-0.001, URIBL_BLOCKED=0.001, USER_IN_DEF_DKIM_WL=-7.5] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=cisco.com header.b=E3yLM51z; dkim=pass (1024-bit key) header.d=cisco.onmicrosoft.com header.b=BiCBW57o
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 fiSGI8iuqvIc; Tue, 28 May 2019 10:31:44 -0700 (PDT)
Received: from rcdn-iport-1.cisco.com (rcdn-iport-1.cisco.com [173.37.86.72]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 16F22120048; Tue, 28 May 2019 10:31:43 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=27198; q=dns/txt; s=iport; t=1559064704; x=1560274304; h=from:to:cc:subject:date:message-id:references: in-reply-to:mime-version; bh=lHXnxajclDC/Vv5PoUIZj3yS7giq/XJHipVORw4WZws=; b=E3yLM51zU6wHHOaV/BiY2FDu86ovKD6M118fk4PqfBxRp6pr90X+g2In Zy7fMtZDMxr4m8dF7VlMP08gubPzo86PXjZNaOJW02uXkuKy+OqoMzQ1S ds/gc9Chk+Qh7Q6yBj27w28o7GdGxtN8u8LJoP4lEb0RINLQveQFgowIZ M=;
IronPort-PHdr: =?us-ascii?q?9a23=3A8QHojR9GQCRWTf9uRHGN82YQeigqvan1NQcJ65?= =?us-ascii?q?0hzqhDabmn44+8ZB7E/fs4iljPUM2b8P9Ch+fM+4HYEW0bqdfk0jgZdYBUER?= =?us-ascii?q?oMiMEYhQslVdSaCEnnK/jCZC0hF8MEX1hgrDm2?=
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: =?us-ascii?q?A0BxAAASb+1c/5FdJa1lHAEBAQQBAQc?= =?us-ascii?q?EAQGBUgYBAQsBgQ4vUANpVSAECygKhAmDRwOOfYJXlyuBLoEkA1QJAQEBDAE?= =?us-ascii?q?BLQIBAYRAAheCTCM1CA4BAwEBBAEBAgEEbRwMhUoBAQEBAxIRBAYTAQE3AQ8?= =?us-ascii?q?CAQYCEQQBASEKAgICMB0IAgQOBQgagwGBHU0DHQECjgOQYAKBOIhfcXwzgnk?= =?us-ascii?q?BAQWEfxiCDwmBNAGLUheBQD+BEUaBTn4+hEaDCDKCJoskIYJHhGOIJYxaaQk?= =?us-ascii?q?Cgg2TMJZJomYCBAIEBQIOAQEFgVEBNYFXcBWDJ4IPg3CKU3KBKYxGAYEgAQE?=
X-IronPort-AV: E=Sophos;i="5.60,523,1549929600"; d="scan'208,217";a="564332119"
Received: from rcdn-core-9.cisco.com ([173.37.93.145]) by rcdn-iport-1.cisco.com with ESMTP/TLS/DHE-RSA-SEED-SHA; 28 May 2019 17:31:38 +0000
Received: from XCH-ALN-007.cisco.com (xch-aln-007.cisco.com [173.36.7.17]) by rcdn-core-9.cisco.com (8.15.2/8.15.2) with ESMTPS id x4SHVcRd013363 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=FAIL); Tue, 28 May 2019 17:31:38 GMT
Received: from xhs-rcd-002.cisco.com (173.37.227.247) by XCH-ALN-007.cisco.com (173.36.7.17) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Tue, 28 May 2019 12:31:37 -0500
Received: from xhs-rcd-002.cisco.com (173.37.227.247) by xhs-rcd-002.cisco.com (173.37.227.247) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Tue, 28 May 2019 12:31:37 -0500
Received: from NAM03-DM3-obe.outbound.protection.outlook.com (72.163.14.9) by xhs-rcd-002.cisco.com (173.37.227.247) with Microsoft SMTP Server (TLS) id 15.0.1473.3 via Frontend Transport; Tue, 28 May 2019 12:31:37 -0500
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cisco.onmicrosoft.com; s=selector2-cisco-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=lHXnxajclDC/Vv5PoUIZj3yS7giq/XJHipVORw4WZws=; b=BiCBW57ok3i5AnRzMvt0GzBvSWB0K7fG2as+1VqJ3vF3K3Rn4P3qESNSidJ+B+o7MN6s364Z9fOPlZ81Vtw1RzgjWBKCb/NXlTf6uP7n9MBDrvuuMAk71YHCbnqHPN63m4D8lIfnkDRlstv4jwKWbbvJmcw3ouOvhW5vOc3rHpA=
Received: from BYAPR11MB2631.namprd11.prod.outlook.com (52.135.227.28) by BYAPR11MB2934.namprd11.prod.outlook.com (20.177.228.19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1922.18; Tue, 28 May 2019 17:31:35 +0000
Received: from BYAPR11MB2631.namprd11.prod.outlook.com ([fe80::d837:c1dd:cdb1:bb78]) by BYAPR11MB2631.namprd11.prod.outlook.com ([fe80::d837:c1dd:cdb1:bb78%7]) with mapi id 15.20.1922.021; Tue, 28 May 2019 17:31:35 +0000
From: "Rob Wilton (rwilton)" <rwilton@cisco.com>
To: Kent Watsen <kent@watsen.net>
CC: "netmod@ietf.org" <netmod@ietf.org>, "draft-ietf-netmod-artwork-folding@ietf.org" <draft-ietf-netmod-artwork-folding@ietf.org>
Thread-Topic: [netmod] WG Last Call: draft-ietf-netmod-artwork-folding-02
Thread-Index: AQHVCQh/oSnqrubDN0ieHZJH/cPGUKaAcPjQgABTS4CAAAF6YIAAE9eAgAAFKaA=
Date: Tue, 28 May 2019 17:31:35 +0000
Message-ID: <BYAPR11MB263188D9E6EAD9B54D915409B51E0@BYAPR11MB2631.namprd11.prod.outlook.com>
References: <e6fc4541-891a-60cb-e956-86f238d09f14@labn.net> <BYAPR11MB263112EA231A41491AC4DF89B51E0@BYAPR11MB2631.namprd11.prod.outlook.com> <0100016aff1640f0-a301a70e-ecae-4754-84d1-12170d5b73fd-000000@email.amazonses.com> <BYAPR11MB263191E71A0E09CE1667AB45B51E0@BYAPR11MB2631.namprd11.prod.outlook.com> <0100016aff5c1a19-32fa804b-b9d4-4411-a350-0a5571256e10-000000@email.amazonses.com>
In-Reply-To: <0100016aff5c1a19-32fa804b-b9d4-4411-a350-0a5571256e10-000000@email.amazonses.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: spf=none (sender IP is ) smtp.mailfrom=rwilton@cisco.com;
x-originating-ip: [173.38.220.56]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: ecae56c6-9186-489e-8258-08d6e3925526
x-microsoft-antispam: BCL:0; PCL:0; RULEID:(2390118)(7020095)(4652040)(8989299)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(5600148)(711020)(4605104)(1401327)(2017052603328)(7193020); SRVR:BYAPR11MB2934;
x-ms-traffictypediagnostic: BYAPR11MB2934:
x-ms-exchange-purlcount: 2
x-microsoft-antispam-prvs: <BYAPR11MB293429F8E603EBF82A49B853B51E0@BYAPR11MB2934.namprd11.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:10000;
x-forefront-prvs: 00514A2FE6
x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(366004)(39860400002)(376002)(136003)(346002)(396003)(189003)(199004)(51444003)(25786009)(7696005)(68736007)(81166006)(33656002)(8676002)(54906003)(74316002)(81156014)(99286004)(53546011)(8936002)(7736002)(66066001)(102836004)(6916009)(790700001)(76176011)(71200400001)(71190400001)(3846002)(6116002)(4326008)(6506007)(53936002)(2906002)(229853002)(55016002)(9686003)(6436002)(54896002)(6246003)(6306002)(256004)(446003)(476003)(76116006)(5660300002)(14454004)(11346002)(66946007)(66476007)(66446008)(64756008)(66556008)(73956011)(26005)(486006)(52536014)(86362001)(316002)(478600001)(186003); DIR:OUT; SFP:1101; SCL:1; SRVR:BYAPR11MB2934; H:BYAPR11MB2631.namprd11.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; MX:1; A:1;
received-spf: None (protection.outlook.com: cisco.com does not designate permitted sender hosts)
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam-message-info: J0u1erg8nf27RG+oWky5u0RHGc9UJfOIKeKaDdVUFXazjDsvgTuaPSzSn+QZWSI+jD42ONh2nKVBNRHwg/FnGk/aynJeeyv02BkS+DQ0/fQpstzAZ6DTBOMeTgwGugpwhBoksh0fCJOybLs+ClE5Qdi1GDxjZGsTIDHfjh/AqoubhitGrYr2C/Y+uOWtiNs3HObY0F7mXZtN/zhHu2/mi95nMgMqqVOuqKFJ/a2F9FQ+3ObwztgtXPr0tLNfY853muzwdHqOddGk1McPBkxUSy8LWhObhh18cF4cznVkZhIE3v6J+cH2gUIc35Mq888xUU3mNtksom2Vm2MJNI3XUn2kseYHfuTYOzqrNuMbcIGvYrOD+VqTOgLBYcy/6CdMH3Om7P7WBvaG2ozzkC+Ol0l6e8TPL7oY28SVWwy3s90=
Content-Type: multipart/alternative; boundary="_000_BYAPR11MB263188D9E6EAD9B54D915409B51E0BYAPR11MB2631namp_"
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: ecae56c6-9186-489e-8258-08d6e3925526
X-MS-Exchange-CrossTenant-originalarrivaltime: 28 May 2019 17:31:35.1082 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 5ae1af62-9505-4097-a69a-c1553ef7840e
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: rwilton@cisco.com
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BYAPR11MB2934
X-OriginatorOrg: cisco.com
X-Outbound-SMTP-Client: 173.36.7.17, xch-aln-007.cisco.com
X-Outbound-Node: rcdn-core-9.cisco.com
Archived-At: <https://mailarchive.ietf.org/arch/msg/netmod/DnMwM3B7pahQuBJ2g4IvaZqHBeY>
Subject: Re: [netmod] WG Last Call: draft-ietf-netmod-artwork-folding-02
X-BeenThere: netmod@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: NETMOD WG list <netmod.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/netmod>, <mailto:netmod-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/netmod/>
List-Post: <mailto:netmod@ietf.org>
List-Help: <mailto:netmod-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/netmod>, <mailto:netmod-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 28 May 2019 17:31:47 -0000


From: Kent Watsen <kent@watsen.net>;
Sent: 28 May 2019 17:53
To: Rob Wilton (rwilton) <rwilton@cisco.com>;
Cc: netmod@ietf.org; draft-ietf-netmod-artwork-folding@ietf.org
Subject: Re: [netmod] WG Last Call: draft-ietf-netmod-artwork-folding-02




[RW]
Yes, I think that is better, and probably OK.

I still slightly question “One strategy is based on the time-proven use of a single backslash ('\') character to indicate where line-folding has occurred, with the continuation occurring with the first non-space (' ') character on the next line.”  Because I don’t think that is how ‘\’ character works, at least in languages such as C.  Specifically, it doesn’t ignore leading whitespace on the following line, instead it is often used where that whitespace is not significant to the compiler.

Would s/time-proven/POSIX/ be better?
[RW]
Perhaps, less is more, e.g.,

“One strategy is based on the historical use of a single backslash ('\') character to indicate where line-folding has occurred.”?

BTW, I also added this to Appendix A:

   Shell-level end-of-line backslash ('\') characters have been
   purposely added to the script so as to ensure that the script is
   itself not folded in this document, thus simplify the ability to
   copy/paste the script for local use.  As should be evident by the
   lack of the mandatory header described in Section 7.1.1, these
   backslashes do not designate a folded line, such as described in
   Section 7.


[RW]
OK.




[RW]
Perhaps “original text content” -> “exact original text content”?  But I’m also OK with your suggested text.

I'm hesitant, because it seems redundant, but it doesn't cause harm, so I added it.



[RW]
According to RFC2119, RECOMMENDED is interpreted exactly the same way as SHOULD.

Yes, when composing my response before I was going to say that it's a downgrade "(in IMO)", but figured it would require more explanation, which I was hoping to avoid.  But here we are now  ;)   While I'm aware that they carry the same RFC 2119 weight, RECOMMENDED reads softer to me, less commanding, hence my comment.
[RW]
Er, I strongly disagree that RECOMMENDED has any different semantic meaning to SHOULD.  RFC 2119 is explicit that they mean exactly the same thing.  If you want it to be softer, then perhaps change it to “recommended” which I do think is softer than “should”. 😊

 I still think that SHOULD/RECOMMENDED is too strong.

I still disagree.    Any tie-breakers out there?

[RW]
So, for example considering the current text, if it was a JSON input file, it would be more appropriate to change the indentation level to 2 spaces rather than 4 spaces to ensure that there are not unnecessary line wraps?  In theory, the only place in a JSON file that we would expect there to be a line breaks would be within a string because that is the only case that can’t be handled within JSON itself.

Is this your intended meaning?

Good point, how about this?

   Scan the text content to ensure no existing lines already end with a
   backslash ('\') character while the subsequent line starts with a
   backslash ('\') character as the first non-space (' ') character, as
   this could lead to an ambiguous result.  If such a line is found, and
   its width is less than the desired maximum, then it SHOULD be flagged
   for forced folding (folding even though unnecessary).  If the folding
   implementation doesn't support forced foldings, it MUST exit.

   <snip>

   For each line in the text content, from top-to-bottom, if the line
   exceeds the desired maximum, or requires a forced folding, then fold
   the line by:


[RW]
OK.

Great.  BTW, I also added this to Appendix A:

   This script does not implement the "forced folding" logic described
   in Section 8.2.1.  In such cases the script will exit with the
   message:

         Error: infile has a line ending with a '\\' character
         followed by a '\\' character as the first non-space
         character on the next line.  This file cannot be folded.


[RW]
Sure.

Thanks,
Rob


Kent // author