Yes and yes. And you might want to test in Bamboo with 3.0 as well. Just to confirm that is really works with this minimum. In Bamboo it is just a matter of adding more jobs to the default stage.
Looking at Bamboo, you should have jobs with both maven-3.0 and maven-3.2

Robert

Op Sat, 07 Feb 2015 11:00:24 +0100 schreef Lennart Jörelid <lennart.jore...@gmail.com>:

That makes sense. So - i f I understand correctly here:


   1. Use maven version 3.0 in the pom.
   2. Use Maven 3.2.5 for the jobs on Bamboo.

Fixing; I'm feeling like starting the release cycle on the
jaxb2-maven-plugin .... soonish.

:)

2015-02-07 10:55 GMT+01:00 Robert Scholte <codeh...@sourcegrounds.com>:

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





---------------------------------------------------------------------
To unsubscribe from this list, please visit:

   http://xircles.codehaus.org/manage_email


Reply via email to