Re: [apps-discuss] Appsdir review of XML schema in https://tools.ietf.org/html/draft-ietf-clue-data-model-schema-14

Simon Pietro Romano <spromano@unina.it> Tue, 31 May 2016 22:36 UTC

Return-Path: <spromano@unina.it>
X-Original-To: apps-discuss@ietfa.amsl.com
Delivered-To: apps-discuss@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 5065612D1E5 for <apps-discuss@ietfa.amsl.com>; Tue, 31 May 2016 15:36:27 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -3.326
X-Spam-Level:
X-Spam-Status: No, score=-3.326 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, RP_MATCHES_RCVD=-1.426, SPF_PASS=-0.001] autolearn=unavailable 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 5TvvRMxuFGAG for <apps-discuss@ietfa.amsl.com>; Tue, 31 May 2016 15:36:25 -0700 (PDT)
Received: from brc2.unina.it (brc2.unina.it [192.132.34.42]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 791B312D634 for <apps-discuss@ietf.org>; Tue, 31 May 2016 15:36:19 -0700 (PDT)
X-ASG-Debug-ID: 1464733558-05f275673b3a6b0001-yjSrr1
Received: from smtp2.unina.it (smtp2.unina.it [192.132.34.62]) by brc2.unina.it with ESMTP id kXGCs2BPpaax2yO7 (version=TLSv1 cipher=AES256-SHA bits=256 verify=NO); Wed, 01 Jun 2016 00:25:58 +0200 (CEST)
X-Barracuda-Envelope-From: spromano@unina.it
X-Barracuda-Apparent-Source-IP: 192.132.34.62
Received: from [192.168.178.20] ([151.70.36.98]) (authenticated bits=0) by smtp2.unina.it (8.14.4/8.14.4) with ESMTP id u4VMPvZL019594 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Wed, 1 Jun 2016 00:25:58 +0200
Content-Type: multipart/alternative; boundary="Apple-Mail=_CDA95F3B-51FE-4EC2-8FDB-695646C9AE08"
Mime-Version: 1.0 (Mac OS X Mail 8.2 \(2104\))
From: Simon Pietro Romano <spromano@unina.it>
X-ASG-Orig-Subj: Re: Appsdir review of XML schema in https://tools.ietf.org/html/draft-ietf-clue-data-model-schema-14
In-Reply-To: <f5bwpn0egdz.fsf@troutbeck.inf.ed.ac.uk>
Date: Wed, 1 Jun 2016 00:25:56 +0200
Message-Id: <BB5F871F-C482-458A-B863-3F730FF3AF53@unina.it>
References: <f5bwpn0egdz.fsf@troutbeck.inf.ed.ac.uk>
To: "Henry S. Thompson" <ht@inf.ed.ac.uk>
X-Mailer: Apple Mail (2.2104)
X-Barracuda-Connect: smtp2.unina.it[192.132.34.62]
X-Barracuda-Start-Time: 1464733558
X-Barracuda-Encrypted: AES256-SHA
X-Barracuda-URL: http://192.132.34.42:8000/cgi-mod/mark.cgi
X-Virus-Scanned: by bsmtpd at unina.it
X-Barracuda-BRTS-Status: 1
X-Barracuda-Spam-Score: 0.00
X-Barracuda-Spam-Status: No, SCORE=0.00 using global scores of TAG_LEVEL=1000.0 QUARANTINE_LEVEL=1000.0 KILL_LEVEL=6.0 tests=BSF_SC0_MISMATCH_TO, HTML_MESSAGE
X-Barracuda-Spam-Report: Code version 3.2, rules version 3.2.3.30065 Rule breakdown below pts rule name description ---- ---------------------- -------------------------------------------------- 0.00 BSF_SC0_MISMATCH_TO Envelope rcpt doesn't match header 0.00 HTML_MESSAGE BODY: HTML included in message
Archived-At: <http://mailarchive.ietf.org/arch/msg/apps-discuss/-H-mirwqK9ZwQ2POrJBR7xs6y9Q>
Cc: draft-ietf-clue-data-model-schema.all@ietf.org, iesg@ietf.org, apps-discuss@ietf.org
Subject: Re: [apps-discuss] Appsdir review of XML schema in https://tools.ietf.org/html/draft-ietf-clue-data-model-schema-14
X-BeenThere: apps-discuss@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: General discussion of application-layer protocols <apps-discuss.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/apps-discuss>, <mailto:apps-discuss-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/apps-discuss/>
List-Post: <mailto:apps-discuss@ietf.org>
List-Help: <mailto:apps-discuss-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/apps-discuss>, <mailto:apps-discuss-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 31 May 2016 22:36:27 -0000

Hello Henry,

let us thank you for your thorough review! Please find in-line (preceded by [spromano]) our answers to your comments.

> Document: draft-ietf-clue-data-model-schema
> Title: An XML Schema for the CLUE data model
> Reviewer: Henry S. Thompson
> Review Date: 2016-05-11
> IETF Last Call Date: 2016-05-23
> ESG Telechat Date: 2016-06-02
> 
> I have been selected as the Applications Area Directorate reviewer for
> this draft (for background on appsdir, please see
> http://trac.tools.ietf.org/area/app/trac/wiki/ApplicationsAreaDirectorate).
> 
> Please resolve these comments along with any other Last Call comments
> you may receive. Please wait for direction from your document shepherd
> or AD before posting a new version of the draft.
> 
> Summary: The XML Schema itself which is included in this draft is
> conformant to the XML Schema 1.0 spec, and is good to go, subject to a
> minor correction.  There is a minor glitch in one of the XML examples,
> also easily corrected.

[spromano] this is good news. Thank you!

> Comments:
> 
> I tested the XML Schema document included as section 4 and it passes as
> valid against the schema for schemas.  The example document in section
> 17 is schema-valid according to the corresponding schema.  See 'Nits'
> below for a minor problem with the example document in section 18.
> 
> I briefly reviewed the schema document and it seems straightforward and
> fit for its intended purpose.
> 
> Minor Issues:
> 
> Section 4, lines 13--14:
> 
> <xs:import namespace="urn:ietf:params:xml:ns:vcard-4.0"
> schemaLocation="xcard.xsd"/>
> 
> As this stands, it's not actually usable for validation purposes,
> because no xcard.xsd file is supplied.  Furthermore, there is no
> Appendix A, which is alleged to provide it (see 11.29.1.2).
> 
> I note further that the xCard RFC (6351) doesn't contain an XSD-format
> schema document either.  The IANA XML Registry [1] schema entry for the
> urn:ietf:params:xml:ns:vcard-4.0 URN namespace _does_ however link to
> such a schema document [2].  I suggest you either
>  a) Edit the draft so the above lines read
> 
> <xs:import namespace="urn:ietf:params:xml:ns:vcard-4.0"
> schemaLocation="http://www.iana.org/assignments/xml-registry/schema/vcard-4.0.xsd"/>
> 
>  (this is what I did to do the validity checks I did);

[spromano] Done.

> or
> 
> b) Delete the schemaLocation attribute and add a comment identifying
>    possible sources of a schema document for the vcard-4.0 namespace,
>    e.g. the above iana.org URI or the contents of Appendix A (if you
>    fill it in).

