Delos,

I know you thanked Hong Fang Han for the patch in the JIRA, but is it
not also the practice to include the thanks in the svn commit log
message?  I have been dormant in the community for a while, and need
to get up to speed with current practice.

Thanks,

Ted

On Fri, Oct 15, 2010 at 5:36 AM,  <[email protected]> wrote:
> Author: delos
> Date: Fri Oct 15 09:36:28 2010
> New Revision: 1022868
>
> URL: http://svn.apache.org/viewvc?rev=1022868&view=rev
> Log:
> GERONIMODEVTOOLS-676 update version number of dependency and moduels to make 
> 2.1 branch build successfully
>
> Modified:
>    geronimo/devtools/eclipse-plugin/branches/2.1.7/eclipse/pom.xml
>    
> geronimo/devtools/eclipse-plugin/branches/2.1.7/plugins/org.apache.geronimo.runtime.common/pom.xml
>    
> geronimo/devtools/eclipse-plugin/branches/2.1.7/plugins/org.apache.geronimo.runtime.v21/pom.xml
>    geronimo/devtools/eclipse-plugin/branches/2.1.7/pom.xml
>
> Modified: geronimo/devtools/eclipse-plugin/branches/2.1.7/eclipse/pom.xml
> URL: 
> http://svn.apache.org/viewvc/geronimo/devtools/eclipse-plugin/branches/2.1.7/eclipse/pom.xml?rev=1022868&r1=1022867&r2=1022868&view=diff
> ==============================================================================
> --- geronimo/devtools/eclipse-plugin/branches/2.1.7/eclipse/pom.xml (original)
> +++ geronimo/devtools/eclipse-plugin/branches/2.1.7/eclipse/pom.xml Fri Oct 
> 15 09:36:28 2010
> @@ -21,14 +21,14 @@
>
>     <groupId>org.apache.geronimo.devtools</groupId>
>     <artifactId>eclipse</artifactId>
> -    <version>3.0.0</version>
> +    <version>2.1.7</version>
>     <packaging>pom</packaging>
>     <name>Geronimo Eclipse Plugin :: Eclipse</name>
>
>     <parent>
>         <groupId>org.apache.geronimo.devtools</groupId>
>         <artifactId>geronimo-eclipse-plugin</artifactId>
> -        <version>3.0.0-SNAPSHOT</version>
> +        <version>2.1.7-SNAPSHOT</version>
>         <relativePath>../pom.xml</relativePath>
>     </parent>
>
>
> Modified: 
> geronimo/devtools/eclipse-plugin/branches/2.1.7/plugins/org.apache.geronimo.runtime.common/pom.xml
> URL: 
> http://svn.apache.org/viewvc/geronimo/devtools/eclipse-plugin/branches/2.1.7/plugins/org.apache.geronimo.runtime.common/pom.xml?rev=1022868&r1=1022867&r2=1022868&view=diff
> ==============================================================================
> --- 
> geronimo/devtools/eclipse-plugin/branches/2.1.7/plugins/org.apache.geronimo.runtime.common/pom.xml
>  (original)
> +++ 
> geronimo/devtools/eclipse-plugin/branches/2.1.7/plugins/org.apache.geronimo.runtime.common/pom.xml
>  Fri Oct 15 09:36:28 2010
> @@ -190,7 +190,7 @@
>         <dependency>
>             <groupId>org.apache.geronimo.devtools</groupId>
>             <artifactId>eclipse</artifactId>
> -            <version>${version}</version>
> +            <version>${pom.version}</version>
>             <type>pom</type>
>         </dependency>
>         <dependency>
>
> Modified: 
> geronimo/devtools/eclipse-plugin/branches/2.1.7/plugins/org.apache.geronimo.runtime.v21/pom.xml
> URL: 
> http://svn.apache.org/viewvc/geronimo/devtools/eclipse-plugin/branches/2.1.7/plugins/org.apache.geronimo.runtime.v21/pom.xml?rev=1022868&r1=1022867&r2=1022868&view=diff
> ==============================================================================
> --- 
> geronimo/devtools/eclipse-plugin/branches/2.1.7/plugins/org.apache.geronimo.runtime.v21/pom.xml
>  (original)
> +++ 
> geronimo/devtools/eclipse-plugin/branches/2.1.7/plugins/org.apache.geronimo.runtime.v21/pom.xml
>  Fri Oct 15 09:36:28 2010
> @@ -27,7 +27,7 @@
>         <relativePath>../pom.xml</relativePath>
>     </parent>
>     <properties>
> -        <geronimoVersion>2.1.6</geronimoVersion>
> +        <geronimoVersion>2.1.7-SNAPSHOT</geronimoVersion>
>     </properties>
>     <build>
>         <resources>
>
> Modified: geronimo/devtools/eclipse-plugin/branches/2.1.7/pom.xml
> URL: 
> http://svn.apache.org/viewvc/geronimo/devtools/eclipse-plugin/branches/2.1.7/pom.xml?rev=1022868&r1=1022867&r2=1022868&view=diff
> ==============================================================================
> --- geronimo/devtools/eclipse-plugin/branches/2.1.7/pom.xml (original)
> +++ geronimo/devtools/eclipse-plugin/branches/2.1.7/pom.xml Fri Oct 15 
> 09:36:28 2010
> @@ -263,7 +263,7 @@
>                 <plugin>
>                     <groupId>org.apache.geronimo.devtools</groupId>
>                     <artifactId>maven-eclipsepde-plugin</artifactId>
> -                    <version>1.0</version>
> +                    <version>1.1</version>
>                     <configuration>
>                         
> <eclipseHome>${settings.localRepository}/eclipse/eclipse/</eclipseHome>
>                     </configuration>
>
>
>

Reply via email to