8000 Add @threadSafe to all maven-plugins for parallel build feature of Maven3 · Issue #27 · alx3apps/jgit-buildnumber · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add @threadSafe to all maven-plugins for parallel build feature of Maven3 #27

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
lidel opened this issue Aug 23, 2016 · 3 comments
Open

Add @threadSafe to all maven-plugins for parallel build feature of Maven3 #27

lidel opened this issue Aug 23, 2016 · 3 comments

Comments

@lidel
Copy link
lidel commented Aug 23, 2016

Please add @threadSafe to JGitBuildNumberMojo to get rid of the warning that Maven 3 issues when one of them is involved in a multi module parallel build.

[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 IGT Common Data Model SDK:
[WARNING] ru.concerteza.buildnumber:maven-jgit-buildnumber-plugin:1.2.10
[WARNING] *****************************************************************

I've tested the plugin with runOnlyAtExecutionRoot=true and it works fine. 👌
@jschneider
Copy link

+1. Would love to see that implemeneted

@elab
Copy link
elab commented Jun 23, 2019

I implemented that in my fork (which is also available on maven central).

@Spaum145
Copy link

+1. I want less warnings in my app log. To rise log level is not an option :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@jschneider @lidel @elab @Spaum145
0