tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mark Thomas <ma...@apache.org>
Subject Re: [VOTE] Release Apache Tomcat 7.0.48
Date Tue, 10 Dec 2013 12:58:45 GMT
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-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org


Mime
View raw message