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 --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
