Skip to content

Conversation

cstamas
Copy link
Member

@cstamas cstamas commented Feb 5, 2025

This change aligns now what we really want:

  • maven-core provides Plexus Interactivity API (package export was already present, but NOT artifactExport)
  • maven-jline provides Plexus Interactivity API implementation that delegates to new M4 API implementation
  • new Maven4 API Prompter implementation provided
  • exploded the super overloaded MessageFactory implementation

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

This change aligns now what we really want:
* maven-core provides Plexus Interactivity API
* maven-jline provides Plexus Interactivity API implementation
* new Maven4 API Prompter simply delegates to maven-jline one

---

https://issues.apache.org/jira/browse/MNG-8530
@cstamas cstamas added this to the 4.0.0-rc-3 milestone Feb 5, 2025
@cstamas cstamas requested a review from gnodet February 5, 2025 12:04
@cstamas cstamas self-assigned this Feb 5, 2025
And now legacy goes for current, and not other way
around.
* make sure out prompter override legacy
* honor proper exceptions
@cstamas cstamas marked this pull request as ready for review February 5, 2025 14:33
Copy link
Contributor

@gnodet gnodet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, nice work !

@cstamas cstamas merged commit 4fc8a2c into apache:master Feb 5, 2025
13 checks passed
@cstamas cstamas deleted the MNG-8530 branch February 5, 2025 15:12
@Experimental
@Named
@Singleton
@Priority(10)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cstamas Would it make sense to move that one to compat/maven-compat ? The idea would be to avoid introducing any new dependency on sisu in impl/* jars ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yup

@jira-importer
Copy link

Resolve #9373

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.

3 participants