Skip to content
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.

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

Please release a new version that includes the threadSafe fix #54

Closed
XN137 opened this issue Mar 10, 2022 · 11 comments
Closed

Please release a new version that includes the threadSafe fix #54

XN137 opened this issue Mar 10, 2022 · 11 comments
Assignees

Comments

@XN137
Copy link

XN137 commented Mar 10, 2022

this was fixed in #27 however no new version was released since then.

we would like to avoid these warnings in all of our modules:

2022-03-10T07:47:46.4483999Z [INFO] --------------------------------[ jar ]---------------------------------
2022-03-10T07:47:46.4484394Z [WARNING] *****************************************************************
2022-03-10T07:47:46.4484675Z [WARNING] * Your build is requesting parallel execution, but project      *
2022-03-10T07:47:46.4484980Z [WARNING] * contains the following plugin(s) that have goals not marked   *
2022-03-10T07:47:46.4485294Z [WARNING] * as @threadSafe to support parallel building.                  *
2022-03-10T07:47:46.4485660Z [WARNING] * While this /may/ work fine, please look for plugin updates    *
2022-03-10T07:47:46.4486065Z [WARNING] * and/or request plugins be made thread-safe.                   *
2022-03-10T07:47:46.4486369Z [WARNING] * If reporting an issue, report it against the plugin in        *
2022-03-10T07:47:46.4486730Z [WARNING] * question, not against maven-core                              *
2022-03-10T07:47:46.4486979Z [WARNING] *****************************************************************
2022-03-10T07:47:46.4487376Z [WARNING] The following plugins are not marked @threadSafe in (project):
2022-03-10T07:47:46.4487791Z [WARNING] org.codehaus.mojo:tidy-maven-plugin:1.1.0
2022-03-10T07:47:46.4488107Z [WARNING] Enable debug to see more precisely which goals are not marked @threadSafe.
2022-03-10T07:47:46.4488392Z [WARNING] *****************************************************************

Thank you

@mohanaraosv
Copy link

Bump!! It's been so long since the last version is released. Please consider for new release.

@bmarwell
Copy link
Contributor

bmarwell commented Aug 9, 2022

Can do, but does it still work properly?

@Stephan202
Copy link
Contributor

@bmarwell we're using this plugin in hundreds/thousands of concurrent builds each day (running mvn tidy:pom) and I don't recall any concurrency issue that was traced back to this plugin. 👍

@mohanaraosv
Copy link

Even I use this plugin in concurrent multi-module builds and never faced any issue.

@bmarwell
Copy link
Contributor

bmarwell commented Aug 9, 2022

Great! Just asking because of the recent java11+ builds. Thanks for your input!

@Stephan202
Copy link
Contributor

Can confirm compatibility with both JDK 11 and 17 👍

@pzygielo
Copy link
Contributor

Please.

@slawekjaranowski
Copy link
Member

Before release please help with:

  • upgrade parent to the latest version
  • cleanup of configuration in pom which is provided by parent
  • upgrade Maven api to 3.2.5 - Maven core component in provided scope
  • check/fix site output

@pzygielo
Copy link
Contributor

Before release please help with:
* check/fix site output

Noticed changes with site generated from 40650bb compared with d151067:

@slawekjaranowski slawekjaranowski self-assigned this Sep 24, 2022
@slawekjaranowski
Copy link
Member

@pzygielo thanks ...

I will try release until end of month

@slawekjaranowski
Copy link
Member

Done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants