I see your point.
If we do this the prepare won't be anymore a "repetition" of the real
perform goal (sources, sources bundle, javadoc).
Sure not a big deal but that means folks have some risks to found
issue too late.

So I'm +1.

btw we can add this arguments again in the maven parent pom.

Others WDYT ?


2011/1/27 Benson Margulies <[email protected]>:
> On the compat front, can you think of a reason why removing this from
> prepare would bust anything for anyone?
>
> On Thu, Jan 27, 2011 at 10:40 AM, Benson Margulies
> <[email protected]> wrote:
>> The problem is the use of <arguments> rather than <releaseProfiles>.
>> The later only applies to 'perform'. the former also applies to
>> prepare.
>>
>> So, gpg is turned on for prepare, which takes a long time and requires
>> keys to be present. If you just used releaseProfiles and
>> useReleaseProfiles it would be fine with me.
>>
>> On Thu, Jan 27, 2011 at 10:35 AM, Olivier Lamy <[email protected]> wrote:
>>>
>>> I don't follow you here.
>>> The goal of this profile activation is to generate a set of standard
>>> ASF materials.
>>> As it has been added, removing will means breaking backward comp.
>>> IMHO it's easier to have it here when folks wants to cut a release.
>>>
>>> BTW you can override this in your pom if you don't want any arguments
>>> with something like (from  mind so not tested :-) ) :
>>>
>>>          <groupId>org.apache.maven.plugins</groupId>
>>>          <artifactId>maven-release-plugin</artifactId>
>>>          <configuration>
>>>            <arguments combine.self="override"></arguments>
>>>          </configuration>
>>>
>>>
>>>
>>> 2011/1/27 Benson Margulies <[email protected]>:
>>> > MPOM-2. The fact that the Codehaus jira is the home of issues with the
>>> > ASF shared POM strikes me as something else that needs fixing.
>>> >
>>> > On Thu, Jan 27, 2011 at 9:31 AM, Olivier Lamy <[email protected]> wrote:
>>> >> Hello,
>>> >>
>>> >> The profile apache-release sounds good for adding various release 
>>> >> materials.
>>> >> Can you explain what's wrong or give the jira id ?
>>> >>
>>> >> Thanks
>>> >>
>>> >> 2011/1/27 Benson Margulies <[email protected]>:
>>> >>> I note that  <arguments>-Papache-release</arguments> is still in there.
>>> >>>
>>> >>> I filed a JIRA about the surprising and unpleasant effects of this. I
>>> >>> don't own a -1, but it seems to me that it would be reasonable to ask
>>> >>> you to either remove this or close my JIRA explaining why I'm wrong.
>>> >>>
>>> >>>
>>> >>> On Thu, Jan 27, 2011 at 4:30 AM, Olivier Lamy <[email protected]> wrote:
>>> >>>> Hello Folks,
>>> >>>>
>>> >>>> I'd like to release the ASF parent pom [1].
>>> >>>> In the maven parent pom [2], we have setup a maven-3 profile for the
>>> >>>> site plugin.
>>> >>>> No objections I move this profile to the ASF parent ?
>>> >>>>
>>> >>>> Current diff :
>>> >>>>
>>> >>>> $ svn diff 
>>> >>>> http://svn.apache.org/repos/asf/maven/pom/tags/apache-8/pom.xml
>>> >>>> http://svn.apache.org/repos/asf/maven/pom/trunk/asf/pom.xml
>>> >>>>
>>> >>>> Index: pom.xml
>>> >>>> ===================================================================
>>> >>>> --- pom.xml     (.../tags/apache-8/pom.xml)     (revision 1064055)
>>> >>>> +++ pom.xml     (.../trunk/asf/pom.xml) (revision 1064055)
>>> >>>> @@ -31,7 +31,7 @@
>>> >>>>        -->
>>> >>>>   <groupId>org.apache</groupId>
>>> >>>>   <artifactId>apache</artifactId>
>>> >>>> -  <version>8</version>
>>> >>>> +  <version>9-SNAPSHOT</version>
>>> >>>>   <packaging>pom</packaging>
>>> >>>>   <name>The Apache Software Foundation</name>
>>> >>>>   <description>
>>> >>>> @@ -94,9 +94,9 @@
>>> >>>>     
>>> >>>> <sourceReleaseAssemblyDescriptor>source-release</sourceReleaseAssemblyDescr
>>> >>>> iptor>
>>> >>>>   </properties>
>>> >>>>   <scm>
>>> >>>> -    
>>> >>>> <connection>scm:svn:http://svn.apache.org/repos/asf/maven/pom/tags/apache-8
>>> >>>> </connection>
>>> >>>> -    
>>> >>>> <developerConnection>scm:svn:https://svn.apache.org/repos/asf/maven/pom/tag
>>> >>>> s/apache-8</developerConnection>
>>> >>>> -    <url>http://svn.apache.org/viewvc/maven/pom/tags/apache-8</url>
>>> >>>> +    
>>> >>>> <connection>scm:svn:http://svn.apache.org/repos/asf/maven/pom/trunk/asf</co
>>> >>>> nnection>
>>> >>>> +    
>>> >>>> <developerConnection>scm:svn:https://svn.apache.org/repos/asf/maven/pom/tru
>>> >>>> nk/asf</developerConnection>
>>> >>>> +    <url>http://svn.apache.org/viewvc/maven/pom/trunk/asf</url>
>>> >>>>   </scm>
>>> >>>>
>>> >>>>   <build>
>>> >>>> @@ -111,7 +111,7 @@
>>> >>>>         <plugin>
>>> >>>>           <groupId>org.apache.maven.plugins</groupId>
>>> >>>>           <artifactId>maven-assembly-plugin</artifactId>
>>> >>>> -          <version>2.2-beta-5</version>
>>> >>>> +          <version>2.2</version>
>>> >>>>         </plugin>
>>> >>>>         <plugin>
>>> >>>>           <groupId>org.apache.maven.plugins</groupId>
>>> >>>> @@ -141,7 +141,7 @@
>>> >>>>         <plugin>
>>> >>>>           <groupId>org.apache.maven.plugins</groupId>
>>> >>>>           <artifactId>maven-enforcer-plugin</artifactId>
>>> >>>> -          <version>1.0-beta-1</version>
>>> >>>> +          <version>1.0</version>
>>> >>>>         </plugin>
>>> >>>>         <plugin>
>>> >>>>           <groupId>org.apache.maven.plugins</groupId>
>>> >>>> @@ -186,7 +186,7 @@
>>> >>>>         <plugin>
>>> >>>>           <groupId>org.apache.maven.plugins</groupId>
>>> >>>>           <artifactId>maven-release-plugin</artifactId>
>>> >>>> -          <version>2.0-beta-9</version>
>>> >>>> +          <version>2.1</version>
>>> >>>>           <configuration>
>>> >>>>             <useReleaseProfile>false</useReleaseProfile>
>>> >>>>             <goals>deploy</goals>
>>> >>>> @@ -225,9 +225,14 @@
>>> >>>>         <plugin>
>>> >>>>           <groupId>org.apache.maven.plugins</groupId>
>>> >>>>           <artifactId>maven-surefire-plugin</artifactId>
>>> >>>> -          <version>2.4.3</version>
>>> >>>> +          <version>2.7.2</version>
>>> >>>>         </plugin>
>>> >>>>         <plugin>
>>> >>>> +          <groupId>org.apache.rat</groupId>
>>> >>>> +          <artifactId>apache-rat-plugin</artifactId>
>>> >>>> +          <version>0.7</version>
>>> >>>> +        </plugin>
>>> >>>> +        <plugin>
>>> >>>>           <groupId>org.codehaus.mojo</groupId>
>>> >>>>           <artifactId>clirr-maven-plugin</artifactId>
>>> >>>>           <version>2.2.2</version>
>>> >>>> @@ -240,7 +245,7 @@
>>> >>>>         <plugin>
>>> >>>>           <groupId>org.codehaus.modello</groupId>
>>> >>>>           <artifactId>modello-maven-plugin</artifactId>
>>> >>>> -          <version>1.2</version>
>>> >>>> +          <version>1.4.1</version>
>>> >>>>         </plugin>
>>> >>>>       </plugins>
>>> >>>>     </pluginManagement>
>>> >>>> @@ -333,9 +338,6 @@
>>> >>>>           <plugin>
>>> >>>>             <groupId>org.apache.maven.plugins</groupId>
>>> >>>>             <artifactId>maven-javadoc-plugin</artifactId>
>>> >>>> -            <configuration>
>>> >>>> -              <encoding>${project.build.sourceEncoding}</encoding>
>>> >>>> -            </configuration>
>>> >>>>             <executions>
>>> >>>>               <execution>
>>> >>>>                 <id>attach-javadocs</id>
>>> >>>> @@ -351,6 +353,7 @@
>>> >>>>             <artifactId>maven-gpg-plugin</artifactId>
>>> >>>>             <configuration>
>>> >>>>               <passphrase>${gpg.passphrase}</passphrase>
>>> >>>> +              <useAgent>true</useAgent>
>>> >>>>             </configuration>
>>> >>>>             <executions>
>>> >>>>               <execution>
>>> >>>>
>>> >>>> Thanks !
>>> >>>> --
>>> >>>> Olivier Lamy
>>> >>>> http://twitter.com/olamy
>>> >>>> http://www.linkedin.com/in/olamy
>>> >>>>
>>> >>>> [1] http://svn.apache.org/repos/asf/maven/pom/trunk/asf/pom.xml
>>> >>>> [2] http://svn.apache.org/repos/asf/maven/pom/trunk/maven/pom.xml
>>> >>>>
>>> >>>> ---------------------------------------------------------------------
>>> >>>> To unsubscribe, e-mail: [email protected]
>>> >>>> For additional commands, e-mail: [email protected]
>>> >>>>
>>> >>>>
>>> >>>
>>> >>> ---------------------------------------------------------------------
>>> >>> To unsubscribe, e-mail: [email protected]
>>> >>> For additional commands, e-mail: [email protected]
>>> >>>
>>> >>>
>>> >>
>>> >>
>>> >>
>>> >> --
>>> >> Olivier Lamy
>>> >> http://twitter.com/olamy
>>> >> http://www.linkedin.com/in/olamy
>>> >>
>>> >> ---------------------------------------------------------------------
>>> >> To unsubscribe, e-mail: [email protected]
>>> >> For additional commands, e-mail: [email protected]
>>> >>
>>> >>
>>> >
>>> > ---------------------------------------------------------------------
>>> > To unsubscribe, e-mail: [email protected]
>>> > For additional commands, e-mail: [email protected]
>>> >
>>> >
>>>
>>>
>>>
>>> --
>>> Olivier Lamy
>>> http://twitter.com/olamy
>>> http://www.linkedin.com/in/olamy
>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: [email protected]
>>> For additional commands, e-mail: [email protected]
>>>
>>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [email protected]
> For additional commands, e-mail: [email protected]
>
>



-- 
Olivier Lamy
http://twitter.com/olamy
http://www.linkedin.com/in/olamy

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to