-
Notifications
You must be signed in to change notification settings - Fork 3
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
Error log about log4j2 when building native or runnin using quarkus:dev #91
Comments
@gastaldi thoughts? |
I think it's just a matter of adding this dependency to quarkus-poi runtime's pom.xml? |
i think so too |
Sounds good to me. Want to provide a PR? |
On it |
BTW can you confirm log4j2 is required by poi? |
|
Thank you |
PR Submitted |
Wow, that was fast. Thank you guys! |
Just as Quarkus meant to be ;) |
@manofthepeace try 2.0.5 when you get a chance. |
@melloware unfortunately something is off with the new version, When I compile native I get this, which fails the compilation;
When running the app via quarkus:dev I get this;
|
That's because you need commons-compress 1.25.0. See quarkusio/quarkus#37620 on how to workaround until the next Quarkus release |
When I do a dependencytree I see compress 1.24 used instead of 1.25 as defined in the quarkus-poi pom
If I force the following in my pom <dependency>
<groupId>org.apache.commons</groupId>
<artifactId>commons-compress</artifactId>
<version>1.25.0</version>
</dependency> it does work as expected |
oh well looks like you where faster than me. but the solution above works as well |
I think we can bypass that by adding commons-compress 1.25.0 in the same pom.xml runtime, WDYT @melloware ? |
Ah but then the Quarkus BOM may still override it, nvm, your workaround is okay |
if this quarkusio/quarkus#37187 gets backported to 3.6.2 next week then all will be good, so I think its all ok and no workarounds will be needed. |
I am developing an app that uses quarkus-poi, when I run (or perform a native build) I get the following
ERROR StatusLogger Log4j2 could not find a logging implementation. Please add log4j-core to the classpath. Using SimpleLogger to log to the console...
Adding the following dependency to my pom fixes the issue.
Is that something that could be done within that project as written here; https://quarkus.io/guides/logging#add-a-logging-adapter-to-your-application
This step is unnecessary for libraries that are dependencies of a Quarkus extension where the extension handles it automatically.
The text was updated successfully, but these errors were encountered: