Re: Access or generate a classpath for dependency not in any scope but attached to plugin.
Thanks, Herve. I'd share this with the Checkstyle user group on Github. I don't have the bandwidth for this right now and I'm not being literal! Regards, Linus. On Fri, 26 Jun 2020, 12:00 Hervé BOUTEMY, wrote: > the Maven Checkstyle Plugin source code is here: > https://github.com/apache/maven-checkstyle-plugin/ > with links to CI and issue tracking > > IMHO, updating its code should be easier than trying to tweak scripting > with > Ant run. > And with the integration tests (in src/its, launched with "mvn -Prun-its > verify"), you could easily check that your new feature has expected results > > then provide us a Pull Requests, to get the new feature integrated into > the > official plugin, and become a new Maven contributor! > > If you're interested, we're having an online "Hack Commit Push" event > tomorrow, where I will be there the whole day (Paris timezone) to help > people > wanting to contribute: > see https://paris2020.hack-commit-pu.sh/ > > Regards, > > Hervé > > Le vendredi 26 juin 2020, 03:21:33 CEST LINUS FERNANDES a écrit : > > While the exec maven plugin will satisfy the reduced classpath > > requirements, I'm constricted by the requirement of the Checkstyle CLI > that > > requires a list of source files as a parameter to be passed; it doesn't > > accept a directory and maven apparently works best with directories. > > There appears to be no maven plugin that lists source files and while I > can > > construct the list of source files via Ant Run plugin, I cannot propagate > > the dynamically added property to the exec maven plugin. > > Any suggestions? > > > > Or should I revert to the original all Ant version? > > > > > > > > On Thu, 25 Jun 2020, 19:51 LINUS FERNANDES, > > > > wrote: > > > I think exec maven plugin might need my requirements better of not > adding > > > Checkstyle jars to the project's dependencies. I'll have to test that > out, > > > though. > > > > > > On Thu, 25 Jun 2020, 14:33 LINUS FERNANDES, > > > > > > > wrote: > > >> I've raised a feature request at > > >> https://issues.apache.org/jira/browse/MCHECKSTYLE-396 > > >> > > >> Regards, > > >> Linus. > > >> > > >> > > >> > > >> > > >> On Thu, 25 Jun 2020, 14:19 LINUS FERNANDES, < > linus.fernan...@gmail.com> > > >> > > >> wrote: > > >>> Thanks, that resolves the issue with running the goal as > > >>> antrun:run@checkstyleg. > > >>> > > >>> Regards, > > >>> Linus. > > >>> > > >>> On Thu, 25 Jun 2020, 13:39 Benjamin Marwell, > wrote: > > Feature requests can be created here: > > https://issues.apache.org/jira/projects/MCHECKSTYLE > > > > You are welcome to create a pull request. > > > > Looking at the ant configuration, you could just use the plugin > > classpath in your java task: > > > > > https://maven.apache.org/plugins/maven-antrun-plugin/examples/classpath > > s.html > > > > > > Am Do., 25. Juni 2020 um 09:16 Uhr schrieb LINUS FERNANDES > > > > : > > > I would like to create a feature request---evidently. > > > > > > This is something that impacts the ant and Gradle tasks as well. > > > > > > I'm not sure where to make this request. > > > > > > Should it be on the Checkstyle Github repo or will I have to make > > > > three > > > > > separate requests, one for each of the above? > > > > > > The feature is available on the CLI using the -g option. > > > > > > I intend to use this during the Maven build, not from my code. > > > > > > Since I already had Ant configured to do exactly that, I > preferred to > > > > use > > > > > the Ant Run plugin to reproduce the functionality within Maven. > > > > > > The pom file is located at: > > > > > > https://github.com/Fernal73/DSAlgos/blob/master/pom.xml > > > > > > I have no interest in creating my own plugin given the attendant > > > > issues of > > > > > maintaining it and for just this feature. > > > > > > This should really be part of the Maven, Ant and Gradle > > > > configurations. > > > > > Perhaps, it's because the feature is still considered experimental > > > and > > > there's no pressure from users to add the ability to generate > Xpath > > > suppressions in the other ways to use Checkstyle. > > > > > > I find it very useful, though although it's complicated to set it > up > > > > in its > > > > > current Avatar. > > > > > > Regards, > > > Linus. > > > > > > > > > > > > > > > On Thu, 25 Jun 2020, 11:56 Benjamin Marwell, > > > > wrote: > > > > Well, you could write your own plugin which has checkstyle as a > > > > dependency. > > > > > > But it might be the easiest to just create a PR for the > checkstyle > > > > plugin. > > > > > > The checkstyle plugin is maintained
shade plugin modifying string constants
Hi I am trying to shade a package with following tags org.example.pkg org.example.shaded.pkg I also have a string in one of my class which matches the pattern gets modified during shading public static void main (String [] args) { System.out.println ("org.example.pkg."); //prints org.example.shaded.pkg. System.out.println ("org.example.PKG.".toLowerCase()); //prints org.example.pkg. } Is this expected behavior ? I was thinking about making use of this behavior. Will it be safe to do so ? Or its something undocumented so might change without prior notice ? thanks Anurag - To unsubscribe, e-mail: users-unsubscr...@maven.apache.org For additional commands, e-mail: users-h...@maven.apache.org
Re: Access or generate a classpath for dependency not in any scope but attached to plugin.
the Maven Checkstyle Plugin source code is here: https://github.com/apache/maven-checkstyle-plugin/ with links to CI and issue tracking IMHO, updating its code should be easier than trying to tweak scripting with Ant run. And with the integration tests (in src/its, launched with "mvn -Prun-its verify"), you could easily check that your new feature has expected results then provide us a Pull Requests, to get the new feature integrated into the official plugin, and become a new Maven contributor! If you're interested, we're having an online "Hack Commit Push" event tomorrow, where I will be there the whole day (Paris timezone) to help people wanting to contribute: see https://paris2020.hack-commit-pu.sh/ Regards, Hervé Le vendredi 26 juin 2020, 03:21:33 CEST LINUS FERNANDES a écrit : > While the exec maven plugin will satisfy the reduced classpath > requirements, I'm constricted by the requirement of the Checkstyle CLI that > requires a list of source files as a parameter to be passed; it doesn't > accept a directory and maven apparently works best with directories. > There appears to be no maven plugin that lists source files and while I can > construct the list of source files via Ant Run plugin, I cannot propagate > the dynamically added property to the exec maven plugin. > Any suggestions? > > Or should I revert to the original all Ant version? > > > > On Thu, 25 Jun 2020, 19:51 LINUS FERNANDES, > > wrote: > > I think exec maven plugin might need my requirements better of not adding > > Checkstyle jars to the project's dependencies. I'll have to test that out, > > though. > > > > On Thu, 25 Jun 2020, 14:33 LINUS FERNANDES, > > > > wrote: > >> I've raised a feature request at > >> https://issues.apache.org/jira/browse/MCHECKSTYLE-396 > >> > >> Regards, > >> Linus. > >> > >> > >> > >> > >> On Thu, 25 Jun 2020, 14:19 LINUS FERNANDES, > >> > >> wrote: > >>> Thanks, that resolves the issue with running the goal as > >>> antrun:run@checkstyleg. > >>> > >>> Regards, > >>> Linus. > >>> > >>> On Thu, 25 Jun 2020, 13:39 Benjamin Marwell, wrote: > Feature requests can be created here: > https://issues.apache.org/jira/projects/MCHECKSTYLE > > You are welcome to create a pull request. > > Looking at the ant configuration, you could just use the plugin > classpath in your java task: > > https://maven.apache.org/plugins/maven-antrun-plugin/examples/classpath > s.html > > > Am Do., 25. Juni 2020 um 09:16 Uhr schrieb LINUS FERNANDES > > : > > I would like to create a feature request---evidently. > > > > This is something that impacts the ant and Gradle tasks as well. > > > > I'm not sure where to make this request. > > > > Should it be on the Checkstyle Github repo or will I have to make > > three > > > separate requests, one for each of the above? > > > > The feature is available on the CLI using the -g option. > > > > I intend to use this during the Maven build, not from my code. > > > > Since I already had Ant configured to do exactly that, I preferred to > > use > > > the Ant Run plugin to reproduce the functionality within Maven. > > > > The pom file is located at: > > > > https://github.com/Fernal73/DSAlgos/blob/master/pom.xml > > > > I have no interest in creating my own plugin given the attendant > > issues of > > > maintaining it and for just this feature. > > > > This should really be part of the Maven, Ant and Gradle > > configurations. > > > Perhaps, it's because the feature is still considered experimental > > and > > there's no pressure from users to add the ability to generate Xpath > > suppressions in the other ways to use Checkstyle. > > > > I find it very useful, though although it's complicated to set it up > > in its > > > current Avatar. > > > > Regards, > > Linus. > > > > > > > > > > On Thu, 25 Jun 2020, 11:56 Benjamin Marwell, > > wrote: > > > Well, you could write your own plugin which has checkstyle as a > > dependency. > > > > But it might be the easiest to just create a PR for the checkstyle > > plugin. > > > > The checkstyle plugin is maintained by maven. So why not just > > create a > > > > feature request? > > > > > > Also, where do you want to execute the "new goal"? In the maven > > build > > > > or inside your java code? This will also make a difference. > > > If it is the latter, you can just pull in the plugin as a regular > > > dependency. > > > You can make it optional or provided (or both) if you do not need > > it at > > > >