You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Olivier Lamy (JIRA)" <ji...@codehaus.org> on 2006/04/27 14:09:19 UTC

[jira] Created: (MRELEASE-99) not all system properties used with release:perform

not all system properties used with release:perform
---------------------------------------------------

         Key: MRELEASE-99
         URL: http://jira.codehaus.org/browse/MRELEASE-99
     Project: Maven 2.x Release Plugin
        Type: Bug

 Environment: all
    Reporter: Olivier Lamy


I need to use others systems properties with release:perform.
Actually only maven.test.skip is passed to the cli used by the plugin.
In my case, I use a cli like :
mvn -DtagBase=http://ip:port/path \
-Dusername=$1 -Dpassword=$2 \
-Dmaven.test.skip=true -Dgoals=deploy,accorscm:make release:prepare release:perform.
In cli generated by the plugin, I only see : -Dmaven.test.skip=true.
My plugin (which commit a war in scm) needs the others insure the authentification.
An other use case is :
mvn -DtagBase=http://ip:port/path \
-Dws.ip=$1 -Dws.port=$2 \
-Dmaven.test.skip=true -Dgoals=deploy release:prepare release:perform.
The ip port are needed by the axistools plugin to generate the java classes from the wsdl.
As I see in the source code the patch is not difficult but I don't know if maven had some properties for internal use (this can polluate the build)
Thanks,
Olivier



-- 
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


[jira] Commented: (MRELEASE-99) not all system properties used with release:perform

Posted by "Olivier Lamy (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MRELEASE-99?page=comments#action_64634 ] 

Olivier Lamy commented on MRELEASE-99:
--------------------------------------

WDYT about adding something like List propagatingProperties (or other name) in the mojo.
And only the configured properties will be added/propagated to the CLI ?
Olivier

> not all system properties used with release:perform
> ---------------------------------------------------
>
>          Key: MRELEASE-99
>          URL: http://jira.codehaus.org/browse/MRELEASE-99
>      Project: Maven 2.x Release Plugin
>         Type: Bug

>  Environment: all
>     Reporter: Olivier Lamy

>
>
> I need to use others systems properties with release:perform.
> Actually only maven.test.skip is passed to the cli used by the plugin.
> In my case, I use a cli like :
> mvn -DtagBase=http://ip:port/path \
> -Dusername=$1 -Dpassword=$2 \
> -Dmaven.test.skip=true -Dgoals=deploy,accorscm:make release:prepare release:perform.
> In cli generated by the plugin, I only see : -Dmaven.test.skip=true.
> My plugin (which commit a war in scm) needs the others insure the authentification.
> An other use case is :
> mvn -DtagBase=http://ip:port/path \
> -Dws.ip=$1 -Dws.port=$2 \
> -Dmaven.test.skip=true -Dgoals=deploy release:prepare release:perform.
> The ip port are needed by the axistools plugin to generate the java classes from the wsdl.
> As I see in the source code the patch is not difficult but I don't know if maven had some properties for internal use (this can polluate the build)
> Thanks,
> Olivier

-- 
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


[jira] Closed: (MRELEASE-99) not all system properties used with release:perform

Posted by "Olivier Lamy (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MRELEASE-99?page=all ]
     
Olivier Lamy closed MRELEASE-99:
--------------------------------

    Resolution: Fixed

Sounds good sorry I have just discovered it .
And all the refactoring concerning this plugin (congratulation code looks great and more human readable ;-) ).
As I see in the sourcecode the -P will be propagated too.
I closed it because I have an other way to do it.
Thanks,
Olivier

> not all system properties used with release:perform
> ---------------------------------------------------
>
>          Key: MRELEASE-99
>          URL: http://jira.codehaus.org/browse/MRELEASE-99
>      Project: Maven 2.x Release Plugin
>         Type: Bug

>  Environment: all
>     Reporter: Olivier Lamy

>
>
> I need to use others systems properties with release:perform.
> Actually only maven.test.skip is passed to the cli used by the plugin.
> In my case, I use a cli like :
> mvn -DtagBase=http://ip:port/path \
> -Dusername=$1 -Dpassword=$2 \
> -Dmaven.test.skip=true -Dgoals=deploy,accorscm:make release:prepare release:perform.
> In cli generated by the plugin, I only see : -Dmaven.test.skip=true.
> My plugin (which commit a war in scm) needs the others insure the authentification.
> An other use case is :
> mvn -DtagBase=http://ip:port/path \
> -Dws.ip=$1 -Dws.port=$2 \
> -Dmaven.test.skip=true -Dgoals=deploy release:prepare release:perform.
> The ip port are needed by the axistools plugin to generate the java classes from the wsdl.
> As I see in the source code the patch is not difficult but I don't know if maven had some properties for internal use (this can polluate the build)
> Thanks,
> Olivier

-- 
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


[jira] Commented: (MRELEASE-99) not all system properties used with release:perform

Posted by "Brett Porter (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MRELEASE-99?page=comments#action_64636 ] 

Brett Porter commented on MRELEASE-99:
--------------------------------------

why not use -Darguments="-Dmaven.test.skip=true" ?

> not all system properties used with release:perform
> ---------------------------------------------------
>
>          Key: MRELEASE-99
>          URL: http://jira.codehaus.org/browse/MRELEASE-99
>      Project: Maven 2.x Release Plugin
>         Type: Bug

>  Environment: all
>     Reporter: Olivier Lamy

>
>
> I need to use others systems properties with release:perform.
> Actually only maven.test.skip is passed to the cli used by the plugin.
> In my case, I use a cli like :
> mvn -DtagBase=http://ip:port/path \
> -Dusername=$1 -Dpassword=$2 \
> -Dmaven.test.skip=true -Dgoals=deploy,accorscm:make release:prepare release:perform.
> In cli generated by the plugin, I only see : -Dmaven.test.skip=true.
> My plugin (which commit a war in scm) needs the others insure the authentification.
> An other use case is :
> mvn -DtagBase=http://ip:port/path \
> -Dws.ip=$1 -Dws.port=$2 \
> -Dmaven.test.skip=true -Dgoals=deploy release:prepare release:perform.
> The ip port are needed by the axistools plugin to generate the java classes from the wsdl.
> As I see in the source code the patch is not difficult but I don't know if maven had some properties for internal use (this can polluate the build)
> Thanks,
> Olivier

-- 
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