Re: [Ecrit] WGLC for draft-ietf-ecrit-lost-planned-changes - end date April 8

[Peter Saint-Andre <[e-mail]>]

On 4/25/18 9:40 AM, Peter Saint-Andre wrote:
> On 3/25/18 2:49 AM, Az Mankin wrote:
>>
>> This starts WGLC for  draft-ietf-ecrit-lost-planned-changes-01,
>> Validation of Locations Around a Planned Change
>>
>> Please send both comments and messages that you support advancement to
>> this mailing list before April 8.
> 
> Again, thanks for your patience with my tardy reply. I took another look
> at the schema. There are a few details I still want to check, but
> overall it seems to be in good shape.

I've looked again at the lost1 schema. This looks slightly off:

   <xs:attributeGroup name="expires">
     <xs:attribute name="expires" use="required">
       <xs:simpleType>
         <xs:union memberTypes="xs:dateTime">
           <xs:simpleType>
             <xs:restriction base="xs:token">
               <xs:enumeration value="NO-CACHE"/>
             </xs:restriction>
           </xs:simpleType>
           <xs:simpleType>
             <xs:restriction base="xs:token">
               <xs:enumeration value="NO-EXPIRATION"/>
             </xs:restriction>
           </xs:simpleType>
         </xs:union>
       </xs:simpleType>
     </xs:attribute>
   </xs:attributeGroup>

As far as I can see (I've never used xs:union before), the dateTime
facet would never get merged in. I think we want this:

   <xs:attributeGroup name="expires">
     <xs:attribute name="expires" use="required">
       <xs:simpleType>
         <xs:union>
           <xs:simpleType>
             <xs:restriction base="xs:dateTime"/>
           </xs:simpleType>
           <xs:simpleType>
             <xs:restriction base="xs:token">
               <xs:enumeration value="NO-CACHE"/>
               <xs:enumeration value="NO-EXPIRATION"/>
             </xs:restriction>
           </xs:simpleType>
         </xs:union>
       </xs:simpleType>
     </xs:attribute>
   </xs:attributeGroup>

The rest seems OK.

Peter