-
Notifications
You must be signed in to change notification settings - Fork 647
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
Clean the build #2638
Comments
I stumbled upon the commit that you re-added Perhaps the option could go into a property which defaults to <!-- in main pom.xml -->
<properties>
<compiler.proc>-proc:none</compiler.proc>
</properties>
...
<compilerArgs>
<arg>${compiler.proc}</arg>
</compilerArgs> <!-- in benchmark pom.xml -->
<properties>
<compiler.proc></compiler.proc>
</properties> if an effectively empty |
It really would be helpful to have the build only output what is necessary because when new output occurs, it can get missed. The build output is currently about 6,500 lines. I'm not sure setting the property will work - when does the substitution happen? The compiler plugin can't be declared in Putting a configuration into the compiler plugin for What should the Other: |
I suppose
I think so too. |
https://inside.java/2024/06/18/quality-heads-up/ :
For OpenJDK, no legacy profile is needed, only an up-to-date compiler for Java17. We could set I tried the POM setup in https://www.baeldung.com/java-microbenchmark-harness#start (briefly) but I found that it still needed |
Change
As far as practical, address the warnings raised by the maven build including hidden warnings about deprecated features.
@SuppressWarnrings("all")
- it needs the specific warning category.@Deprecate
implementations of deprecated interfaces.See also #2533.
Are you interested in contributing a pull request for this task?
Yes
The text was updated successfully, but these errors were encountered: