-
Notifications
You must be signed in to change notification settings - Fork 54
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
fix outputDirectory not work at running spotbugs:spotbugs goal #807
fix outputDirectory not work at running spotbugs:spotbugs goal #807
Conversation
Thanks. Great catch. Will get merged and a release this weekend
Sent from my Verizon, Samsung Galaxy smartphone
Get Outlook for Android<https://aka.ms/AAb9ysg>
…________________________________
From: Exia.huangxy ***@***.***>
Sent: Wednesday, May 8, 2024 10:01:59 AM
To: spotbugs/spotbugs-maven-plugin ***@***.***>
Cc: Subscribed ***@***.***>
Subject: [spotbugs/spotbugs-maven-plugin] fix outputDirectory not work at running spotbugs:spotbugs goal (PR #807)
outputDirectory<https://spotbugs.github.io/spotbugs-maven-plugin/spotbugs-mojo.html#outputDirectory> is not working pretty well when I try to run mvn spotbugs:spotbugs command.
I think maybe the htmlTempFile should be use outputDirectory.
Can you take a look of my PR, Thank you.
________________________________
You can view, comment on, or merge this pull request online at:
#807
Commit Summary
* 5591f3c<5591f3c> fix outputDirectory not work at running spotbugs:spotbugs goal
File Changes
(1 file<https://github.com/spotbugs/spotbugs-maven-plugin/pull/807/files>)
* M src/main/groovy/org/codehaus/mojo/spotbugs/SpotBugsMojo.groovy<https://github.com/spotbugs/spotbugs-maven-plugin/pull/807/files#diff-a8270ec7df0fe9dcf1a1e8120b333c540187439faf5963d157f3a3e846a851e6> (2)
Patch Links:
* https://github.com/spotbugs/spotbugs-maven-plugin/pull/807.patch
* https://github.com/spotbugs/spotbugs-maven-plugin/pull/807.diff
—
Reply to this email directly, view it on GitHub<#807>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AAHODI7SWMNJOCKSZ6AX53TZBIV5PAVCNFSM6AAAAABHNBGATOVHI2DSMVQWIX3LMV43ASLTON2WKOZSGI4DKNRYGMZDCMY>.
You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>
|
@exiahuang Can you show what isn't working for you here? That location is a temp file and there are others in that same spot. project.build.directory is used in a lot of places. |
If I set outputDirectory at build tag, run <build>
<plugins>
<plugin>
<groupId>com.github.spotbugs</groupId>
<artifactId>spotbugs-maven-plugin</artifactId>
<!-- <version>4.8.5.1-SNAPSHOT</version> -->
<version>4.8.5.0</version>
<dependencies>
<dependency>
<groupId>com.github.spotbugs</groupId>
<artifactId>spotbugs</artifactId>
<version>4.8.5</version>
</dependency>
</dependencies>
<configuration>
<htmlOutput>true</htmlOutput>
<outputDirectory>target/test-output-directory</outputDirectory>
</configuration>
</plugin>
</plugins>
</build> I Think htmlTempFile is not just a temp file, |
ok I'll need to look more at it. Currently this PR breaks the integration tests.... |
5591f3c
to
f22809b
Compare
7790b6b
to
f449bf1
Compare
@hazendaz |
@hazendaz |
Jdk 23 is a known and reported issue to error prone. It only started appearing couple days ago when github updated to latest build.
Sent from my Verizon, Samsung Galaxy smartphone
Get Outlook for Android<https://aka.ms/AAb9ysg>
…________________________________
From: Exia.huangxy ***@***.***>
Sent: Tuesday, May 28, 2024 9:37:44 AM
To: spotbugs/spotbugs-maven-plugin ***@***.***>
Cc: Jeremy Landis ***@***.***>; Mention ***@***.***>
Subject: Re: [spotbugs/spotbugs-maven-plugin] fix outputDirectory not work at running spotbugs:spotbugs goal (PR #807)
@hazendaz<https://github.com/hazendaz>
There are some thing wrong with JDK23 test. I don't think it's my fault, but I don't know how to fix it.
—
Reply to this email directly, view it on GitHub<#807 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AAHODI2A5U6P33LJB2GK2TDZESCCRAVCNFSM6AAAAABHNBGATOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZVGI2DCMJSHA>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
outputDirectory is not working pretty well when I try to run
mvn spotbugs:spotbugs
command.I think maybe the htmlTempFile should be use
outputDirectory
.Can you take a look of my PR, Thank you.