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.

Sign up for GitHub

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

Jump to bottom

[Bug] tar breaks on Bazel 6 #975

Closed
jmthvt opened this issue Oct 28, 2024 · 0 comments · Fixed by #976
Closed

[Bug] tar breaks on Bazel 6 #975

jmthvt opened this issue Oct 28, 2024 · 0 comments · Fixed by #976

Comments

@jmthvt
Copy link

jmthvt commented Oct 28, 2024

It looks like #953 introduced a regression on Bazel 6.x

Traceback (most recent call last):
	File "/home/jmthvt/.cache/bazel/_bazel_jmthvt/2299fc45cf64d298be53c0b783603165/external/aspect_bazel_lib/lib/private/tar.bzl", line 304, column 38, in _tar_impl
		src[DefaultInfo].files_to_run.repo_mapping_manifest
Error: 'FilesToRunProvider' value has no field or method 'repo_mapping_manifest'

Which is indeed not present in 6.5.0: https://bazel.build/versions/6.5.0/rules/lib/FilesToRunProvider
plobsing added a commit to plobsing/bazel-lib that referenced this issue Oct 28, 2024
@plobsing
fix: access FilesToRunProvider.repo_mapping_manifest indirectly
5e7297d 
This field is only present in Bazel versions v7.0 or later. An
accodation for its absence is required while v6.x versions are still
supported by this library.

Fixes bazel-contrib#975
@plobsing plobsing mentioned this issue Oct 28, 2024
Merged
@jjmaestro jjmaestro mentioned this issue Nov 11, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: access FilesToRunProvider.repo_mapping_manifest indirectly plobsing/bazel-lib
1 participant
@jmthvt