From a user/community perspective I would go for 3.0 as a minimum.
Testing with a wide range of versions would be good.
So the minimum (3.0) and the latest (3.2.5 if available on Bamboo) would
already cover a lot.
Op Sat, 07 Feb 2015 10:51:50 +0100 schreef Lennart Jörelid
<lennart.jore...@gmail.com>:
Hello there,
I took the version defined/used by the Codehaus Bamboo servers for ITs.
I can easily pick another version for more sensible reasons.
Which 3.0.x version is the better one to use?
(I suppose I should adjust the bamboo jobs as well, right?)
2015-02-07 10:29 GMT+01:00 Robert Scholte <codeh...@sourcegrounds.com>:
Hi Lennart,
Why 3.0.4? This version has a security issue[1] fixed in 3.0.5 (it is
actually the only fix in 3.0.5).
And it is probably easier to say the plugin is Maven3 compatible, which
implies at least Maven 3.0.
Is there a specific interface change in 3.0.4 you depend on?
thanks,
Robert
[1] http://maven.apache.org/security.html
Op Fri, 06 Feb 2015 05:44:13 +0100 schreef <lenna...@codehaus.org>:
Revision:
20398
Author:
lennartj
Date:
2015-02-05 22:44:12 -0600 (Thu, 05 Feb 2015)
Log Message
Assigned Maven 3.0.4 as minimum Maven version for the
jaxb2-maven-plugin.
Modified Paths
trunk/mojo/jaxb2-maven-plugin/pom.xml
trunk/mojo/jaxb2-maven-plugin/src/test/java/org/codehaus/
mojo/jaxb2/junit/PlexusContainerRule.java
Diff
Modified: trunk/mojo/jaxb2-maven-plugin/pom.xml (20397 => 20398)
--- trunk/mojo/jaxb2-maven-plugin/pom.xml 2015-02-05
21:54:17 UTC (rev 20397)
+++ trunk/mojo/jaxb2-maven-plugin/pom.xml 2015-02-06
04:44:12 UTC (rev 20398)
@@ -101,11 +101,11 @@
<properties>
<!-- Define Maven and JDK versions-->
- <mavenVersion>2.2.1</mavenVersion>
+ <mavenVersion>3.0.4</mavenVersion>
<mojo.java.target>1.6</mojo.java.target>
<!-- Plexus component versions -->
- <plexus-compiler-api.version>2.4</plexus-compiler-api.version>
+ <plexus-compiler-api.version>2.5</plexus-compiler-api.version>
<plexus-utils.version>3.0.21</plexus-utils.version>
<plexus-build-api.version>0.0.7</plexus-build-api.version>
@@ -199,11 +199,6 @@
</dependency>
<dependency>
<groupId>org.apache.maven</groupId>
- <artifactId>maven-project</artifactId>
- <version>${mavenVersion}</version>
- </dependency>
- <dependency>
- <groupId>org.apache.maven</groupId>
<artifactId>maven-model</artifactId>
<version>${mavenVersion}</version>
</dependency>
Modified: trunk/mojo/jaxb2-maven-plugin/src/test/java/org/codehaus/
mojo/jaxb2/junit/PlexusContainerRule.java
(20397 => 20398)
--- trunk/mojo/jaxb2-maven-plugin/src/test/java/org/codehaus/
mojo/jaxb2/junit/PlexusContainerRule.java
2015-02-05 21:54:17 UTC (rev 20397)
+++ trunk/mojo/jaxb2-maven-plugin/src/test/java/org/codehaus/
mojo/jaxb2/junit/PlexusContainerRule.java
2015-02-06 04:44:12 UTC (rev 20398)
@@ -2,13 +2,14 @@
import org.codehaus.mojo.jaxb2.shared.FileSystemUtilities;
import org.codehaus.mojo.jaxb2.shared.Validate;
+import org.codehaus.plexus.ContainerConfiguration;
+import org.codehaus.plexus.DefaultContainerConfiguration;
import org.codehaus.plexus.DefaultPlexusContainer;
import org.codehaus.plexus.PlexusContainer;
import org.codehaus.plexus.context.Context;
import org.junit.runner.Description;
import java.io.File;
-import java.io.InputStreamReader;
import java.net.URL;
/**
@@ -33,6 +34,7 @@
// Internal state
private PlexusContainer container;
+ private ContainerConfiguration config;
private URL configurationURL;
/**
@@ -94,10 +96,16 @@
final Class<?> testClass = description.getTestClass();
Validate.notNull(testClass, "Cannot handle null
'description.getTestClass()' value.");
- // Create and configure the Container
- container = new DefaultPlexusContainer();
- container.addContextValue(BASEDIR, getBasedir());
+ // Configure and create the Container
+ final URL configURL = this.configurationURL == null
+ ? getDefaultConfigurationURL(testClass)
+ : this.configurationURL;
+ config = new DefaultContainerConfiguration();
+ config.setContainerConfigurationURL(configURL);
+ container = new DefaultPlexusContainer(config);
+ container.getContext().put(BASEDIR, getBasedir());
+
// Configure the Plexus context
final Context context = container.getContext();
if (!context.contains(PLEXUS_HOME)) {
@@ -109,16 +117,9 @@
context.put(PLEXUS_HOME, plexusHomeDir.getAbsolutePath());
}
- final URL configURL = this.configurationURL == null
- ? getDefaultConfigurationURL(testClass)
- : this.configurationURL;
- if (configURL != null) {
- container.setConfigurationResource(new
InputStreamReader(configURL.openStream()));
- }
-
// Initialize and start the Plexus Container.
- container.initialize();
- container.start();
+ // container.initialize();
+ // container.start();
}
/**
To unsubscribe from this list please visit:
http://xircles.codehaus.org/manage_email
---------------------------------------------------------------------
To unsubscribe from this list, please visit:
http://xircles.codehaus.org/manage_email
---------------------------------------------------------------------
To unsubscribe from this list, please visit:
http://xircles.codehaus.org/manage_email
