bazel-deps icon indicating copy to clipboard operation
bazel-deps copied to clipboard

Use resolved parent for resolving children in normalizer

Open kmate-ct opened this issue 1 year ago • 5 comments

A potential solution to #379 by not allowing an evicted version to have effect on selecting a version of its children.

kmate-ct avatar Feb 06 '24 11:02 kmate-ct

If we think this is a good addition, I'm willing to add some unit tests!

kmate-ct avatar Feb 06 '24 11:02 kmate-ct

Update: we consider to change to maven_install from rules_jvm_external for multiple reasons later. I'd need to put this work on hold until we figure that out, as my resources are currently very limited towards this project. Still the patch should work and give the same resolution as using maven_install with pinned version conflict resolution, so whoever needs it could use this version. If I can get more resources to it, I'll return to it, but I cannot guarantee that unfortunately at this time.

kmate-ct avatar Feb 08 '24 08:02 kmate-ct

Thanks for the update.

BTW: maven_install seems like a good choice. I think the main benefit of Bazel-deps is that it has better scala support. For scala you don't want to run ijar in libraries that have macros or they will fail.

If you don't use scala or leverage any of the customization that Bazel-deps offers I think maven_install could be a fine choice.

johnynek avatar Feb 08 '24 16:02 johnynek

As far as I know maven_install doesn't use ijar anymore but their own simplified import rule, exactly because Scala macros and some issues with Kotlin as well. But yeah, it'd be great to make maven_install use scala_import and version the artifacts automatically, but based on the conversations I've seen I don't see much willingness to go to that direction.

kmate-ct avatar Feb 08 '24 16:02 kmate-ct

Update: for now we went with maven_install, which means that I won't be able to do any more progress on this PR in the foreseeable future. (We missed some of the Scala specifics that bazel-deps has but managed to work around those for ourselves for now.)

kmate-ct avatar Mar 25 '24 11:03 kmate-ct