EngFlow / bazel_invocation_analyzer

A tool that automatically analyzes a Bazel build's profile and offers suggestions on how to make it faster.
https://analyzer.engflow.com
Apache License 2.0
90 stars 10 forks source link

Update to rules_jvm_external 6.2, repin maven deps #183

Closed mbland closed 2 months ago

mbland commented 2 months ago

Details:

This new version resolves the issue mentioned in #182:

Fixed by:

Per my comment on that issue, source JARs are no longer fetched without explicitly setting fetch_sources = True. This is why they no longer appear in maven_install.json.

Finally, @shs96c noted to me in private that:

...with recent rules_jvm_external releases, all you need to update is bazel run @maven//:pin. There’s no need for the unpinned_maven repo any more.

I removed the unpinned_maven repo and ran REPIN=1 bazel run @maven//:pin to regenerate maven_install.json. This also removed the unpinned_maven entries from MODULE.bazel.lock.

I'll update this section of my Bzlmod migration blog post after merging this change: