Though I have to say, it requires a bump in the xsd version, cause if this is required now it's a major version upgrade. :) regards, Achim
2011/11/19 Jean-Baptiste Onofré <[email protected]> > Hi Ioannis, > > My comments: > > 1/ it's on trunk, I display just a warning on 2.2.x > 2/ the main reason is all features without name will use the same > repository name (local-0 or so). It means that features:listrepositories is > not helpful at all. With a mandatory name, we have a clean repository by > features. > 3/ all projects (Camel, CXF, SMX, etc) define the name on the recent > versions. For instance, the last version without name is Camel 2.6.0. > > That's why I changed it on trunk, as it makes sense for me. > > Regards > JB > > > On 11/19/2011 02:02 PM, Ioannis Canellos wrote: > >> I came across this one today: >> >> >> >> -<xs:attribute name="name" type="xs:string" /> >>> +<xs:attribute name="name" type="xs:string" use="required" /> >>> </xs:complexType> >>> >>> >> >> Though I don't mind setting things strict, this will render some of the >> existing existing feature descriptors useless (I am refering to 3rd party >> descriptors). >> This sounds kinda heavy and I do not see a good reason for it (what I mean >> is that in order to cover a trivial need we seriously break backwards >> compatibility). >> If we still have to do so, then we might need to bump the version of the >> xsd. >> >> > -- > Jean-Baptiste Onofré > [email protected] > http://blog.nanthrax.net > Talend - http://www.talend.com > -- Apache Karaf <http://karaf.apache.org/> Committer & PMC OPS4J Pax Web <http://wiki.ops4j.org/display/paxweb/Pax+Web/> Committer & Project Lead blog <http://notizblog.nierbeck.de/>
