Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 370373 - Fix jetty aggregates to exclude new orbit deps
Summary: Fix jetty aggregates to exclude new orbit deps
Status: CLOSED FIXED
Alias: None
Product: Jetty
Classification: RT
Component: build (show other bugs)
Version: 7.6.1   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 7.5.x   Edit
Assignee: Joakim Erdfelt CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-02-01 15:56 EST by Joakim Erdfelt CLA
Modified: 2012-04-02 13:27 EDT (History)
2 users (show)

See Also:

Attachments
diff of changes between jetty-all-server 7.5.4 -> 7.6.1-SNAPSHOT (15.49 KB, patch)
2012-02-01 16:10 EST, Joakim Erdfelt CLA 		no flags Details | Diff
diff of changes between jetty-client 7.5.4 -> 7.6.1-SNAPSHOT (6.04 KB, patch)
2012-02-01 16:13 EST, Joakim Erdfelt CLA 		no flags Details | Diff
diff of changes between jetty-plus 7.5.4 -> 7.6.1-SNAPSHOT (7.85 KB, patch)
2012-02-01 16:22 EST, Joakim Erdfelt CLA 		no flags Details | Diff
diff of changes between jetty-plus 7.5.4 -> 7.6.1-SNAPSHOT (10.01 KB, patch)
2012-02-01 16:33 EST, Joakim Erdfelt CLA 		no flags Details | Diff
diff of changes between jetty-servlet 7.5.4 -> 7.6.1-SNAPSHOT (10.02 KB, patch)
2012-02-01 16:38 EST, Joakim Erdfelt CLA 		no flags Details | Diff
diff of changes between jetty-webapp 7.5.4 -> 7.6.1-SNAPSHOT (7.47 KB, patch)
2012-02-01 16:45 EST, Joakim Erdfelt CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Joakim Erdfelt CLA 2012-02-01 15:56:37 EST 
Since the introduction of new orbit deps, the jetty-aggregates now include all of the orbit dep classes as well.
Comment 1 Joakim Erdfelt CLA 2012-02-01 16:00:07 EST 
jetty-all aggregate has been completed.
However, the old 7.5.4 jetty-all at 
http://repo1.maven.org/maven2/org/eclipse/jetty/aggregate/jetty-all/7.5.4.v20111024/jetty-all-7.5.4.v20111024.jar

includes references to various jasper/geronimo artifacts.

(diff - 7.5.4 to 7.6.1-SNAPSHOT)

-META-INF/maven/org.apache.geronimo.specs/geronimo-annotation_1.0_spec/pom.properties
-META-INF/maven/org.apache.geronimo.specs/geronimo-annotation_1.0_spec/pom.xml
 META-INF/maven/org.apache.geronimo.specs/geronimo-jaspic_1.0_spec/pom.properties
 META-INF/maven/org.apache.geronimo.specs/geronimo-jaspic_1.0_spec/pom.xml
-META-INF/maven/org.apache.geronimo.specs/geronimo-jta_1.1_spec/pom.properties
-META-INF/maven/org.apache.geronimo.specs/geronimo-jta_1.1_spec/pom.xml


-org/apache/jasper/compiler/JDTJavaCompiler$1.class
-org/apache/jasper/compiler/JDTJavaCompiler$1CompilationUnit.class
-org/apache/jasper/compiler/JDTJavaCompiler$2.class
-org/apache/jasper/compiler/JDTJavaCompiler.class

oddly, it appears that the 7.5.4 jetty-all is bad in this regard.
It doesn't have all of the classes it probably was intended to have.
Comment 2 Joakim Erdfelt CLA 2012-02-01 16:10:07 EST 
Created attachment 210402 [details]
diff of changes between jetty-all-server 7.5.4 -> 7.6.1-SNAPSHOT

Work against jetty-all-server completed.
Attaching file diff of changes in jetty-all-server (aggregate) jar, from version 7.5.4 to 7.6.1-SNAPSHOT
Comment 3 Joakim Erdfelt CLA 2012-02-01 16:13:44 EST 
Created attachment 210403 [details]
diff of changes between jetty-client 7.5.4 -> 7.6.1-SNAPSHOT

Work against jetty-client not needed (no orbit deps being used in jetty-client).
Attaching file diff of changes in jetty-all-server (aggregate) jar, from
version 7.5.4 to 7.6.1-SNAPSHOT
Comment 4 Joakim Erdfelt CLA 2012-02-01 16:22:59 EST 
Created attachment 210405 [details]
diff of changes between jetty-plus 7.5.4 -> 7.6.1-SNAPSHOT

Work against jetty-plus completed.
Attaching file diff of changes in jetty-plus (aggregate) jar, from
version 7.5.4 to 7.6.1-SNAPSHOT
Comment 5 Joakim Erdfelt CLA 2012-02-01 16:33:36 EST 
Created attachment 210408 [details]
diff of changes between jetty-plus 7.5.4 -> 7.6.1-SNAPSHOT

Work against jetty-server completed.
Attaching file diff of changes in jetty-server (aggregate) jar, from
version 7.5.4 to 7.6.1-SNAPSHOT
Comment 6 Joakim Erdfelt CLA 2012-02-01 16:38:00 EST 
Created attachment 210409 [details]
diff of changes between jetty-servlet 7.5.4 -> 7.6.1-SNAPSHOT

Work against jetty-servlet completed.
Attaching file diff of changes in jetty-servlet (aggregate) jar, from
version 7.5.4 to 7.6.1-SNAPSHOT
Comment 7 Joakim Erdfelt CLA 2012-02-01 16:45:14 EST 
Created attachment 210410 [details]
diff of changes between jetty-webapp 7.5.4 -> 7.6.1-SNAPSHOT

Work against jetty-webapp completed.
Attaching file diff of changes in jetty-webapp (aggregate) jar, from
version 7.5.4 to 7.6.1-SNAPSHOT
Comment 8 Jan Bartel CLA 2012-02-01 21:16:49 EST 
Actually, the central-orbit-repo branch should have already contained fixes for the aggregates for the orbit deps.
Comment 9 Joakim Erdfelt CLA 2012-04-02 13:27:11 EDT 
Closing as resolved (a few versions ago)