[netmod] Benjamin Kaduk's Discuss on draft-ietf-netmod-artwork-folding-11: (with DISCUSS and COMMENT)
Benjamin Kaduk via Datatracker <noreply@ietf.org> Thu, 09 January 2020 23:28 UTC
Return-Path: <noreply@ietf.org>
X-Original-To: netmod@ietf.org
Delivered-To: netmod@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 4DFF412003E; Thu, 9 Jan 2020 15:28:30 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Benjamin Kaduk via Datatracker <noreply@ietf.org>
To: The IESG <iesg@ietf.org>
Cc: draft-ietf-netmod-artwork-folding@ietf.org, Lou Berger <lberger@labn.net>, netmod-chairs@ietf.org, lberger@labn.net, netmod@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.115.0
Auto-Submitted: auto-generated
Precedence: bulk
Reply-To: Benjamin Kaduk <kaduk@mit.edu>
Message-ID: <157861251031.11821.9719140738474484600.idtracker@ietfa.amsl.com>
Date: Thu, 09 Jan 2020 15:28:30 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/netmod/1fYZVgwskR3pH2cYzpWAH8lI0to>
Subject: [netmod] Benjamin Kaduk's Discuss on draft-ietf-netmod-artwork-folding-11: (with DISCUSS and COMMENT)
X-BeenThere: netmod@ietf.org
X-Mailman-Version: 2.1.29
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: Thu, 09 Jan 2020 23:28:30 -0000
Benjamin Kaduk has entered the following ballot position for draft-ietf-netmod-artwork-folding-11: Discuss When responding, please keep the subject line intact and reply to all email addresses included in the To and CC lines. (Feel free to cut this introductory paragraph, however.) Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html for more information about IESG DISCUSS and COMMENT positions. The document, along with other ballot positions, can be found here: https://datatracker.ietf.org/doc/draft-ietf-netmod-artwork-folding/ ---------------------------------------------------------------------- DISCUSS: ---------------------------------------------------------------------- Thank you for the updates in the -10 and -11; the content looks a lot better and I am not uncomfortable about publishing as Informational (vs. BCP)! That said, I think the edits to the script have introduced a regression: # ensure input file doesn't contain the fold-sequence already if [[ -n "$("$SED" -n '/\\$/{N;s/\\\n[ ]*\\/&/p}' "$infile")" ]] Unfortunately, I'm not sure this gets all cases, since the 'N' command reads a line and prevents it from being considered as the first half of the wrapped sequence: kaduk$:~/git/openssl$ cat /tmp/a this is a line\ another line\ \that wraps kaduk$:~/git/openssl$ cat /tmp/b this is a line another line\ \that wraps kaduk$:~/git/openssl$ sed -n '/\\$/{N;s/\\\n[ ]*\\/&/p}' < /tmp/a kaduk$:~/git/openssl$ sed -n '/\\$/{N;s/\\\n[ ]*\\/&/p}' < /tmp/b another line\ \that wraps ---------------------------------------------------------------------- COMMENT: ---------------------------------------------------------------------- A few other comments from reviewing the version of the script in the -11: When processing input, it's perhaps more robust to check $# before assigning $2 to a named parameter. printf "Exit status code: 1 on error, 0 on success, 255 on no-op." Interesting to have no newline here but two on the next line's printf, but I guess it might be at the column limit already. (The quotes on 'Error'/'Warning'/'Debug' in err()/warn()/dbg() are noops.) # warn if a non-GNU sed utility is used "$SED" --version < /dev/null 2> /dev/null \ | grep GNU > /dev/null 2>&1 || \ `grep -q` should be usable instead of `grep >/dev/null`
- [netmod] Benjamin Kaduk's Discuss on draft-ietf-n… Benjamin Kaduk via Datatracker
- Re: [netmod] Benjamin Kaduk's Discuss on draft-ie… Erik Auerswald
- Re: [netmod] Benjamin Kaduk's Discuss on draft-ie… Kent Watsen
- Re: [netmod] Benjamin Kaduk's Discuss on draft-ie… Benjamin Kaduk
- Re: [netmod] Benjamin Kaduk's Discuss on draft-ie… Kent Watsen