You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by cr...@apache.org on 2001/07/17 22:24:18 UTC

cvs commit: jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/util/xml XmlMapper.java

craigmcc    01/07/17 13:24:18

  Modified:    catalina/src/share/org/apache/catalina/startup
                        Constants.java ContextConfig.java
               catalina/src/share/org/apache/catalina/util/xml
                        XmlMapper.java
  Removed:     catalina/src/conf tld_11.dtd tld_12.dtd web_22.dtd
                        web_23.dtd
  Log:
  Make Catalina dependent on the DTD files inside servlet.jar (built from
  the jakarta-servletapi-4 repository) rather than maintaining its own
  copies, which run the risk of being out of date all the time.
  
  Revision  Changes    Path
  1.4       +12 -8     jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/startup/Constants.java
  
  Index: Constants.java
  ===================================================================
  RCS file: /home/cvs/jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/startup/Constants.java,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -r1.3 -r1.4
  --- Constants.java	2000/10/04 02:26:30	1.3
  +++ Constants.java	2001/07/17 20:24:17	1.4
  @@ -1,7 +1,7 @@
   /*
  - * $Header: /home/cvs/jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/startup/Constants.java,v 1.3 2000/10/04 02:26:30 craigmcc Exp $
  - * $Revision: 1.3 $
  - * $Date: 2000/10/04 02:26:30 $
  + * $Header: /home/cvs/jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/startup/Constants.java,v 1.4 2001/07/17 20:24:17 craigmcc Exp $
  + * $Revision: 1.4 $
  + * $Date: 2001/07/17 20:24:17 $
    *
    * ====================================================================
    *
  @@ -69,7 +69,7 @@
    * String constants for the startup package.
    *
    * @author Craig R. McClanahan
  - * @version $Revision: 1.3 $ $Date: 2000/10/04 02:26:30 $
  + * @version $Revision: 1.4 $ $Date: 2001/07/17 20:24:17 $
    */
   
   public final class Constants {
  @@ -82,21 +82,25 @@
       public static final String TldDtdPublicId_11 =
           "-//Sun Microsystems, Inc.//DTD JSP Tag Library 1.1//EN";
       public static final String TldDtdResourcePath_11 =
  -        "conf/tld_11.dtd";
  +        //        "conf/tld_11.dtd";
  +        "/javax/servlet/jsp/resources/web-jsptaglibrary_1_1.dtd";
   
       public static final String TldDtdPublicId_12 =
           "-//Sun Microsystems, Inc.//DTD JSP Tag Library 1.2//EN";
       public static final String TldDtdResourcePath_12 =
  -        "conf/tld_12.dtd";
  +        //        "conf/tld_12.dtd";
  +        "/javax/servlet/jsp/resources/web-jsptaglibrary_1_2.dtd";
   
       public static final String WebDtdPublicId_22 =
   	"-//Sun Microsystems, Inc.//DTD Web Application 2.2//EN";
       public static final String WebDtdResourcePath_22 =
  -	"conf/web_22.dtd";
  +        //	"conf/web_22.dtd";
  +        "/javax/servlet/resources/web-app_2_2.dtd";
   
       public static final String WebDtdPublicId_23 =
   	"-//Sun Microsystems, Inc.//DTD Web Application 2.3//EN";
       public static final String WebDtdResourcePath_23 =
  -	"conf/web_23.dtd";
  +        //	"conf/web_23.dtd";
  +        "/javax/servlet/resources/web-app_2_3.dtd";
   
   }
  
  
  
  1.48      +12 -20    jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/startup/ContextConfig.java
  
  Index: ContextConfig.java
  ===================================================================
  RCS file: /home/cvs/jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/startup/ContextConfig.java,v
  retrieving revision 1.47
  retrieving revision 1.48
  diff -u -r1.47 -r1.48
  --- ContextConfig.java	2001/06/19 02:20:32	1.47
  +++ ContextConfig.java	2001/07/17 20:24:17	1.48
  @@ -1,7 +1,7 @@
   /*
  - * $Header: /home/cvs/jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/startup/ContextConfig.java,v 1.47 2001/06/19 02:20:32 remm Exp $
  - * $Revision: 1.47 $
  - * $Date: 2001/06/19 02:20:32 $
  + * $Header: /home/cvs/jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/startup/ContextConfig.java,v 1.48 2001/07/17 20:24:17 craigmcc Exp $
  + * $Revision: 1.48 $
  + * $Date: 2001/07/17 20:24:17 $
    *
    * ====================================================================
    *
  @@ -128,7 +128,7 @@
    * of that Context, and the associated defined servlets.
    *
    * @author Craig R. McClanahan
  - * @version $Revision: 1.47 $ $Date: 2001/06/19 02:20:32 $
  + * @version $Revision: 1.48 $ $Date: 2001/07/17 20:24:17 $
    */
   
   public final class ContextConfig
  @@ -422,14 +422,10 @@
           if (debug > 0)
               mapper.setDebug(3);
           mapper.setValidating(true);
  -	File resourceFile = new File(System.getProperty("catalina.home"),
  -				     Constants.TldDtdResourcePath_11);
  -	mapper.registerDTDFile(Constants.TldDtdPublicId_11,
  -			       resourceFile.toString());
  -	resourceFile = new File(System.getProperty("catalina.home"),
  -				Constants.TldDtdResourcePath_12);
  -	mapper.registerDTDFile(Constants.TldDtdPublicId_12,
  -			       resourceFile.toString());
  +        mapper.registerDTDRes(Constants.TldDtdPublicId_11,
  +                              Constants.TldDtdResourcePath_11);
  +        mapper.registerDTDRes(Constants.TldDtdPublicId_12,
  +                              Constants.TldDtdResourcePath_12);
   
   	mapper.addRule("taglib/listener/listener-class",
   		       mapper.methodSetter("addApplicationListener", 0));
  @@ -449,14 +445,10 @@
   	if (debug > 0)
   	    mapper.setDebug(3);
   	mapper.setValidating(true);
  -	File resourceFile = new File(System.getProperty("catalina.home"),
  -				     Constants.WebDtdResourcePath_22);
  -	mapper.registerDTDFile(Constants.WebDtdPublicId_22,
  -			       resourceFile.toString());
  -	resourceFile = new File(System.getProperty("catalina.home"),
  -				Constants.WebDtdResourcePath_23);
  -	mapper.registerDTDFile(Constants.WebDtdPublicId_23,
  -			       resourceFile.toString());
  +        mapper.registerDTDRes(Constants.WebDtdPublicId_22,
  +                              Constants.WebDtdResourcePath_22);
  +        mapper.registerDTDRes(Constants.WebDtdPublicId_23,
  +                              Constants.WebDtdResourcePath_23);
   
           mapper.addRule("web-app",
                          new SetPublicIdAction("setPublicId"));
  
  
  
  1.7       +6 -1      jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/util/xml/XmlMapper.java
  
  Index: XmlMapper.java
  ===================================================================
  RCS file: /home/cvs/jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/util/xml/XmlMapper.java,v
  retrieving revision 1.6
  retrieving revision 1.7
  diff -u -r1.6 -r1.7
  --- XmlMapper.java	2001/05/23 22:19:49	1.6
  +++ XmlMapper.java	2001/07/17 20:24:18	1.7
  @@ -443,6 +443,8 @@
   	    File dtdF=new File( dtd );
   	    if( dtdF.exists() )
   		try {
  +                    if (debug >= 1)
  +                        System.out.println("  Using alternate DTD " + dtd);
   		    return new InputSource(new FileInputStream(dtdF));
   		} catch( FileNotFoundException ex ) {
   		}
  @@ -452,8 +454,11 @@
   	dtd = (String) resDTDs.get( publicId );
   	if( dtd != null ) {
   	    InputStream is = this.getClass().getResourceAsStream( dtd );
  -	    if( is!= null )
  +	    if( is!= null ) {
  +                if (debug >= 1)
  +                    System.out.println("  Using alternate DTD " + dtd);
   		return new InputSource(is);
  +            }
   	    System.out.println("XXX resource not found !!! " + dtd);
   	    System.out.println(this.getClass().getClassLoader().getClass().getName());
   	}