I think you missed the point sir :-P I was not talking about "_" vs "-" ...

I was suggesting that the v.v bits are you use below are whack, they should not be needed... all versioning should be in mavenVersion.

--jason


On Dec 6, 2006, at 11:01 AM, Matt Hogstrom wrote:

I dunno... I kinda like the artifact_v.v-mavenVersion as artifact- v.v-mavenVersion seems more confusing. I'll have to noodle on it though. I think the _ in the name clearly identifies the version number as being related to the artifact content rather then the version of the specs.

On Dec 6, 2006, at 1:44 PM, Jason Dillon wrote:

<opinion>
I think it is really loopy that we have version information in artifact ids. A few people outside of the group have mentioned this too me too.

IMO it would be better if the artifact ids of the specs did not have any version muck, and the version of that module was based on the spec version + a revision number.

So for example, the spec here would be:

<groupId>org.apache.geronimo.specs</groupId>
<artifactId>geronimo-j2ee-management</artifactId>
<version>1.1-1</version>

(or 1.1-1-SNAPSHOT for evil mvn snapshot muck)

Anyways, I'm not going to do anything about this... but I will continue to point out that putting version information into artifactIds is a poor idea... and generally a bad one at that.
</opinion>

--jason


On Dec 6, 2006, at 7:46 AM, [EMAIL PROTECTED] wrote:

Author: akulshreshtha
Date: Wed Dec  6 07:46:42 2006
New Revision: 483121

URL: http://svn.apache.org/viewvc?view=rev&rev=483121
Log:
GERONIMO-2629 Upgrade to J2EE Management 1.1

Modified:
    geronimo/server/trunk/configs/client-system/pom.xml
    geronimo/server/trunk/configs/jee-specs/pom.xml
    geronimo/server/trunk/modules/ge-activemq-rar/pom.xml
    geronimo/server/trunk/modules/geronimo-management/pom.xml
    geronimo/server/trunk/modules/geronimo-tomcat/pom.xml
    geronimo/server/trunk/pom.xml

Modified: geronimo/server/trunk/configs/client-system/pom.xml
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/configs/ client-system/pom.xml?view=diff&rev=483121&r1=483120&r2=483121 ==================================================================== ==========
--- geronimo/server/trunk/configs/client-system/pom.xml (original)
+++ geronimo/server/trunk/configs/client-system/pom.xml Wed Dec 6 07:46:42 2006
@@ -44,7 +44,7 @@

         <dependency>
             <groupId>org.apache.geronimo.specs</groupId>
- <artifactId>geronimo-j2ee-management_1.0_spec</ artifactId> + <artifactId>geronimo-j2ee-management_1.1_spec</ artifactId>
         </dependency>

     </dependencies>

Modified: geronimo/server/trunk/configs/jee-specs/pom.xml
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/configs/ jee-specs/pom.xml?view=diff&rev=483121&r1=483120&r2=483121 ==================================================================== ==========
--- geronimo/server/trunk/configs/jee-specs/pom.xml (original)
+++ geronimo/server/trunk/configs/jee-specs/pom.xml Wed Dec 6 07:46:42 2006
@@ -58,7 +58,7 @@

         <dependency>
             <groupId>org.apache.geronimo.specs</groupId>
- <artifactId>geronimo-j2ee-management_1.0_spec</ artifactId> + <artifactId>geronimo-j2ee-management_1.1_spec</ artifactId>
         </dependency>

         <dependency>

Modified: geronimo/server/trunk/modules/ge-activemq-rar/pom.xml
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/modules/ ge-activemq-rar/pom.xml?view=diff&rev=483121&r1=483120&r2=483121 ==================================================================== ==========
--- geronimo/server/trunk/modules/ge-activemq-rar/pom.xml (original)
+++ geronimo/server/trunk/modules/ge-activemq-rar/pom.xml Wed Dec 6 07:46:42 2006
@@ -120,7 +120,7 @@

                 <exclusion>
                     <groupId>org.apache.geronimo.specs</groupId>
- <artifactId>geronimo-j2ee- management_1.0_spec</artifactId> + <artifactId>geronimo-j2ee- management_1.1_spec</artifactId>
                 </exclusion>

                 <exclusion>

Modified: geronimo/server/trunk/modules/geronimo-management/pom.xml
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/modules/ geronimo-management/pom.xml?view=diff&rev=483121&r1=483120&r2=483121 ==================================================================== ========== --- geronimo/server/trunk/modules/geronimo-management/pom.xml (original) +++ geronimo/server/trunk/modules/geronimo-management/pom.xml Wed Dec 6 07:46:42 2006
@@ -46,7 +46,7 @@

         <dependency>
             <groupId>org.apache.geronimo.specs</groupId>
- <artifactId>geronimo-j2ee-management_1.0_spec</ artifactId> + <artifactId>geronimo-j2ee-management_1.1_spec</ artifactId>
         </dependency>

     </dependencies>

Modified: geronimo/server/trunk/modules/geronimo-tomcat/pom.xml
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/modules/ geronimo-tomcat/pom.xml?view=diff&rev=483121&r1=483120&r2=483121 ==================================================================== ==========
--- geronimo/server/trunk/modules/geronimo-tomcat/pom.xml (original)
+++ geronimo/server/trunk/modules/geronimo-tomcat/pom.xml Wed Dec 6 07:46:42 2006
@@ -54,7 +54,7 @@

         <dependency>
             <groupId>org.apache.geronimo.specs</groupId>
- <artifactId>geronimo-j2ee-management_1.0_spec</ artifactId> + <artifactId>geronimo-j2ee-management_1.1_spec</ artifactId>
             <scope>provided</scope>
         </dependency>


Modified: geronimo/server/trunk/pom.xml
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/pom.xml? view=diff&rev=483121&r1=483120&r2=483121 ==================================================================== ==========
--- geronimo/server/trunk/pom.xml (original)
+++ geronimo/server/trunk/pom.xml Wed Dec  6 07:46:42 2006
@@ -135,8 +135,8 @@

             <dependency>
                 <groupId>org.apache.geronimo.specs</groupId>
- <artifactId>geronimo-j2ee-management_1.0_spec</ artifactId>
-                <version>1.0.1</version>
+ <artifactId>geronimo-j2ee-management_1.1_spec</ artifactId>
+                <version>1.0-SNAPSHOT</version>
             </dependency>

             <dependency>





Matt Hogstrom
[EMAIL PROTECTED]



Reply via email to