Re: Revision numbers in MANIFEST.MF file (was: Re: svn commit: r1069504 - in /myfaces/trinidad/trunk: trinidad-api/pom.xml trinidad-impl/pom.xml)
I have commented it out, again (see vote thread for more information) -M On Fri, Feb 11, 2011 at 12:59 PM, Scott O'Bryan darkar...@gmail.com wrote: +1, I totally agree with Bernd.. Releases have VERSION numbers. It's snapshots that need revision and I see no harm either way. On Feb 11, 2011, at 2:44 AM, Gerhard gerhard.petra...@gmail.com wrote: hi, +1 for adding the revision number in any case. regards, gerhard http://www.irian.at Your JSF powerhouse - JSF Consulting, Development and Courses in English and German Professional Support for Apache MyFaces 2011/2/11 Volker Weber v.we...@inexso.de Hi, i agree to Bernd, the revision number for a release is not needed, but what is the Problem having this few more lines in any manifest? I would prefer having this info packaged. Regards, Volker 2011/2/10 Bernd Bohmann bernd.bohm...@googlemail.com: 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=1069504view=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=1069504r1=1069503r2=1069504view=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 groupIdorg.codehaus.mojo/groupId artifactIdbuildnumber-maven-plugin/artifactId version1.0-beta-4/version @@ -190,7 +191,7 @@ getRevisionOnlyOncetrue/getRevisionOnlyOnce buildNumberPropertyNamescm.revision/buildNumberPropertyName /configuration - /plugin + /plugin-- plugin groupIdorg.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=1069504r1=1069503r2=1069504view=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 groupIdorg.codehaus.mojo/groupId artifactIdbuildnumber-maven-plugin/artifactId version1.0-beta-4/version @@ -229,7 +230,7 @@ getRevisionOnlyOncetrue/getRevisionOnlyOnce buildNumberPropertyNamescm.revision/buildNumberPropertyName /configuration - /plugin + /plugin-- plugin groupIdorg.apache.maven.plugins/groupId -- Matthias Wessendorf blog: http://matthiaswessendorf.wordpress.com/ sessions: http://www.slideshare.net/mwessendorf twitter: http://twitter.com/mwessendorf -- inexso - information exchange solutions GmbH Ofener Str. 30 | 26121 Oldenburg Tel.: +49 441 219 730 56 | FAX: +49 441 219 730 66 | eMail: volker.we...@inexso.de Firmensitz: Oldenburg | Amtsgericht Oldenburg HRB 205251 Geschäftsführer: Stefan Schulte, Michael Terschüren -- Matthias Wessendorf blog: http://matthiaswessendorf.wordpress.com/ sessions: http://www.slideshare.net/mwessendorf twitter: http://twitter.com/mwessendorf
Re: Revision numbers in MANIFEST.MF file (was: Re: svn commit: r1069504 - in /myfaces/trinidad/trunk: trinidad-api/pom.xml trinidad-impl/pom.xml)
Hi, i agree to Bernd, the revision number for a release is not needed, but what is the Problem having this few more lines in any manifest? I would prefer having this info packaged. Regards, Volker 2011/2/10 Bernd Bohmann bernd.bohm...@googlemail.com: 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=1069504view=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=1069504r1=1069503r2=1069504view=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 groupIdorg.codehaus.mojo/groupId artifactIdbuildnumber-maven-plugin/artifactId version1.0-beta-4/version @@ -190,7 +191,7 @@ getRevisionOnlyOncetrue/getRevisionOnlyOnce buildNumberPropertyNamescm.revision/buildNumberPropertyName /configuration - /plugin + /plugin-- plugin groupIdorg.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=1069504r1=1069503r2=1069504view=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 groupIdorg.codehaus.mojo/groupId artifactIdbuildnumber-maven-plugin/artifactId version1.0-beta-4/version @@ -229,7 +230,7 @@ getRevisionOnlyOncetrue/getRevisionOnlyOnce buildNumberPropertyNamescm.revision/buildNumberPropertyName /configuration - /plugin + /plugin-- plugin groupIdorg.apache.maven.plugins/groupId -- Matthias Wessendorf blog: http://matthiaswessendorf.wordpress.com/ sessions: http://www.slideshare.net/mwessendorf twitter: http://twitter.com/mwessendorf -- inexso - information exchange solutions GmbH Ofener Str. 30 | 26121 Oldenburg Tel.: +49 441 219 730 56 | FAX: +49 441 219 730 66 | eMail: volker.we...@inexso.de Firmensitz: Oldenburg | Amtsgericht Oldenburg HRB 205251 Geschäftsführer: Stefan Schulte, Michael Terschüren
Re: Revision numbers in MANIFEST.MF file (was: Re: svn commit: r1069504 - in /myfaces/trinidad/trunk: trinidad-api/pom.xml trinidad-impl/pom.xml)
hi, +1 for adding the revision number in any case. regards, gerhard http://www.irian.at Your JSF powerhouse - JSF Consulting, Development and Courses in English and German Professional Support for Apache MyFaces 2011/2/11 Volker Weber v.we...@inexso.de Hi, i agree to Bernd, the revision number for a release is not needed, but what is the Problem having this few more lines in any manifest? I would prefer having this info packaged. Regards, Volker 2011/2/10 Bernd Bohmann bernd.bohm...@googlemail.com: 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=1069504view=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=1069504r1=1069503r2=1069504view=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 groupIdorg.codehaus.mojo/groupId artifactIdbuildnumber-maven-plugin/artifactId version1.0-beta-4/version @@ -190,7 +191,7 @@ getRevisionOnlyOncetrue/getRevisionOnlyOnce buildNumberPropertyNamescm.revision/buildNumberPropertyName /configuration - /plugin + /plugin-- plugin groupIdorg.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=1069504r1=1069503r2=1069504view=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 groupIdorg.codehaus.mojo/groupId artifactIdbuildnumber-maven-plugin/artifactId version1.0-beta-4/version @@ -229,7 +230,7 @@ getRevisionOnlyOncetrue/getRevisionOnlyOnce buildNumberPropertyNamescm.revision/buildNumberPropertyName /configuration - /plugin + /plugin-- plugin groupIdorg.apache.maven.plugins/groupId -- Matthias Wessendorf blog: http://matthiaswessendorf.wordpress.com/ sessions: http://www.slideshare.net/mwessendorf twitter: http://twitter.com/mwessendorf -- inexso - information exchange solutions GmbH Ofener Str. 30 | 26121 Oldenburg Tel.: +49 441 219 730 56 | FAX: +49 441 219 730 66 | eMail: volker.we...@inexso.de Firmensitz: Oldenburg | Amtsgericht Oldenburg HRB 205251 Geschäftsführer: Stefan Schulte, Michael Terschüren
Re: Revision numbers in MANIFEST.MF file (was: Re: svn commit: r1069504 - in /myfaces/trinidad/trunk: trinidad-api/pom.xml trinidad-impl/pom.xml)
+1, I totally agree with Bernd.. Releases have VERSION numbers. It's snapshots that need revision and I see no harm either way. On Feb 11, 2011, at 2:44 AM, Gerhard gerhard.petra...@gmail.com wrote: hi, +1 for adding the revision number in any case. regards, gerhard http://www.irian.at Your JSF powerhouse - JSF Consulting, Development and Courses in English and German Professional Support for Apache MyFaces 2011/2/11 Volker Weber v.we...@inexso.de Hi, i agree to Bernd, the revision number for a release is not needed, but what is the Problem having this few more lines in any manifest? I would prefer having this info packaged. Regards, Volker 2011/2/10 Bernd Bohmann bernd.bohm...@googlemail.com: 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=1069504view=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=1069504r1=1069503r2=1069504view=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 groupIdorg.codehaus.mojo/groupId artifactIdbuildnumber-maven-plugin/artifactId version1.0-beta-4/version @@ -190,7 +191,7 @@ getRevisionOnlyOncetrue/getRevisionOnlyOnce buildNumberPropertyNamescm.revision/buildNumberPropertyName /configuration - /plugin + /plugin-- plugin groupIdorg.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=1069504r1=1069503r2=1069504view=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 groupIdorg.codehaus.mojo/groupId artifactIdbuildnumber-maven-plugin/artifactId version1.0-beta-4/version @@ -229,7 +230,7 @@ getRevisionOnlyOncetrue/getRevisionOnlyOnce buildNumberPropertyNamescm.revision/buildNumberPropertyName /configuration - /plugin + /plugin-- plugin groupIdorg.apache.maven.plugins/groupId -- Matthias Wessendorf blog: http://matthiaswessendorf.wordpress.com/ sessions: http://www.slideshare.net/mwessendorf twitter: http://twitter.com/mwessendorf -- inexso - information exchange solutions GmbH Ofener Str. 30 | 26121 Oldenburg Tel.: +49 441 219 730 56 | FAX: +49 441 219 730 66 | eMail: volker.we...@inexso.de Firmensitz: Oldenburg | Amtsgericht Oldenburg HRB 205251 Geschäftsführer: Stefan Schulte, Michael Terschüren
Re: Revision numbers in MANIFEST.MF file (was: Re: svn commit: r1069504 - in /myfaces/trinidad/trunk: trinidad-api/pom.xml trinidad-impl/pom.xml)
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=1069504view=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=1069504r1=1069503r2=1069504view=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 groupIdorg.codehaus.mojo/groupId artifactIdbuildnumber-maven-plugin/artifactId version1.0-beta-4/version @@ -190,7 +191,7 @@ getRevisionOnlyOncetrue/getRevisionOnlyOnce buildNumberPropertyNamescm.revision/buildNumberPropertyName /configuration - /plugin + /plugin-- plugin groupIdorg.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=1069504r1=1069503r2=1069504view=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 groupIdorg.codehaus.mojo/groupId artifactIdbuildnumber-maven-plugin/artifactId version1.0-beta-4/version @@ -229,7 +230,7 @@ getRevisionOnlyOncetrue/getRevisionOnlyOnce buildNumberPropertyNamescm.revision/buildNumberPropertyName /configuration - /plugin + /plugin-- plugin groupIdorg.apache.maven.plugins/groupId -- Matthias Wessendorf blog: http://matthiaswessendorf.wordpress.com/ sessions: http://www.slideshare.net/mwessendorf twitter: http://twitter.com/mwessendorf