Interesting java8 bug....
It would appear that the ThreadPoolExecutor in UNIXProcess can keep the vm running for up to 12 seconds after the last process has been forked. Once you see the symptom it is clearly visible; maven appears to finish the build normally, but there is a short pause after this message and the return of the command prompt: [INFO] [INFO] BUILD SUCCESS [INFO] [INFO] Total time: 2.155s [INFO] Finished at: Thu Oct 09 08:07:38 CEST 2014 [INFO] Final Memory: 14M/334M [INFO] Insert pause here The issue is https://jira.codehaus.org/browse/MEXEC-145 and applies to a truckload of maven stuff since we fork a lot of processes, the bug only seems to affect VM shutdown (which is also NOT very good since we fork a lot of VM's too :). I searched the jdk issue tracker but could not find any evidence of this bug, hence the cc to Rory. Kristian - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
Re: Interesting java8 bug....
Making minimal test cases is always interesting :) It appears there is some additional constraint that needs to be satisfied, so further research needs to be done to determine the exact cause of the problem. It does seem to affect most of the forking code in maven, which should imply some kind of interaction with commons-exec or plexus-utils. Kristian 2014-10-09 9:18 GMT+02:00 Rory O'Donnell Oracle, Dublin Ireland rory.odonn...@oracle.com: Hi Kristian, Can you log a Java Incident and I will follow up ? Rgds,Rory On 09/10/2014 07:14, Kristian Rosenvold wrote: It would appear that the ThreadPoolExecutor in UNIXProcess can keep the vm running for up to 12 seconds after the last process has been forked. Once you see the symptom it is clearly visible; maven appears to finish the build normally, but there is a short pause after this message and the return of the command prompt: [INFO] [INFO] BUILD SUCCESS [INFO] [INFO] Total time: 2.155s [INFO] Finished at: Thu Oct 09 08:07:38 CEST 2014 [INFO] Final Memory: 14M/334M [INFO] Insert pause here The issue is https://jira.codehaus.org/browse/MEXEC-145 and applies to a truckload of maven stuff since we fork a lot of processes, the bug only seems to affect VM shutdown (which is also NOT very good since we fork a lot of VM's too :). I searched the jdk issue tracker but could not find any evidence of this bug, hence the cc to Rory. Kristian -- Rgds,Rory O'Donnell Quality Engineering Manager Oracle EMEA , Dublin, Ireland - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
Re: Interesting java8 bug....
On Thu, 09 Oct 2014 08:18:34 +0100, Rory O'Donnell Oracle, Dublin Ireland wrote: Can you log a Java Incident and I will follow up ? I am quite certain the problem has to do with tiered compilation being on by default in JDK8 and kicking in/still running while the VM tries to exit after a brief run. So far the best way I found to mitigate this problem is to set -XX:CICompilerCount=1 -XX:TieredStopAtLevel=1 in $MAVEN_OPTS. That seems to work for me, and gives much faster and more consistent startup times as well. The fact that less code is JITted early does not seem to matter much. -h - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
Re: Interesting java8 bug....
Wow, that certainly fixed the problem. Stuff seems noticeably faster, too ! Do you know if this has been reported to the JDK ? Kristian 2014-10-09 10:41 GMT+02:00 Holger Hoffstätte holger.hoffstae...@googlemail.com: On Thu, 09 Oct 2014 08:18:34 +0100, Rory O'Donnell Oracle, Dublin Ireland wrote: Can you log a Java Incident and I will follow up ? I am quite certain the problem has to do with tiered compilation being on by default in JDK8 and kicking in/still running while the VM tries to exit after a brief run. So far the best way I found to mitigate this problem is to set -XX:CICompilerCount=1 -XX:TieredStopAtLevel=1 in $MAVEN_OPTS. That seems to work for me, and gives much faster and more consistent startup times as well. The fact that less code is JITted early does not seem to matter much. -h - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
Re: Interesting java8 bug....
Hi Kristian, you can log an incident at bugs.sun.com. Pass me the incident number and I will monitor it's progress. Rgds,Rory On 09/10/2014 09:38, Kristian Rosenvold wrote: Making minimal test cases is always interesting :) It appears there is some additional constraint that needs to be satisfied, so further research needs to be done to determine the exact cause of the problem. It does seem to affect most of the forking code in maven, which should imply some kind of interaction with commons-exec or plexus-utils. Kristian 2014-10-09 9:18 GMT+02:00 Rory O'Donnell Oracle, Dublin Ireland rory.odonn...@oracle.com: Hi Kristian, Can you log a Java Incident and I will follow up ? Rgds,Rory On 09/10/2014 07:14, Kristian Rosenvold wrote: It would appear that the ThreadPoolExecutor in UNIXProcess can keep the vm running for up to 12 seconds after the last process has been forked. Once you see the symptom it is clearly visible; maven appears to finish the build normally, but there is a short pause after this message and the return of the command prompt: [INFO] [INFO] BUILD SUCCESS [INFO] [INFO] Total time: 2.155s [INFO] Finished at: Thu Oct 09 08:07:38 CEST 2014 [INFO] Final Memory: 14M/334M [INFO] Insert pause here The issue is https://jira.codehaus.org/browse/MEXEC-145 and applies to a truckload of maven stuff since we fork a lot of processes, the bug only seems to affect VM shutdown (which is also NOT very good since we fork a lot of VM's too :). I searched the jdk issue tracker but could not find any evidence of this bug, hence the cc to Rory. Kristian -- Rgds,Rory O'Donnell Quality Engineering Manager Oracle EMEA , Dublin, Ireland - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org -- Rgds,Rory O'Donnell Quality Engineering Manager Oracle EMEA , Dublin, Ireland - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
Re: Interesting java8 bug....
Hi Holger, I am quite certain the problem has to do with tiered compilation being on by default in JDK8 and kicking in/still Isn't it on by default in JDK7 as well (last I can remember it was turned on in one of the builds and stayed on). Dawid - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
Re: Interesting java8 bug....
On Thu, 09 Oct 2014 10:54:35 +0200, Kristian Rosenvold wrote: Wow, that certainly fixed the problem. Stuff seems noticeably faster, too ! Do you know if this has been reported to the JDK ? Good to know that you see the same effect. :) I think I read that the JRuby folks set this by default as well. No idea if this has been reported; it's not citical for anything that stays running for more than a few seconds, and since it's so very timing-dependent/not easily reproducible probably nobody has noticed/investigated yet.. -h - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
Early Access builds for JDK 9 b33 and JDK 8u40 b09 are available on java.net
Hi Robert Kristian, Early Access build for JDK 9 b33 https://jdk9.java.net/download/ is available on java.net, summary of changes are listed here http://www.java.net/download/jdk9/changes/jdk9-b33.html Early Access build for JDK 8u40 b09 http://jdk8.java.net/download.html is available on java.net, summary of changes are listed here. http://www.java.net/download/jdk8u40/changes/jdk8u40-b09.html Rgds,Rory -- Rgds,Rory O'Donnell Quality Engineering Manager Oracle EMEA , Dublin, Ireland
Re: [VOTE] Release Apache Maven Source Plugin version 2.4
Hi, here is my +1 for this release. On 10/7/14 9:45 PM, Karl Heinz Marbaise wrote: Hi, We solved 4 issues: http://jira.codehaus.org/secure/ReleaseNote.jspa?projectId=11147version=20597 There are still a couple of issues left in JIRA: http://jira.codehaus.org/issues/?jql=project%20%3D%20MSOURCES%20AND%20status%20%3D%20Open%20ORDER%20BY%20key%20DESC%2C%20priority%20DESC Staging repo: https://repository.apache.org/content/repositories/maven-1072/ http://repository.apache.org/content/repositories/maven-1072/org/apache/maven/plugins/maven-source-plugin/2.4/maven-source-plugin-2.4-source-release.zip Source release checksum(s): maven-source-plugin-2.4-source-release.zip sha1: da0d6e369f4841d018ccd29d44f3b70afb42d662 Staging site: http://maven.apache.org/plugins-archives/maven-source-plugin-LATEST/ Guide to testing staged releases: http://maven.apache.org/guides/development/guide-testing-releases.html Vote open for 72 hours. [ ] +1 [ ] +0 [ ] -1 Kind regards Karl-Heinz Marbaise - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
Re: [VOTE] Release Apache Maven LinkCheck Plugin version 1.2
Hi, here is my +1. On 10/8/14 1:30 PM, Karl Heinz Marbaise wrote: Hi, We solved 7 issues: http://jira.codehaus.org/secure/ReleaseNote.jspa?projectId=12100version=16906 There is still one issues left in JIRA: http://jira.codehaus.org/issues/?jql=project%20%3D%20MLINKCHECK%20AND%20status%20%3D%20Open%20ORDER%20BY%20priority%20DESC Staging repo: https://repository.apache.org/content/repositories/maven-1073 http://repository.apache.org/content/repositories/maven-1073/org/apache/maven/plugins/maven-linkcheck-plugin/1.2/maven-linkcheck-plugin-1.2-source-release.zip Source release checksum(s): maven-linkcheck-plugin-1.2-source-release.zip sha1: ee5ffd6fa81a9d49fd833bb78f1730283ef18df4 Staging site: http://maven.apache.org/plugins-archives/maven-linkcheck-plugin-LATEST/ Guide to testing staged releases: http://maven.apache.org/guides/development/guide-testing-releases.html Vote open for 72 hours. [ ] +1 [ ] +0 [ ] -1 Kind regards Karl-Heinz Marbaise - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
Re: svn commit: r1630358 [1/6] - in /maven/plugins/trunk: maven-acr-plugin/ maven-ant-plugin/ maven-antrun-plugin/ maven-assembly-plugin/ maven-changelog-plugin/ maven-changes-plugin/ maven-checkstyle
Hi Karl-Heinz, there was a discussion about this: are LICENSE and NOTICE also required for the project-roots or only for the distributions? The latter is already done. Please read http://maven.markmail.org/thread/jlw4rjte2snu4tck thanks, Robert Op Thu, 09 Oct 2014 11:49:33 +0200 schreef khmarba...@apache.org: Author: khmarbaise Date: Thu Oct 9 09:49:32 2014 New Revision: 1630358 URL: http://svn.apache.org/r1630358 Log: - Added to all plugins LICENSE file - Added to all plugins NOTICE file with copyright year 2007-2014 Added: maven/plugins/trunk/maven-acr-plugin/LICENSE maven/plugins/trunk/maven-acr-plugin/NOTICE maven/plugins/trunk/maven-ant-plugin/LICENSE maven/plugins/trunk/maven-ant-plugin/NOTICE maven/plugins/trunk/maven-antrun-plugin/LICENSE maven/plugins/trunk/maven-antrun-plugin/NOTICE maven/plugins/trunk/maven-assembly-plugin/LICENSE maven/plugins/trunk/maven-assembly-plugin/NOTICE maven/plugins/trunk/maven-changelog-plugin/LICENSE maven/plugins/trunk/maven-changelog-plugin/NOTICE maven/plugins/trunk/maven-changes-plugin/LICENSE maven/plugins/trunk/maven-changes-plugin/NOTICE maven/plugins/trunk/maven-checkstyle-plugin/LICENSE maven/plugins/trunk/maven-checkstyle-plugin/NOTICE maven/plugins/trunk/maven-clean-plugin/LICENSE maven/plugins/trunk/maven-clean-plugin/NOTICE maven/plugins/trunk/maven-compiler-plugin/LICENSE maven/plugins/trunk/maven-compiler-plugin/NOTICE maven/plugins/trunk/maven-dependency-plugin/LICENSE maven/plugins/trunk/maven-dependency-plugin/NOTICE maven/plugins/trunk/maven-deploy-plugin/LICENSE maven/plugins/trunk/maven-deploy-plugin/NOTICE maven/plugins/trunk/maven-doap-plugin/LICENSE maven/plugins/trunk/maven-doap-plugin/NOTICE maven/plugins/trunk/maven-docck-plugin/LICENSE maven/plugins/trunk/maven-docck-plugin/NOTICE maven/plugins/trunk/maven-ear-plugin/LICENSE maven/plugins/trunk/maven-ear-plugin/NOTICE maven/plugins/trunk/maven-eclipse-plugin/LICENSE maven/plugins/trunk/maven-eclipse-plugin/NOTICE maven/plugins/trunk/maven-ejb-plugin/LICENSE maven/plugins/trunk/maven-ejb-plugin/NOTICE maven/plugins/trunk/maven-gpg-plugin/LICENSE maven/plugins/trunk/maven-gpg-plugin/NOTICE maven/plugins/trunk/maven-help-plugin/LICENSE maven/plugins/trunk/maven-help-plugin/NOTICE maven/plugins/trunk/maven-invoker-plugin/LICENSE maven/plugins/trunk/maven-invoker-plugin/NOTICE maven/plugins/trunk/maven-jar-plugin/LICENSE maven/plugins/trunk/maven-jar-plugin/NOTICE maven/plugins/trunk/maven-jarsigner-plugin/LICENSE maven/plugins/trunk/maven-jarsigner-plugin/NOTICE maven/plugins/trunk/maven-javadoc-plugin/LICENSE maven/plugins/trunk/maven-javadoc-plugin/NOTICE maven/plugins/trunk/maven-linkcheck-plugin/LICENSE maven/plugins/trunk/maven-linkcheck-plugin/NOTICE maven/plugins/trunk/maven-patch-plugin/LICENSE maven/plugins/trunk/maven-patch-plugin/NOTICE maven/plugins/trunk/maven-pdf-plugin/LICENSE maven/plugins/trunk/maven-pdf-plugin/NOTICE maven/plugins/trunk/maven-plugins/NOTICE maven/plugins/trunk/maven-pmd-plugin/LICENSE maven/plugins/trunk/maven-pmd-plugin/NOTICE maven/plugins/trunk/maven-project-info-reports-plugin/LICENSE maven/plugins/trunk/maven-project-info-reports-plugin/NOTICE maven/plugins/trunk/maven-rar-plugin/LICENSE maven/plugins/trunk/maven-rar-plugin/NOTICE maven/plugins/trunk/maven-remote-resources-plugin/LICENSE maven/plugins/trunk/maven-remote-resources-plugin/NOTICE maven/plugins/trunk/maven-repository-plugin/LICENSE maven/plugins/trunk/maven-repository-plugin/NOTICE maven/plugins/trunk/maven-resources-plugin/LICENSE maven/plugins/trunk/maven-resources-plugin/NOTICE maven/plugins/trunk/maven-scm-publish-plugin/LICENSE maven/plugins/trunk/maven-scm-publish-plugin/NOTICE maven/plugins/trunk/maven-shade-plugin/LICENSE maven/plugins/trunk/maven-shade-plugin/NOTICE maven/plugins/trunk/maven-site-plugin/LICENSE maven/plugins/trunk/maven-site-plugin/NOTICE maven/plugins/trunk/maven-source-plugin/LICENSE maven/plugins/trunk/maven-source-plugin/NOTICE maven/plugins/trunk/maven-stage-plugin/LICENSE maven/plugins/trunk/maven-stage-plugin/NOTICE maven/plugins/trunk/maven-toolchains-plugin/LICENSE maven/plugins/trunk/maven-toolchains-plugin/NOTICE maven/plugins/trunk/maven-verifier-plugin/LICENSE maven/plugins/trunk/maven-verifier-plugin/NOTICE maven/plugins/trunk/maven-war-plugin/LICENSE maven/plugins/trunk/maven-war-plugin/NOTICE Modified: maven/plugins/trunk/maven-install-plugin/NOTICE Added: maven/plugins/trunk/maven-acr-plugin/LICENSE URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-acr-plugin/LICENSE?rev=1630358view=auto == ---
Re: [VOTE] Release Apache Maven LinkCheck Plugin version 1.2
Op Wed, 08 Oct 2014 22:35:08 +0200 schreef Karl Heinz Marbaise khmarba...@gmx.de: Hi Robert, +1 for the code/artifact I'm a bit surprised by the site. The menu seems outdated: e.g. it contains a reference to maven-1.x Do you mean the links under Maven Projects ? Yes Kind regards Karl-Heinz Marbaise thanks, Robert Op Wed, 08 Oct 2014 13:30:19 +0200 schreef Karl Heinz Marbaise khmarba...@gmx.de: Hi, We solved 7 issues: http://jira.codehaus.org/secure/ReleaseNote.jspa?projectId=12100version=16906 There is still one issues left in JIRA: http://jira.codehaus.org/issues/?jql=project%20%3D%20MLINKCHECK%20AND%20status%20%3D%20Open%20ORDER%20BY%20priority%20DESC Staging repo: https://repository.apache.org/content/repositories/maven-1073 http://repository.apache.org/content/repositories/maven-1073/org/apache/maven/plugins/maven-linkcheck-plugin/1.2/maven-linkcheck-plugin-1.2-source-release.zip Source release checksum(s): maven-linkcheck-plugin-1.2-source-release.zip sha1: ee5ffd6fa81a9d49fd833bb78f1730283ef18df4 Staging site: http://maven.apache.org/plugins-archives/maven-linkcheck-plugin-LATEST/ Guide to testing staged releases: http://maven.apache.org/guides/development/guide-testing-releases.html Vote open for 72 hours. [ ] +1 [ ] +0 [ ] -1 Kind regards Karl-Heinz Marbaise - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
Assembly plugin bug scrub
With plexus-io (2.1.4) and plexus-archiver (2.6.4) in a smashing state, I figured I'd go bug scrubbing in assembly-plugin, which has a bug list the size of something big. I am planning to release in the second half of next week, if anyone wants to chip in ! Kristian - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
Re: svn commit: r1630358 [1/6] - in /maven/plugins/trunk: maven-acr-plugin/ maven-ant-plugin/ maven-antrun-plugin/ maven-assembly-plugin/ maven-changelog-plugin/ maven-changes-plugin/ maven-checkstyle
Hi Robert, On 10/9/14 7:23 PM, Robert Scholte wrote: there was a discussion about this: are LICENSE and NOTICE also required for the project-roots or only for the distributions? The latter is already done. For the distribution archives i know...so you don't see the need at the momemt for adding them in svn ? I can simply undone this... Kind regards Karl Heinz Marbaise - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
Re: Assembly plugin bug scrub
Hi Kristian, On 10/9/14 7:31 PM, Kristian Rosenvold wrote: With plexus-io (2.1.4) and plexus-archiver (2.6.4) in a smashing state, I figured I'd go bug scrubbing in assembly-plugin, which has a bug list the size of something big. I am planning to release in the second half of next week, if anyone wants to chip in ! I'm already on some of the issues...If i can help...sure...two pair of eyes see more than one pair of eyes... Kind regards Karl-Heinz Marbaise - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
Re: svn commit: r1630358 [1/6] - in /maven/plugins/trunk: maven-acr-plugin/ maven-ant-plugin/ maven-antrun-plugin/ maven-assembly-plugin/ maven-changelog-plugin/ maven-changes-plugin/ maven-checkstyle
Putting them in the svn project roots would mean maintenance, e.g. the generated LICENSE auto-updates the years. For that reason I wouldn't put them there. IIRC the ASF is not very clear about it, so for that reason there hasn't been made a real decision. thanks, Robert Op Thu, 09 Oct 2014 20:03:05 +0200 schreef Karl Heinz Marbaise khmarba...@gmx.de: Hi Robert, On 10/9/14 7:23 PM, Robert Scholte wrote: there was a discussion about this: are LICENSE and NOTICE also required for the project-roots or only for the distributions? The latter is already done. For the distribution archives i know...so you don't see the need at the momemt for adding them in svn ? I can simply undone this... Kind regards Karl Heinz Marbaise - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
Re: svn commit: r1630358 [1/6] - in /maven/plugins/trunk: maven-acr-plugin/ maven-ant-plugin/ maven-antrun-plugin/ maven-assembly-plugin/ maven-changelog-plugin/ maven-changes-plugin/ maven-checkstyle
If I recall correctly, we are still waiting for a response whether this is necessary or not. Personally I'm against adding these files in version control because of the unwanted maintenance, as Robert noted. Remember that we have over 100 release roots that we need to maintain. -- Dennis Lundberg Den 9 okt 2014 20:47 skrev Robert Scholte rfscho...@apache.org: Putting them in the svn project roots would mean maintenance, e.g. the generated LICENSE auto-updates the years. For that reason I wouldn't put them there. IIRC the ASF is not very clear about it, so for that reason there hasn't been made a real decision. thanks, Robert Op Thu, 09 Oct 2014 20:03:05 +0200 schreef Karl Heinz Marbaise khmarba...@gmx.de: Hi Robert, On 10/9/14 7:23 PM, Robert Scholte wrote: there was a discussion about this: are LICENSE and NOTICE also required for the project-roots or only for the distributions? The latter is already done. For the distribution archives i know...so you don't see the need at the momemt for adding them in svn ? I can simply undone this... Kind regards Karl Heinz Marbaise - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
Re: svn commit: r1630358 [1/6] - in /maven/plugins/trunk: maven-acr-plugin/ maven-ant-plugin/ maven-antrun-plugin/ maven-assembly-plugin/ maven-changelog-plugin/ maven-changes-plugin/ maven-checkstyle
Hi, So it would be better to remove them...sorry for this... Ok...i will undo the change... Kind regards Karl Heinz Marbaise corret On 10/9/14 10:28 PM, Dennis Lundberg wrote: If I recall correctly, we are still waiting for a response whether this is necessary or not. Personally I'm against adding these files in version control because of the unwanted maintenance, as Robert noted. Remember that we have over 100 release roots that we need to maintain. -- Dennis Lundberg Den 9 okt 2014 20:47 skrev Robert Scholte rfscho...@apache.org: Putting them in the svn project roots would mean maintenance, e.g. the generated LICENSE auto-updates the years. For that reason I wouldn't put them there. IIRC the ASF is not very clear about it, so for that reason there hasn't been made a real decision. thanks, Robert Op Thu, 09 Oct 2014 20:03:05 +0200 schreef Karl Heinz Marbaise khmarba...@gmx.de: Hi Robert, On 10/9/14 7:23 PM, Robert Scholte wrote: there was a discussion about this: are LICENSE and NOTICE also required for the project-roots or only for the distributions? The latter is already done. For the distribution archives i know...so you don't see the need at the momemt for adding them in svn ? I can simply undone this... Kind regards Karl Heinz Marbaise - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
Re: svn commit: r1630358 [1/6] - in /maven/plugins/trunk: maven-acr-plugin/ maven-ant-plugin/ maven-antrun-plugin/ maven-assembly-plugin/ maven-changelog-plugin/ maven-changes-plugin/ maven-checkstyle
Hi, undone the change... Kind regards Karl-Heinz Marbaise On 10/9/14 10:34 PM, Karl Heinz Marbaise wrote: Hi, So it would be better to remove them...sorry for this... Ok...i will undo the change... Kind regards Karl Heinz Marbaise corret On 10/9/14 10:28 PM, Dennis Lundberg wrote: If I recall correctly, we are still waiting for a response whether this is necessary or not. Personally I'm against adding these files in version control because of the unwanted maintenance, as Robert noted. Remember that we have over 100 release roots that we need to maintain. -- Dennis Lundberg Den 9 okt 2014 20:47 skrev Robert Scholte rfscho...@apache.org: Putting them in the svn project roots would mean maintenance, e.g. the generated LICENSE auto-updates the years. For that reason I wouldn't put them there. IIRC the ASF is not very clear about it, so for that reason there hasn't been made a real decision. thanks, Robert Op Thu, 09 Oct 2014 20:03:05 +0200 schreef Karl Heinz Marbaise khmarba...@gmx.de: Hi Robert, On 10/9/14 7:23 PM, Robert Scholte wrote: there was a discussion about this: are LICENSE and NOTICE also required for the project-roots or only for the distributions? The latter is already done. For the distribution archives i know...so you don't see the need at the momemt for adding them in svn ? I can simply undone this... Kind regards Karl Heinz Marbaise - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org