-
Notifications
You must be signed in to change notification settings - Fork 19
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
Add support for BOM dependencies (Depends on TDEPS-202) #77
Comments
Interesting, I added that check because I assumed only git dependencies could have multiple paths. If you already have an open source project using |
Thanks for taking a look. I have a project that currently has some native and Python deps which I am in the process of cleaning up, but I found a relatively lightweight repo here: https://github.com/Swirrl/rdf-validator Locking with clj-nix fails as in my report. Running the testsuite with the Clojure CLI works fine (:test-runner is my cognitect-labs/test-runner alias): clj -A:dev:test-runner I believe a workaround would be to reference the specific Jena deps needed per project. Unfortunately, there is another level of similar nested dependencies, which makes this quite arduous: org.apache.jena/apache-jena-libs dependency tree
|
@borh Thanks for the example, I can reproduce the issue, and I'll take a look to it |
@borh Is
I make the same assumption in Did you try to replace org.apache.jena/jena-rdfpatch {:mvn/version "4.7.0"}
org.apache.jena/jena-shacl {:mvn/version "4.7.0"}
org.apache.jena/jena-shex {:mvn/version "4.7.0"}
org.apache.jena/jena-tdb {:mvn/version "4.7.0"}
org.apache.jena/jena-tdb2 {:mvn/version "4.7.0"}
org.apache.jena/jena-rdfconnection {:mvn/version "4.7.0"} |
I found some more info:
If TDEPS-202 is implemented, I'll add support for it on |
Trying to lock the
org.apache.jena/apache-jena-libs
dep given the following deps.edn fails with a "Maven deps can have only 1 path":I.e.: nix run github:jlesquembre/clj-nix#deps-lock -- --deps-include deps.edn
The dep in question is a POM artifact that just points to other deps (recursively). I haven't tried yet, but couldn't the test for paths be relaxed to handle this condition?
The text was updated successfully, but these errors were encountered: