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

[MNG-8386] Pull out executor #1932

Merged
merged 13 commits into from
Nov 22, 2024
Merged

[MNG-8386] Pull out executor #1932

merged 13 commits into from
Nov 22, 2024

Conversation

cstamas
Copy link
Member

@cstamas cstamas commented Nov 21, 2024

Yet another CLIng cleanup.

Changes:

  • pull out Executor, it does not belong to CLIng (new maven-executor module created with no deps)
  • resident and maven invoker fixes (proper handling of resources now), no more SO/OOMs
  • enabled UTs in maven-cli (that revealed the issues)
  • small bug fixes discovered in cli, improved executor to reveal maven version

https://issues.apache.org/jira/browse/MNG-8386

There was a mixup, as "executor" unlike "invoker" does
not expect all (in fact it expects nothing) on classpath.

So introduce new dependency less (minimal ones) maven-executor
that implements two simple executors (forked and embedded)
and leave maven-cli for invoker implementations.
@cstamas cstamas self-assigned this Nov 21, 2024
@cstamas cstamas requested a review from gnodet November 21, 2024 12:25
@cstamas cstamas added this to the 4.0.0-rc-1 milestone Nov 21, 2024
Now that the issue is figured out...
I was sitting on a horse in opposite direction.

Context close needs to happen with all registered stuff,
ONLY the container (and looked up stuff) needs to be "carried
over".
it makes AnsiTerminal install counter offset -1
As they fail on Windows due junit being unable to delete
the log file mvn3 produces.
@cstamas cstamas marked this pull request as ready for review November 22, 2024 13:19
@cstamas cstamas changed the title Pull out executor [MNG-8386] Pull out executor Nov 22, 2024
@cstamas cstamas merged commit 7ad2578 into apache:master Nov 22, 2024
13 checks passed
@cstamas cstamas deleted the pull-out-executor branch November 22, 2024 14:09
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

Successfully merging this pull request may close these issues.

1 participant