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

Replace the use of deprecated APIs within the plugin #239

Open
ascopes opened this issue Aug 16, 2024 · 3 comments
Open

Replace the use of deprecated APIs within the plugin #239

ascopes opened this issue Aug 16, 2024 · 3 comments

Comments

@ascopes
Copy link

ascopes commented Aug 16, 2024

Running the Maven plugin on v2.5.1 with -Dmaven.plugin.validation=verbose in the MAVEN_ARGS results in the following warning:

[WARNING]   Mojo EXTERNAL issue(s):
[WARNING]    * Mojo apigee-enterprise:configure (io.apigee.buildTools.enterprise4g.mavenplugin.ConfigureMojo)
[WARNING]      - Mojo implements `Contextualizable` interface from Plexus Container, which is EOL.
[WARNING] 
[WARNING] 
[WARNING] Fix reported issues by adjusting plugin configuration or by upgrading above listed plugins. If no upgrade available, please notify plugin maintainers about reported issues.
[WARNING] For more or less details, use 'maven.plugin.validation' property with one of the values (case insensitive): [NONE, INLINE, SUMMARY, BRIEF, VERBOSE]
[WARNING] 

The use of Plexus can be replaced with the use of Eclipse Sisu, which should also reduce some complexity as it implements the Java CDI spec, and is the recommended way to achieve CDI in Maven Plugins now.

The use of end-of-life features should probably be fixed since it would pose a blocker to Maven 4.0 migration once it becomes available.

@ssvaidyanathan
Copy link
Collaborator

@ascopes - thanks for sharing. Apologies for the delayed response

I am not seeing any docs around this. The GitHub repo is marked as publicly archived and the main repo links seems to be broken. Can you share any info on this so that I can take a look and see how much work is needed?

@ascopes
Copy link
Author

ascopes commented Sep 3, 2024

Eclipse Sisu, you mean?

I have a working set up for it in my Maven plugin I maintain as well if you need an example of how to configure it: https://github.com/ascopes/protobuf-maven-plugin/tree/main/protobuf-maven-plugin

Hope that helps. Thanks for the reply!

@ssvaidyanathan
Copy link
Collaborator

Thank you! I already saw the first two links and couldnt really find a lot of useful info to build plugins. The last one is useful. Thank you

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

2 participants