Hello Matthias, For a release the revision number is not needed. For a snapshot it might be helpful if someone reports a bug and it's not clear with revision was the base for the snapshot.
Regards Bernd Am 10.02.2011 19:23 schrieb "Matthias Wessendorf" <mat...@apache.org>: > Having the actual revision number inside of the manifest.mf file is nice. > > However, not sure if that is really needed for every build, therefore > I commented it out. > > Perhaps this should be done only in the release profile ? > > What do you think ? > > -Matthias > > On Thu, Feb 10, 2011 at 7:05 PM, <mat...@apache.org> wrote: >> Author: matzew >> Date: Thu Feb 10 18:05:24 2011 >> New Revision: 1069504 >> >> URL: http://svn.apache.org/viewvc?rev=1069504&view=rev >> Log: >> disabling the svn revision number plugin - should it be done only on release profile??? >> >> Modified: >> myfaces/trinidad/trunk/trinidad-api/pom.xml >> myfaces/trinidad/trunk/trinidad-impl/pom.xml >> >> Modified: myfaces/trinidad/trunk/trinidad-api/pom.xml >> URL: http://svn.apache.org/viewvc/myfaces/trinidad/trunk/trinidad-api/pom.xml?rev=1069504&r1=1069503&r2=1069504&view=diff >> ============================================================================== >> --- myfaces/trinidad/trunk/trinidad-api/pom.xml (original) >> +++ myfaces/trinidad/trunk/trinidad-api/pom.xml Thu Feb 10 18:05:24 2011 >> @@ -172,7 +172,8 @@ >> <!-- >> To make the current revision number, we use the buildnumber-maven-plugin. >> --> >> - <plugin> >> + <!-- Perhaps this should be only enabled on release profile ? --> >> + <!--plugin> >> <groupId>org.codehaus.mojo</groupId> >> <artifactId>buildnumber-maven-plugin</artifactId> >> <version>1.0-beta-4</version> >> @@ -190,7 +191,7 @@ >> <getRevisionOnlyOnce>true</getRevisionOnlyOnce> >> <buildNumberPropertyName>scm.revision</buildNumberPropertyName> >> </configuration> >> - </plugin> >> + </plugin--> >> >> <plugin> >> <groupId>org.apache.maven.plugins</groupId> >> >> Modified: myfaces/trinidad/trunk/trinidad-impl/pom.xml >> URL: http://svn.apache.org/viewvc/myfaces/trinidad/trunk/trinidad-impl/pom.xml?rev=1069504&r1=1069503&r2=1069504&view=diff >> ============================================================================== >> --- myfaces/trinidad/trunk/trinidad-impl/pom.xml (original) >> +++ myfaces/trinidad/trunk/trinidad-impl/pom.xml Thu Feb 10 18:05:24 2011 >> @@ -211,7 +211,8 @@ >> <!-- >> To make the current revision number, we use the buildnumber-maven-plugin. >> --> >> - <plugin> >> + <!-- Perhaps this should be only enabled on release profile ? --> >> + <!--plugin> >> <groupId>org.codehaus.mojo</groupId> >> <artifactId>buildnumber-maven-plugin</artifactId> >> <version>1.0-beta-4</version> >> @@ -229,7 +230,7 @@ >> <getRevisionOnlyOnce>true</getRevisionOnlyOnce> >> <buildNumberPropertyName>scm.revision</buildNumberPropertyName> >> </configuration> >> - </plugin> >> + </plugin--> >> >> <plugin> >> <groupId>org.apache.maven.plugins</groupId> >> >> >> > > > > -- > Matthias Wessendorf > > blog: http://matthiaswessendorf.wordpress.com/ > sessions: http://www.slideshare.net/mwessendorf > twitter: http://twitter.com/mwessendorf