Skip to content

Trim threadConfiguration to accept input surrounded with spaces#11851

Merged
slawekjaranowski merged 6 commits intoapache:maven-3.10.xfrom
pzygielo:threads-config
Mar 28, 2026
Merged

Trim threadConfiguration to accept input surrounded with spaces#11851
slawekjaranowski merged 6 commits intoapache:maven-3.10.xfrom
pzygielo:threads-config

Conversation

@pzygielo
Copy link
Copy Markdown
Contributor

  • space-prefixed core multiplier - is already supported - added support for space-suffix
  • extended that behaviour for direct number of threads argument
  • can be used in bizarre command line like mvn -T ' 1 ' or - why I'm here - with value indented in maven.config:
--threads
  1

+ Run all the tests with `mvn -Prun-its verify` to assure nothing else was accidentally broken.

[WARNING] The requested profile "run-its" could not be activated because it does not exist.


@slawekjaranowski slawekjaranowski linked an issue Mar 26, 2026 that may be closed by this pull request
@slawekjaranowski slawekjaranowski changed the base branch from maven-3.9.x to maven-3.10.x March 28, 2026 13:42
@slawekjaranowski
Copy link
Copy Markdown
Member

@pzygielo we are starting on next minor version 3.10.x so new PR should be target to this branch, I will try to change a base branch ...

@pzygielo
Copy link
Copy Markdown
Contributor Author

we are starting on next minor version 3.10.x so new PR should be target to this branch,

I missed that 3.10.x exists.

I will try to change a base branch ...

Thanks for checking!

@slawekjaranowski slawekjaranowski added the bug Something isn't working label Mar 28, 2026
@slawekjaranowski slawekjaranowski merged commit 6f646d7 into apache:maven-3.10.x Mar 28, 2026
18 checks passed
@github-actions github-actions bot added this to the 3.10.0 milestone Mar 28, 2026
@pzygielo pzygielo deleted the threads-config branch March 28, 2026 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow argument to --threads to be surrounded by spaces

2 participants