2013/12/10 Mark Thomas <ma...@apache.org> > > On 10/12/2013 12:23, Mark Thomas wrote: > > On 09/12/2013 21:21, Violeta Georgieva wrote: > >> The proposed Apache Tomcat 7.0.48 release is now available for voting. > >> > >> It can be obtained from: > >> https://dist.apache.org/repos/dist/dev/tomcat/tomcat-7/v7.0.48/ > >> The Maven staging repo is: > >> https://repository.apache.org/content/repositories/orgapachetomcat-030/ > >> The svn tag is: > >> http://svn.apache.org/repos/asf/tomcat/tc7.0.x/tags/TOMCAT_7_0_48/ > >> > >> The proposed 7.0.48 release is: > >> [X] Broken - do not release > >> [ ] Stable - go ahead and release as 7.0.48 Stable > > > > XML validation is broken. I'm pretty sure I broke it. Looking at it now > > and I should have a fix shortly. > > It is the Tomcat 7 version of > https://issues.apache.org/bugzilla/show_bug.cgi?id=55166 which is a > regression introduced by the switch to the new LocalResolver. > > I am current testing the following patch: > > Index: java/org/apache/tomcat/util/descriptor/DigesterFactory.java > =================================================================== > --- java/org/apache/tomcat/util/descriptor/DigesterFactory.java > (revision 1549529) > +++ java/org/apache/tomcat/util/descriptor/DigesterFactory.java (working > copy) > @@ -96,6 +96,7 @@ > private static void addSelf(Map<String, String> ids, String id) { > String systemId = idFor(id); > ids.put(systemId, systemId); > + ids.put(id, systemId); > } > > private static String idFor(String url) { > > > It is a little bit of a hack but so were all the other options I > considered. So far the testing looks good. > > Mark
I'm canceling the vote and I'll tag 7.0.49 in order to include this fix. Regards Violeta
