Skip to content

Broken Bazel build with --incompatible_disallow_empty_glob #2432

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

Open
lalten opened this issue Mar 14, 2025 · 1 comment · May be fixed by #2433
Open

Broken Bazel build with --incompatible_disallow_empty_glob #2432

lalten opened this issue Mar 14, 2025 · 1 comment · May be fixed by #2433

Comments

@lalten
Copy link
Contributor

lalten commented Mar 14, 2025

Eclipse iceoryx version:
main branch

Observed result or behaviour:
Error in glob: glob pattern 'source/error_handling/**' didn't match anything, but allow_empty is set to False (the default value of allow_empty can be set with --incompatible_disallow_empty_glob). (and many more)

Expected result or behaviour:
Working Bazel 8 build and working Bazel 7 build with --incompatible_disallow_empty_glob

Conditions where it occurred / Performed steps:
USE_BAZEL_VERSION=8.x bazel build //... --enable_bzlmod

@lalten lalten linked a pull request Mar 14, 2025 that will close this issue
21 tasks
lalten added a commit to lalten/iceoryx that referenced this issue Mar 14, 2025
lalten added a commit to lalten/iceoryx that referenced this issue Mar 14, 2025
lalten added a commit to lalten/iceoryx that referenced this issue Mar 14, 2025
lalten added a commit to lalten/iceoryx that referenced this issue Mar 14, 2025
lalten added a commit to lalten/iceoryx that referenced this issue Mar 14, 2025
lalten added a commit to lalten/iceoryx that referenced this issue Mar 14, 2025
@gpalmer-latai
Copy link

We noticed this as well. Someone @lalten and I both know had to apply a patch for it.

If I or a colleague find time we may attempt to incorporate that patch upstream.

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 a pull request may close this issue.

2 participants