Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [platform-releng-dev] [jdt-dev] Fwd: [platform-dev] Why some inner jars?

Hi Mickael

Almost all our (Platform, JDT, PDE) test bundles have their class files inside a JAR that's inside the JARed bundle. This is simply for historical reasons: back in the old days, each bundle was in a directory with a JAR that contained the class files. When we started to JAR most of our bundles, we converted them to have the class files directly in the bundle JAR. We did not do this for the test bundles just because there was neither a reason nor a benefit. We can accept patches that fix this, but each patch must fix all test projects inside the same repository for consistency and of course you must verify that all tests are still green before submitting the patch.

Dani


From:        Mickael Istria <mistria@xxxxxxxxxx>
To:        jdt-dev@xxxxxxxxxxx, platform-dev@xxxxxxxxxxx, "Eclipse platform release engineering list." <platform-releng-dev@xxxxxxxxxxx>
Date:        29.08.2013 11:47
Subject:        Re: [jdt-dev] Fwd: [platform-dev] Why some inner jars?
Sent by:        jdt-dev-bounces@xxxxxxxxxxx




On 08/23/2013 07:22 PM, Mickael Istria wrote:
I'm having a look at how to get test run automatically as part of Tycho build for Platform.
I noticed some Tycho pitfalls ( https://bugs.eclipse.org/bugs/show_bug.cgi?id=415701 ), caused by unusual patterns on some bundles: some actually compile code, and package it in a jar file that is then added into the bundle as a Bundle-ClassPath element. See for example https://hudson.eclipse.org/hudson/job/platform-sonar/ws/eclipse.jdt.core/org.eclipse.jdt.compiler.tool.tests/target/ (that you can reproduce with "mvn clean install -Dmaven.test.skip=false"). What is the technical reason for this? Is this something we could get rid of?
After some investigation, it appears that the issue on Tycho-side requires refactoring on the Maven-Surefire side, so it most likely won't be resolved soon.
So I don't see any other way to have surefire running on those eclipse-test-plugins than stopping to use nested jars.

Benefits of not using inner jars for these tests:
* Makes tycho-surefire-plugin usable
* Simpler/recommanded OSGi pattern
Drawbacks
* ?

Can anyone elaborate on the drawbacks of not using nested jars for those tests (and in general in JDT) ? If there is no identified drawback, I'd like probably create a patch to fix that.

WDYT?

--
Mickael Istria
Eclipse developer at
JBoss, by Red Hat
My blog - My Tweets_______________________________________________
jdt-dev mailing list
jdt-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/jdt-dev


Back to the top