rquinio / l10n-maven-plugin

:abcd: Maven plugin to validate localization resources in Java properties files
MIT License
3 stars 1 forks source link

Maven Plugin not thread-safe #11

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Hi,

Please make plugin thread safe, to allow parallel execution ("-T [n]C" maven 
argument).

...
[WARNING] *****************************************************************
[WARNING] * Your build is requesting parallel execution, but project      *
[WARNING] * contains the following plugin(s) that are not marked as       *
[WARNING] * @threadSafe to support parallel building.                     *
[WARNING] * While this /may/ work fine, please look for plugin updates    *
[WARNING] * and/or request plugins be made thread-safe.                   *
[WARNING] * If reporting an issue, report it against the plugin in        *
[WARNING] * question, not against maven-core                              *
[WARNING] *****************************************************************
[WARNING] The following plugins are not marked @threadSafe in [war] 
SID_COMMON_STATIC :: 2.0.2-SNAPSHOT:
[WARNING] com.googlecode.l10n-maven-plugin:l10n-maven-plugin:1.7
[WARNING] *****************************************************************
...

Thanks.
Saad Benbouzid

Original issue reported on code.google.com by Makaveli...@gmail.com on 7 Jul 2014 at 11:03

GoogleCodeExporter commented 9 years ago
Thanks for reporting this.
The plug-in code itself should be threadsafe, but two points need to be checked:
- the dependencies, apparently plexus-utils had some known issues, needs to be 
upgraded
- the logging, to make sure bundle names are logged and item logs are atomic, 
otherwise it could become a mess.

Original comment by romain.q...@gmail.com on 7 Jul 2014 at 9:51

GoogleCodeExporter commented 9 years ago
What is your plugin roadmap ? Is there a hope we can get this bug fixed soon ?

Original comment by Makaveli...@gmail.com on 17 Oct 2014 at 4:15

GoogleCodeExporter commented 9 years ago

Original comment by romain.q...@gmail.com on 30 Nov 2014 at 9:49

GoogleCodeExporter commented 9 years ago
This has been fixed in 1.8: dependencies have been upgraded, and plugin checked 
for thread safety issues. There was no change for the logging, feel free to 
provide feedback if you see some improvements.

Original comment by romain.q...@gmail.com on 1 Dec 2014 at 12:00