adding this to the web.xml means requiring a servlet container that supports servlets 2.5 (unless the container just ignores attributes it doesn't understand)

but that seems fine for trunk - 2.5 came out in 2005, and we already compile and test against 3.0

(updated summary & description to note this isn't jetty specific)

Can someone provide a specific patch for this one? Seems like we want this for 4.0

web.xml patch.

doing some testing now

Committed revision 1331529.

verified no problems with our jetty8 example or when running in tomcat7

Jboss 5.1.0 won't start with the new <web-app /> declaration. This works, however:

<web-app xmlns="http://java.sun.com/xml/ns/javaee"
         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
         xsi:schemaLocation="http://java.sun.com/xml/ns/javaee http://java.sun.com/xml/ns/javaee/web-app_2_5.xsd"
         version="2.5"
         metadata-complete="true"
>

(changing "j2ee" to "javaee"). I do not know, however, if this is a problem with Jboss or if what we have in Trunk currently is incorrect ?

James: Unless I'm missing something, the specifics of whether it's "j2ee" or "javaee" shouldn't matter to XML parser (or the servlet container). All that should matter is that each of the schemaLocation keys listed correspond to an xml namespace being used. (ie: you changed http://java.sun.com/xml/ns/j2ee -> http://java.sun.com/xml/ns/javaee in two places, but as long as those two places are the same it shouldn't matter what we use ... i think it could be "http://giberish-foo/uwesays" and it should still work).

So it seems like maybe your jboss configuration has something wonky going on?

I'm inclined to think we should be consistent with the examples in the 2.5 spec...

http://download.oracle.com/otn-pub/jcp/servlet-2.5-mrel2-eval-oth-JSpec/servlet-2_5-mrel2-spec.pdf

...(see pages 153 & 154) which use j2ee – but oddly enough the example snippets embedded in the 2.5 XSD (using xsd:documentation tags) use "javaee"...

http://java.sun.com/xml/ns/javaee/web-app_2_5.xsd

...so apparently the spec and the xsd shipped with the spec are inconsistent? which i guess means feel free to change it to follow the examples in the XSD?

...so apparently the spec and the xsd shipped with the spec are inconsistent? which i guess means feel free to change it to follow the examples in the XSD?

If I validate the change with Jetty8 & Tomcat7 (so as not to break the 2 you validated before) would you be comfortable with it being changed to "javaee"?

i just used what was in the examples from the spec, and as far as i know it shouldn't matter what we use ... if you're comfortable changing it, then feel free to change it.

re-opening to change the <web-app /> tag as discussed.

The original commit for this issue followed a misprint in the 2.5 servlet spec on the examples, pages 153-4 (http://download.oracle.com/otn-pub/jcp/servlet-2.5-mrel2-eval-oth-JSpec/servlet-2_5-mrel2-spec.pdf). While this works on Tomcat7 & Jetty8, it does not work with Jboss5 (which apparently is doing strict verification). This patch follows the information on page 109ff of the same specification and also the xsd file at http://java.sun.com/xml/ns/javaee/web-app_2_5.xsd (note: There is no xsd file available at http://java.sun.com/xml/ns/j2ee/web-app_2_5.xsd). I have tested this with JBoss5, Tomcat7 & the Jetty8-based solr example.

I will commit shortly unless anyone disagrees.

Committed the "javaee" change, Trunk r1341897.