[GitHub] [maven-scm] pzygielo commented on pull request #107: [SCM-946] Added @threadSafe for parallel execution

2021-11-21 Thread GitBox
pzygielo commented on pull request #107: URL: https://github.com/apache/maven-scm/pull/107#issuecomment-974698770 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

[GitHub] [maven-scm] pzygielo commented on pull request #107: [SCM-946] Added @threadSafe for parallel execution

2021-11-20 Thread GitBox
pzygielo commented on pull request #107: URL: https://github.com/apache/maven-scm/pull/107#issuecomment-974704060 > Perhaps plugin-plugin could add this to the site, like is done for other [attributes] It does:

[GitHub] [maven-scm] pzygielo commented on pull request #107: [SCM-946] Added @threadSafe for parallel execution

2021-11-20 Thread GitBox
pzygielo commented on pull request #107: URL: https://github.com/apache/maven-scm/pull/107#issuecomment-974698770 > I think we should merge this and rename the ticket to make it explicitly clear that this plugin is _not_ threadsafe. Goals *are* already marked with threadSafe=false,

[GitHub] [maven-scm] pzygielo commented on pull request #107: [SCM-946] Added @threadSafe for parallel execution

2021-11-18 Thread GitBox
pzygielo commented on pull request #107: URL: https://github.com/apache/maven-scm/pull/107#issuecomment-972711956 I guess in runtime just `plugin.xml` is used, as

[GitHub] [maven-scm] pzygielo commented on pull request #107: [SCM-946] Added @threadSafe for parallel execution

2021-11-18 Thread GitBox
pzygielo commented on pull request #107: URL: https://github.com/apache/maven-scm/pull/107#issuecomment-972653112 @Lonzak - what is your maven version? I still can't see it being fixed by [current changes](85b640113bf9e943534c0f63850bf0a5bd6942fb). ```shell $ git tag -d v34 ;

[GitHub] [maven-scm] pzygielo commented on pull request #107: [SCM-946] Added @threadSafe for parallel execution

2021-11-17 Thread GitBox
pzygielo commented on pull request #107: URL: https://github.com/apache/maven-scm/pull/107#issuecomment-971879439 I don't understand how this shall help, as `plugin.xml` _already_ has following: ```xml tag ... false ... ``` As there is no