Re: [TOOLS-DEVELOPMENT] Preview release of Text Submission Converter, id2xml

Henrik Levkowetz <henrik@levkowetz.com> Tue, 19 December 2017 22:40 UTC

Return-Path: <henrik@levkowetz.com>
X-Original-To: tools-development@ietfa.amsl.com
Delivered-To: tools-development@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 6840812D879 for <tools-development@ietfa.amsl.com>; Tue, 19 Dec 2017 14:40:12 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.899
X-Spam-Level:
X-Spam-Status: No, score=-1.899 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, URIBL_BLOCKED=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 sZtkcfq6cZTL for <tools-development@ietfa.amsl.com>; Tue, 19 Dec 2017 14:40:10 -0800 (PST)
Received: from zinfandel.tools.ietf.org (zinfandel.tools.ietf.org [IPv6:2001:1890:126c::1:2a]) (using TLSv1.2 with cipher DHE-RSA-AES128-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 2A8B512D877 for <tools-development@ietf.org>; Tue, 19 Dec 2017 14:40:04 -0800 (PST)
Received: from h-99-61.a357.priv.bahnhof.se ([82.196.99.61]:54375 helo=[192.168.1.120]) by zinfandel.tools.ietf.org with esmtpsa (TLS1.2:DHE_RSA_AES_128_CBC_SHA1:128) (Exim 4.80) (envelope-from <henrik@levkowetz.com>) id 1eRQYN-0002qf-7O; Tue, 19 Dec 2017 14:40:04 -0800
To: Megan Ferguson <mferguson@amsl.com>
References: <ED7FD9D2-169F-4329-B883-E8455938E936@amsl.com>
Cc: tools-development@ietf.org
From: Henrik Levkowetz <henrik@levkowetz.com>
Message-ID: <8e4c832a-5b71-f94d-70a5-e1259f4912a7@levkowetz.com>
Date: Tue, 19 Dec 2017 23:39:54 +0100
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.11; rv:45.0) Gecko/20100101 Thunderbird/45.8.0
MIME-Version: 1.0
In-Reply-To: <ED7FD9D2-169F-4329-B883-E8455938E936@amsl.com>
Content-Type: multipart/signed; micalg="pgp-sha256"; protocol="application/pgp-signature"; boundary="dflTWR0adgFViM37aPqpOAAdJ3VGXiHnp"
X-SA-Exim-Connect-IP: 82.196.99.61
X-SA-Exim-Rcpt-To: tools-development@ietf.org, mferguson@amsl.com
X-SA-Exim-Mail-From: henrik@levkowetz.com
X-SA-Exim-Version: 4.2.1 (built Mon, 26 Dec 2011 16:24:06 +0000)
X-SA-Exim-Scanned: Yes (on zinfandel.tools.ietf.org)
X-Clacks-Overhead: GNU Terry Pratchett
Archived-At: <https://mailarchive.ietf.org/arch/msg/tools-development/OfConkSGKhBAYUI-1rhuN87DHOk>
Subject: Re: [TOOLS-DEVELOPMENT] Preview release of Text Submission Converter, id2xml
X-BeenThere: tools-development@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: Tools Development list server <tools-development.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/tools-development>, <mailto:tools-development-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/tools-development/>
List-Post: <mailto:tools-development@ietf.org>
List-Help: <mailto:tools-development-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/tools-development>, <mailto:tools-development-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 19 Dec 2017 22:40:12 -0000

Hi Megan,

On 2017-12-19 22:47, Megan Ferguson wrote:
> Hi Henrik,
> 
> Input file: draft-ietf-curdle-ssh-ext-info-15
> id2xml version: 1.4.0
> Issue: “Updates" truncation
> Level: Minor
> Files available: 
> https://www.rfc-editor.org/rfc/v3test/draft-ietf-curdle-ssh-ext-info-15.original
> https://www.rfc-editor.org/rfc/v3test/draft-ietf-curdle-ssh-ext-info-15.original.xml
> 
> Just an FYI that it looks like there is some trouble translating the
> “Updates” field when it contains a comma-separated list.

Huh.  That's silly and unnecessary.  The tool should handle that better.

I've fixed this in my sources; it will be part of the next release.

Thanks for the alert!


Best regards,

	Henrik

> 
> From .original
> Updates: 4252, 4253, 4254 (if approved) 
> 
> From .xml
> <rfc submissionType=“IETF" docName="draft-ietf-curdle-ssh-ext-info-15.txt" category="std" updates="4254”>
> 
> [Note - this also occurs if there is an “and” before the final value or if there is a different number of updated RFCs.]
> 
> When we removed the commas (as in the following), the xml output was correct:
> 
> Updates: 4252 4253 4254 (if approved)
> or
> Updates: 4252 4253 and 4254 (if approved)
> 
> Translated in xml to:
> 
> <rfc submissionType="IETF" docName="draft-ietf-curdle-ssh-ext-info-15.txt" category="std" updates="4252, 4253, 4254”>
> 
> Note - we get correct output for each of the following:
> 
> Updates: 4252 and 4253
> Updates: 4252 & 4253
> 
> We will continue to manually update the comma-separated versions unless we hear from you that it can be addressed with the tool.
> 
> Happy holidays!
> 
> Megan
> 
> 
> 
> 
>