[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]
Subject: Re: [sca-bindings] WS Binding CD02 rev1
Simon Nash wrote: > Anish, > Here are my comments on the latest draft. > > 1. The colon following [BWS40005] in section 4.2.2 is not correct and > should be changed to a period. Also, [BWS40006] and [BRS40007] > should not be a bulleted list. > > The default transport binding rules are defined in the previous > bulleted list at the start of section 4.2.2. This non-normative > definition is followed by three normative statements BWS40005, > BWS40006 and BWS40007 that refer back to the previous definition. > These three normative statements are independent of each other and > should not be linked together, either by a colon or by bulleted > indentation. > Fixed in CD02-rev3 (which isn't posted yet, but will be later). > 2. The text for BWS20018 refers to the "semantics of this > attribute". When reading the standalone conformance statement in > Appendix B, it isn't clear which attribute is intended. This text > should read "semantics of the @wsdli:wsdlLocation attribute". > BWS20018 says: "The semantics of this attribute are specified in Section 7.1 of WSDL 2.0 [WSDL20]. The value of the @wsdli:wsdlLocation attribute MUST identify an existing WSDL 1.1 document." Going by our conventions, I don't think the 1st sentence should be part of BWS20018. I would like to suggest the following: Leave the 1st statement as is, but remove it from BWS20018. BWS20018 would now read -- "The value of the @wsdli:wsdlLocation attribute MUST identify an existing WSDL 1.1 document." I'll make this change in CD02-rev3, please let me know if you disagree. -Anish -- > Simon > > Anish Karmarkar wrote: >> Created CD02-rev2 by making the ed changes pointed out by SimonN as >> well as reversed the way the 2119 keyword stmt bookmarks/crossrefs >> worked to match that of assembly (was a major PITA to change it). I >> also did some formating/ed changes and fixed a few refs/crossrefs. >> More detailed comments inlined below. >> >> CD02-rev2 (Word & PDF): >> http://www.oasis-open.org/apps/org/workgroup/sca-bindings/download.php/32884/sca-binding-ws-1.1-spec-cd02-rev2.doc >> >> http://www.oasis-open.org/apps/org/workgroup/sca-bindings/download.php/32885/sca-binding-ws-1.1-spec-cd02-rev2.pdf >> >> >> Diff between CD02-rev1 and CD02-rev2: >> http://www.oasis-open.org/apps/org/workgroup/sca-bindings/download.php/32886/sca-binding-ws-1.1-spec-cd02-rev1-rev2-diff.pdf >> >> >> Diff between CD02 and CD02-rev2: >> http://www.oasis-open.org/apps/org/workgroup/sca-bindings/download.php/32887/sca-binding-ws-1.1-spec-cd02-rev2-diff.pdf >> >> >> -Anish >> -- >> >> Simon Nash wrote: >>> Anish, >>> Thanks for your work on this. Here are my comments. >>> >>> Major issue: >>> >>> All cross-references for conformance statements and tags are wrong. >>> >>> The bookmark R_BWSxxxxx should be the [BWSxxxxx] tag in the main >>> chapter text, with a cross-reference in the "Conformance ID" column >>> of Appendix B. Instead this bookmark is the main chapter text >>> describing BWSxxxxx, with a cross-reference in the "Description" >>> column of Appendix B. >>> >>> The bookmark BWSxxxxx should be the text describing BWSxxxxx in the >>> "Description" column of Appendix B, with a cross-reference preceding >>> the >>> [BWSxxxxx] tag in the main chapter text. Instead this bookmark is the >>> [BWSxxxxx] tag in the "Conformance ID" column of Appendix B, with a >>> cross-reference following the main chapter text describing BWSxxxxx. >>> >>> Editorial comments: >>> >>> 1. In front matter "Related Work", change "supercedes" to "supersedes". >>> >> >> Done. >> >>> 2. In front matter "Abstract", fix typo "of an SCA composites". >>> >> >> Done. >> >>> 3. In front matter "Notices" last paragraph, fix typo "names OASIS is". >>> >> >> Done. >> >>> 4. In section 1.1, change [[RCF2119]] to [RFC2119]. >>> >> >> Done. >> >>> 5. In section 1.4, use Word cross-reference for [SCA-Assembly]. >>> >> >> Done. >> >>> 6. In section 2 bulleted list, there are 3 cross-references to >>> [SCA-Assembly] under /binding.ws/wsdlElement which should be >>> Word cross-references. >>> >> >> Done. >> >>> 7. In section 2 bulleted list, /binding.ws/wireFormat and >>> /binding.ws/operationSelector should be in bold italic font. >>> >> >> Done. >> >>> 8. In section 2 bulleted list, cross-references to [SCA-Assembly] >>> in /binding.ws/wireFormat and /binding.ws/operationSelector >>> should be Word cross-references. >>> >> >> Done. >> >>> 9. In section 2.3, the Word cross-reference to [SCA-JCAA] points >>> to [SCA-Policy]. >>> >> >> Done. >> >>> 10. In section 4.1, the cross-reference (R2705) should use square >>> brackets instead of parentheses. Also, it should point to a >>> reference in section 1.2 instead of directly to a section of >>> http://www.ws-i.org/Profiles/BasicProfile-1.2.html (which is >>> not one of the documents listed in section 1.2). >>> >> >> Fixed to point to the reference in section 1.2. >> >> >>> 11. In section 4.2.1, conformance statement [BWS40004], fix >>> typo "from the that interface". >>> >> >> Done. >> >>> 12. In section 4.2.2, delete the colon after [BWS40005]. >>> >> >> Colon is necessary (per the issue resolution). >> >>> 13. In section 4.2.2, the paragraphs containing conformance statments >>> [BWS40006] and [BWS40007] should not be indented. >>> >> >> They have to be indented. It is supposed to be a list. Fixed. >> >>> 14. In section 4.2.2, conformance statement [BWS40007], fix >>> typo "structureal". >>> >> >> Done. >> >>> 15. In section 5.1 first paragraph, add a Word cross-reference for >>> [SCA-Assembly]. >>> >> >> Done. >> >>> 16. In section 5.1 second paragraph, change [SCA-ASSEMBLY] to >>> [SCA-Assembly} and add a Word cross-reference. >>> >> >> Done. >> >>> 17. In section 5.2, change Appendix XXX to Appendix B. >>> >> >> Done. >> >>> 18. In section 5.2, add Word cross-references for [SCA-Assembly] and >>> [SCA-Policy]. >>> >> >> Done. >> >>> 19. In conformance statement BWS20005, fix double blank in >>> "runtime MUST". >>> >> >> Done. >> >>> 20. In conformance statement BWS20019, fix wrong font (Verdana >>> instead of Arial) and spurious indentation in opening paragraph >>> and first two bullets. >>> >> >> Not sure why this was showing up. >> Encountered some strange word bug (or lack of word skills). BWS20019 >> now does not have any bullets. >> >>> 21. In conformance statement BWS20019, third bulleted item does >>> not have a bullet and is wrongly indented. >>> >> >> See above. >> >>> 22. In conformance statements BWS20021/22/23/24, fix wrong font >>> (Verdana instead of Arial). >>> >> >> Fixed. >> >>> 23. In conformance statement BWS20032, fix wrong font (Verdana >>> instead of Arial) and spurious indentation. >>> >> >> Fixed. >> >>> 24. In Appendix D, add the list of contributors. >>> >> >> SimonH: can you provide me a list. This needs to go in all the binding >> specs. >> >>> Simon >>> >>> Anish Karmarkar wrote: >>>> Available at: >>>> http://www.oasis-open.org/committees/download.php/32787/sca-binding-ws-1.1-spec-cd02-rev1.doc >>>> >>>> http://www.oasis-open.org/committees/download.php/32788/sca-binding-ws-1.1-spec-cd02-rev1.pdf >>>> >>>> >>>> I used SimonH's issue 61 rev3 document as the starting point. CD02 >>>> rev1 includes resolutions of all the resolved issues + all the >>>> action items against the ws binding editors. Appendix F contains the >>>> revision history. >>>> >>>> The front page/footer isn't quite right (links, date, title etc) wrt >>>> what we need for CD03/PRD and will be changed before publication. >>>> >>>> I have also attached the diff version (PDF). >>>> >>>> -Anish >>>> -- >>>> >>>> >>>> ------------------------------------------------------------------------ >>>> >>>> >>>> --------------------------------------------------------------------- >>>> To unsubscribe from this mail list, you must leave the OASIS TC that >>>> generates this mail. Follow this link to all your TCs in OASIS at: >>>> https://www.oasis-open.org/apps/org/workgroup/portal/my_workgroups.php >>> >>> >> >> > > > > --------------------------------------------------------------------- > To unsubscribe from this mail list, you must leave the OASIS TC that > generates this mail. Follow this link to all your TCs in OASIS at: > https://www.oasis-open.org/apps/org/workgroup/portal/my_workgroups.php
[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]