[spromano] See above (we chose the former option…).

> Nits:
> 
> Section 18, the XML example has a (copy-paste?) error, which renders
> that example invalid against the schema.  The line
> 
>             <encGroupIDREF>EG0</encGroupIDREF>
> 
> appears twice inside mediaCapture VC7, once on line 204 and once on line
> 237.  The first occurrence should be deleted.

[spromano] You are referring to VC4, right? If so, the issue has been solved. Good catch!

Thank you once again,

Simon and Roberta

> 
> [1] http://www.iana.org/assignments/xml-registry/xml-registry.xhtml#schema
> [2] http://www.iana.org/assignments/xml-registry/schema/vcard-4.0.xsd
> -- 
>       Henry S. Thompson, School of Informatics, University of Edinburgh
>      10 Crichton Street, Edinburgh EH8 9AB, SCOTLAND -- (44) 131 650-4440
>                Fax: (44) 131 650-4587, e-mail: ht@inf.ed.ac.uk
>                       URL: http://www.ltg.ed.ac.uk/~ht/
> [mail from me _always_ has a .sig like this -- mail without it is forged spam]
> 

                     					       _\\|//_
                           				      ( O-O )
   ~~~~~~~~~~~~~~~~~~~~~~o00~~(_)~~00o~~~~~~~~~~~~~~~~~~~~~~~~
                    				Simon Pietro Romano
             				 Universita' di Napoli Federico II
                		     Computer Engineering Department 
	             Phone: +39 081 7683823 -- Fax: +39 081 7683816
                                           e-mail: spromano@unina.it

		    <<Molti mi dicono che lo scoraggiamento è l'alibi degli 
		    idioti. Ci rifletto un istante; e mi scoraggio>>. Magritte.
               			                     oooO
  ~~~~~~~~~~~~~~~~~~~~~~~(   )~~~ Oooo~~~~~~~~~~~~~~~~~~~~~~~~~
					                 \ (            (   )
			                                  \_)          ) /
                                                                       (_/