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

Dependency on tools.logging is incompatible with leiningen pedantic feature #89

Closed
technomancy opened this issue Nov 30, 2018 · 3 comments
Labels

Comments

@technomancy
Copy link
Collaborator

technomancy commented Nov 30, 2018

When using the latest version of Leiningen from git, a dependency on nrepl is injected into the list of dependencies whenever the :base profile is used (which is most of the time).

When this happens, org.clojure/tools.logging is pulled in as an indirect dependency. Any medium-to-large sized Clojure project already pulls in a different version of that library. Such projects which have the :pedantic? feature enabled will abort due to the dependency conflict, with no clear way to resolve the conflict.

We can't have any dependencies in the :base profile of Leiningen which themselves have dependencies.

Expected behavior

You can use org.clojure/tools.logging in a project with :pedantic? enabled.

Actual behavior

Aborting due to :pedantic? :abort

Steps to reproduce the problem

Run lein test on any project which has a dependency on org.clojure/tools.logging other than 0.4.1 when :pedantic? is turned on.

Environment & Version information

Leiningen running from git master.

@bbatsov
Copy link
Contributor

bbatsov commented Nov 30, 2018 via email

@technomancy
Copy link
Collaborator Author

Thanks. Besides this we've got only one more issue and then should be good to release; I'm hoping within a week.

@bbatsov
Copy link
Contributor

bbatsov commented Nov 30, 2018

Done.

technomancy added a commit to technomancy/leiningen that referenced this issue Dec 1, 2018
The previous version had a dependency on tools.logging which would
cause build failures in most nontrivial projects which had pedantic
mode enabled.

See nrepl/nrepl#89
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants