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.

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

Change databinding dependent lib artifact to a directory #228

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

arunkumar9t2
Copy link

@arunkumar9t2 arunkumar9t2 commented Apr 24, 2024

Dependent lib artifact is supposed to be a directory but was registered as a file. This is an error in Bazel 7 due to --incompatible_disallow_unsound_directory_outputs which will be enforced in Bazel 8.

Ref

Bazel needs to know whether an artifact is a file or a directory before its contents are produced, and coverage.dat is declared as a file. Before Bazel 7, producing a directory where a file is expected was tolerated, but broken in subtle ways. Bazel 7 forbids it by default (--incompatible_disallow_unsound_directory_outputs) and Bazel 8 will not allow it at all.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant