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.

Sign up for GitHub

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

Jump to bottom

[ZOOKEEPER-4820] Fix propagation of Logback dependencies #2155

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

[ZOOKEEPER-4820] Fix propagation of Logback dependencies #2155

wants to merge 3 commits into from
+10 −12

Commits on Apr 9, 2024

  1. [ZOOKEEPER-4820] Fix propagation of Logback dependencies 

    Java libraries should not have `runtime` dependencies on logging
backends, since they are often included in third-party applications that
might make a different choice of backend.

This PR:

* removes the Logback dependency from the runtime scope of the
  `zookeeper` artifact and moves it into the `test` Maven scope,
* adds Logback to `zookeeper-assembly`, which is used to produce the
  standalone [binary
  distribution](https://zookeeper.apache.org/releases) available on
  Zookeeper's site.
    @ppkarwasz
    ppkarwasz committed Apr 9, 2024
    Configuration menu
    Copy the full SHA
    41a0571 View commit details
    Browse the repository at this point in the history

Commits on Apr 10, 2024

  1. Switch Logback to optional runtime scope

    @ppkarwasz
    ppkarwasz committed Apr 10, 2024
    Configuration menu
    Copy the full SHA
    4d908e2 View commit details
    Browse the repository at this point in the history

Commits on Apr 19, 2024

  1. Fix zookeeper-contrib-zooinspector tests

    @ppkarwasz
    ppkarwasz committed Apr 19, 2024
    Configuration menu
    Copy the full SHA
    9b53573 View commit details
    Browse the repository at this point in the history