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

Inconsistent versions of slf4j-api #22569

Closed
gunnarmorling opened this issue Jan 1, 2022 · 2 comments · Fixed by #22570 or #22571
Closed

Inconsistent versions of slf4j-api #22569

gunnarmorling opened this issue Jan 1, 2022 · 2 comments · Fixed by #22570 or #22571
Labels
kind/bug Something isn't working
Milestone

Comments

@gunnarmorling
Copy link
Contributor

gunnarmorling commented Jan 1, 2022

Describe the bug

Getting this enforcer failure when pulling in 2.6.1.Final:

[INFO] --- maven-enforcer-plugin:3.0.0-M3:enforce (default) @ jfrunit-core ---
[WARN] Rule 4: org.apache.maven.plugins.enforcer.RequireUpperBoundDeps failed with message:
Failed while enforcing RequireUpperBoundDeps. The error(s) are [
Require upper bound dependencies error for org.slf4j:slf4j-api:1.7.30 paths to dependency are:
+-org.moditect.jfrunit:jfrunit-core:1.0.0-SNAPSHOT
  +-io.quarkus:quarkus-junit5:2.6.1.Final
    +-io.quarkus:quarkus-core:2.6.1.Final
      +-org.slf4j:slf4j-api:1.7.30
and
+-org.moditect.jfrunit:jfrunit-core:1.0.0-SNAPSHOT
  +-io.quarkus:quarkus-junit5:2.6.1.Final
    +-io.quarkus:quarkus-bootstrap-core:2.6.1.Final
      +-io.quarkus:quarkus-bootstrap-maven-resolver:2.6.1.Final
        +-org.slf4j:slf4j-api:1.7.30 (managed) <-- org.slf4j:slf4j-api:1.7.32

Expected behavior

One version of the slf4j-api artifact should be used consistently.

Actual behavior

No response

How to Reproduce?

quarkus$ ag -Q "1.7.32"
independent-projects/bootstrap/pom.xml
53:        <slf4j-api.version>1.7.32</slf4j-api.version>
quarkus$ ag -Q "1.7.30"
bom/application/pom.xml
105:        <slf4j.version>1.7.30</slf4j.version>

Output of uname -a or ver

No response

Output of java -version

No response

GraalVM version (if different from Java)

No response

Quarkus version or git rev

No response

Build tool (ie. output of mvnw --version or gradlew --version)

No response

Additional information

No response

@gunnarmorling
Copy link
Contributor Author

PR at #22570

@famod
Copy link
Member

famod commented Jan 1, 2022

I think updating to 1.7.32 makes sense. Apart form this specific update, there is a fundamental decision yet to be made w.r.t RequireUpperBoundDeps, see #21247.

@quarkus-bot quarkus-bot bot added this to the 2.7 - main milestone Jan 1, 2022
gsmet pushed a commit that referenced this issue Jan 2, 2022
@gsmet gsmet modified the milestones: 2.7 - main, 2.6.2.Final Jan 7, 2022
gsmet pushed a commit to gsmet/quarkus that referenced this issue Jan 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working
Projects
None yet
3 participants