Remove it from geronimo side On Thu, Jun 30, 2011 at 9:24 AM, Ivan <[email protected]> wrote:
> I checked the codes in XPathUtil, it uses the class XPathContext class > directly, so if we would ship that Apache JSTL library in Geronimo 3.0, we > need to add that dependency explicitly, not using bootdelegation here. Or it > will not work with Oracle JRE, which does not ship the required classes. > > 2011/6/30 viola lu <[email protected]> > >> It's very specific test case. If we use Apache JSTL >> implementation(jstl.jstl.1.2) not ri , and run jstl test cases on IBM JDK, >> it will report org.apache.xpath.XpathContext ,and org.apache.xml.utils.Qname >> class not found although i already add ibm jdk xml.jar on test case >> classpath. >> Appreciate if you can provide an option from JVM side. >> Error trace. >> <pre>javax.servlet.ServletException: java.lang.NoClassDefFoundError: >> org/apache/xpath/XPathContext >> >> org.apache.jasper.runtime.PageContextImpl.doHandlePageException(PageContextImpl.java:915) >> >> org.apache.jasper.runtime.PageContextImpl.handlePageException(PageContextImpl.java:844) >> >> org.apache.jsp.positiveForEachStepTest_jsp._jspService(positiveForEachStepTest_jsp.java:142) >> org.apache.jasper.runtime.HttpJspBase.service(HttpJspBase.java:70) >> javax.servlet.http.HttpServlet.service(HttpServlet.java:668) >> >> org.apache.jasper.servlet.JspServletWrapper.service(JspServletWrapper.java:417) >> org.apache.jasper.servlet.JspServlet.serviceJspFile(JspServlet.java:391) >> org.apache.jasper.servlet.JspServlet.service(JspServlet.java:334) >> javax.servlet.http.HttpServlet.service(HttpServlet.java:668) >> </pre></p><p><b>root cause</b> <pre>java.lang.NoClassDefFoundError: >> org/apache/xpath/XPathContext >> >> org.apache.taglibs.standard.tag.common.xml.XPathUtil.selectNodes(XPathUtil.java:519) >> >> org.apache.taglibs.standard.tag.common.xml.ForEachTag.prepare(ForEachTag.java:50) >> >> javax.servlet.jsp.jstl.core.LoopTagSupport.doStartTag(LoopTagSupport.java:231) >> >> org.apache.jsp.positiveForEachStepTest_jsp._jspService(positiveForEachStepTest_jsp.java:99) >> org.apache.jasper.runtime.HttpJspBase.service(HttpJspBase.java:70) >> javax.servlet.http.HttpServlet.service(HttpServlet.java:668) >> >> org.apache.jasper.servlet.JspServletWrapper.service(JspServletWrapper.java:417) >> org.apache.jasper.servlet.JspServlet.serviceJspFile(JspServlet.java:391) >> org.apache.jasper.servlet.JspServlet.service(JspServlet.java:334) >> javax.servlet.http.HttpServlet.service(HttpServlet.java:668) >> >> On Thu, Jun 30, 2011 at 12:43 AM, Jarek Gawor <[email protected]> wrote: >> >>> I'm still not sure about this. Can you open a bug on this with the >>> exact information why this is needed in the first place? What >>> exceptions or problems did you see? Maybe that option needs to be JVM >>> specific? >>> >>> Jarek >>> >>> On Wed, Jun 29, 2011 at 10:41 AM, viola lu <[email protected]> wrote: >>> > OK, i already update it to accurate package names. >>> > >>> > On Wed, Jun 29, 2011 at 8:28 PM, Ivan <[email protected]> wrote: >>> >> >>> >> It looks to me that org.apache is a very common package name, once it >>> is >>> >> added in the bootdelegation list, each class belongs to this package >>> (even >>> >> class from Geronimo) will be delegated to framework classloader first, >>> which >>> >> is not a good idea in my opinion. If some packages are really required >>> to >>> >> load from there, the package names should be more accurate. >>> >> >>> >> 2011/6/29 <[email protected]> >>> >>> >>> >>> Author: violalu >>> >>> Date: Wed Jun 29 09:03:40 2011 >>> >>> New Revision: 1141014 >>> >>> >>> >>> URL: http://svn.apache.org/viewvc?rev=1141014&view=rev >>> >>> Log: >>> >>> export org.apache.* in ibm jdk such as org.apache.xpath.* >>> >>> >>> >>> Modified: >>> >>> >>> >>> >>> >>> geronimo/server/trunk/framework/configs/karaf-framework/src/main/filtered-resources/etc/config.properties >>> >>> >>> >>> Modified: >>> >>> >>> geronimo/server/trunk/framework/configs/karaf-framework/src/main/filtered-resources/etc/config.properties >>> >>> URL: >>> >>> >>> http://svn.apache.org/viewvc/geronimo/server/trunk/framework/configs/karaf-framework/src/main/filtered-resources/etc/config.properties?rev=1141014&r1=1141013&r2=1141014&view=diff >>> >>> >>> >>> >>> ============================================================================== >>> >>> --- >>> >>> >>> geronimo/server/trunk/framework/configs/karaf-framework/src/main/filtered-resources/etc/config.properties >>> >>> (original) >>> >>> +++ >>> >>> >>> geronimo/server/trunk/framework/configs/karaf-framework/src/main/filtered-resources/etc/config.properties >>> >>> Wed Jun 29 09:03:40 2011 >>> >>> @@ -55,7 +55,7 @@ org.osgi.framework.system.packages.extra >>> >>> org.apache.geronimo.hook >>> >>> >>> >>> # javax.transaction is needed to avoid class loader constraint >>> violation >>> >>> when using javax.sql >>> >>> >>> >>> >>> -org.osgi.framework.bootdelegation=sun.*,com.sun.*,javax.management.remote.rmi,javax.transaction,javax.transaction.*,com.ibm.*,org.apache.harmony.* >>> >>> >>> >>> >>> +org.osgi.framework.bootdelegation=sun.*,com.sun.*,javax.management.remote.rmi,javax.transaction,javax.transaction.*,com.ibm.*,org.apache.harmony.*,org.apache.* >>> >>> >>> >>> # OSGi Execution Environment >>> >>> >>> >>> >>> >>> org.osgi.framework.executionenvironment=OSGi/Minimum-1.2,J2SE-1.2,J2SE-1.3,J2SE-1.4,J2SE-1.5,JavaSE-1.6 >>> >>> >>> >>> >>> >> >>> >> >>> >> >>> >> -- >>> >> Ivan >>> > >>> > >>> > >>> > -- >>> > viola >>> > Apache Geronimo >>> > >>> >> >> >> >> -- >> viola >> >> Apache Geronimo >> >> > > > -- > Ivan > -- viola Apache Geronimo
