[ http://jira.codehaus.org/browse/MECLIPSE-467?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=142705#action_142705 ]
charles prevot commented on MECLIPSE-467: ----------------------------------------- Just to make sure we're talking about the same thing, the problem I had was not the char between the name of the artifact and its release, but was in a '-' inside the name of the qualifier version. For exemple, with group artifact 3.4.0 and qualifier, from eclipse to maven to eclipse: group.artifact.3.4.0_qualifier.jar => group.arftifact.3.4.0-qualifier.jar => group.artifact.3.4.0_qualifier.jar (OK) with group artifact 3.4.0 and 'qualifier-with-dashes' group.artifact.3.4.0_qualifier-with-dashes.jar => group.arftifact.3.4.0-qualifier_with_dashes.jar => group.artifact.3.4.0_qualifier_with_dashes.jar (KO) I think the problem comes from the regexp in DefaultMaven2OsgiConverter: there should be no character replacement in the qualifier part of the version. > problem with the way the _ and - get transformed > ------------------------------------------------ > > Key: MECLIPSE-467 > URL: http://jira.codehaus.org/browse/MECLIPSE-467 > Project: Maven 2.x Eclipse Plugin > Issue Type: Bug > Components: OSGi, Manifest > Reporter: charles prevot > Priority: Critical > > I am having some troubles building an RCP application with maven. I had to > modify DefaultMaven2OsgiConverter. > Here is a brief explanation of my problem: > * I have {{org.eclipse.core.commands_3.4.0.I20080509-2000.jar}} in my > {{eclipse/plugins}} directory > * After a {{mvn eclipse:to-maven}}, I have > {{org/eclipse/core/commands/3.4.0-I20080509-2000/commands-3.4.0-I20080509-2000.jar}} > in my repository. > *Then I try to build a target platform against which I would build my > application, so I run {{mvn eclipse:install-plugins}} and it generates me a > {{org.eclipse.core.commands_3.4.0.I20080509_2000.jar}} (notice the {{_}} > instead of the {{-}} in the qualifier part) . > Doing {{mvn eclipse:to-maven}} followed by {{mvn eclipse :install-plugins}} > should end up with the same jar names. > I modfied DefaultMaven2OsgiConverter.java to remove the conversions. Though > it is definitely not a fix, here is the patch: > {noformat}Index: > src/main/java/org/apache/maven/shared/osgi/DefaultMaven2OsgiConverter.java > =================================================================== > --- > src/main/java/org/apache/maven/shared/osgi/DefaultMaven2OsgiConverter.java > (revision 677553) > +++ > src/main/java/org/apache/maven/shared/osgi/DefaultMaven2OsgiConverter.java > (working copy) > @@ -257,7 +257,8 @@ > String major = m.group( 1 ); > String minor = ( m.group( 3 ) != null ) ? m.group( 3 ) : "0"; > String service = ( m.group( 5 ) != null ) ? m.group( 5 ) : "0"; > - String qualifier = m.group( 6 ).replaceAll( "-", "_" > ).replaceAll( "\\.", "_" ); > + String qualifier = m.group( 6 ); > + //String qualifier = m.group( 6 ).replaceAll( "-", "_" > ).replaceAll( "\\.", "_" ); > osgiVersion = major + "." + minor + "." + service + "." + > qualifier; > } > > @@ -263,7 +264,7 @@ > > /* else transform first - to . and others to _ */ > osgiVersion = osgiVersion.replaceFirst( "-", "\\." ); > - osgiVersion = osgiVersion.replaceAll( "-", "_" ); > + //osgiVersion = osgiVersion.replaceAll( "-", "_" ); > m = OSGI_VERSION_PATTERN.matcher( osgiVersion ); > if ( m.matches() ) > { > {noformat} -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira