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, 01 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~~~~~~~~~~~~~~~~~~~~~~~~~ \ ( ( ) \_) ) / (_/
- [apps-discuss] Appsdir review of XML schema in ht… Henry S. Thompson
- [apps-discuss] Appsdir review of XML schema in ht… Henry S. Thompson
- Re: [apps-discuss] Appsdir review of XML schema i… Eliot Lear
- Re: [apps-discuss] Appsdir review of XML schema i… Roni Even
- Re: [apps-discuss] Appsdir review of XML schema i… Simon Pietro Romano
- Re: [apps-discuss] Appsdir review of XML schema i… Henry S. Thompson