P. Oscar Boykin
P. Oscar Boykin
scala_import seems to be doing the right thing, btw. So, this is only for java libraries.
It generally means that there is a cycle somewhere: x export y, y exports z, z exports x. Something like that. The error message should be clearer, but I would...
btw: this may be a real circular dependency in the maven graph. We could probably support this by combining all those into a single target since bazel forbids loops.
A feature like that would be nice but it doesn't exist. We could merge a PR if you have time. I guess we would look for an API we can...
huh... I don't think we've considered this case. Due to the `_version` approach of scala jar publishing there could be two different jars which collide if you chop off the...
So, the key things would be: 1. add support for representing the data in the DepsModel file: https://github.com/johnynek/bazel-deps/blob/3bed1b71bf94a62b566d6d98f119f2708b28c367/src/scala/com/github/johnynek/bazel_deps/DepsModel.scala#L601 2. check for duplicates after language normalization I think in this method:...
BTW: you might be able to make two PRs: the second one: error when there is a duplicate with a nice error, might be the easier of the two and...
The exception seems to indicate it cannot make a file I guess Coursier uses for caching. I don’t know why the tests pass for me, I assume they do and...
I assume the sandboxing is making the tests fail since they may use some undeclared files. I'm not sure how that relates to a genrule in your repo though.
sounds like a good plan. I would like to move away from aether see #23. But I assume that wouldn't block this at all. Coursier is much faster (has parallel...