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

Introduction of Commons CLI breaks CLI for invocation by maven org.codehaus.mojo:javancss-maven-plugin #5

Open
gtsh opened this issue Mar 27, 2022 · 0 comments

Comments

@gtsh
Copy link

gtsh commented Mar 27, 2022

The org.codehaus.mojo:javancss-maven-plugin:2.2-SNAPSHOT still uses Version 33.54 of javancss.

Commit 4a34312 "Use Commons CLI for the command line parsing" switched the parsing to Apache commons cli and is introduced with the not yet released version 34.55-SNAPSHOT.

But using this in the latest snapshot breaks the invocation.

The execution fails, missing a file "javancss-raw-report.xml". The Problem here is, that the maven plugin passes this as argument to the parameter out as an argument for the file write the result to, but javancss interprets this as another file to parse, and promptly misses it.

The PR attached will fix this.

gtsh added a commit to gtsh/javancss that referenced this issue Mar 27, 2022
... also replaces the deprecated cli utility class used for this.
Task JavaNCSS#5 - Introduction of Commons CLI breaks CLI for invocation by maven org.codehaus.mojo:javancss-maven-plugin
gtsh added a commit to gtsh/javancss that referenced this issue Mar 30, 2022
... also replaces the deprecated cli utility class used for this.
Task JavaNCSS#5 - Introduction of Commons CLI breaks CLI for invocation by maven org.codehaus.mojo:javancss-maven-plugin
gtsh added a commit to gtsh/javancss that referenced this issue Mar 30, 2022
... also replaces the deprecated cli utility class used for this.
Task JavaNCSS#5 - Introduction of Commons CLI breaks CLI for invocation by maven org.codehaus.mojo:javancss-maven-plugin
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

1 participant