Regression: bazel 3.2.0 + glob allow_empty=false + android ndk fails

[iphydf]

Description of the problem:

As of bazel 3.2.0, passing --incompatible_disallow_empty_glob fails for android builds using NDK r16b:

glob pattern 'ndk/sources/third_party/vulkan/src/build-android/jniLibs/armeabi/libVkLayer_*.so' didn't match anything, but allow_empty is set to False.

There is armeabi-v7a, but no armeabi. This works fine with bazel 3.1.0.

Bugs: what's the simplest, easiest way to reproduce this bug? Please provide a minimal example if possible.

I don't have a minimal repro, but essentially: pass --incompatible_disallow_empty_glob and have an android_ndk_repository.

What operating system are you running Bazel on?

Debian GNU/Linux.

What's the output of bazel info release?

release 3.2.0

Have you found anything relevant by searching the web?

No (I did look, but nothing so far).

Any other information, logs, or outputs that you want to share?

I have not found a workaround for this. I tried --fat_apk_cpu=armeabi-v7a, but that doesn't fix it.

[jin]

While this is something that should be fixed before enabling the breaking change in #8195, I don't think it's recommended to use an un-flipped --incompatible flag for day to day use. Usually, it's because we haven't provided migration tooling or fixed breaking changes with Bazel's own code (like this issue).

https://docs.bazel.build/versions/master/backward-compatibility.html#at-a-glance

Users should never run their production builds with --experimental_* or --incompatible_* flags.

[laurentlb]

Who should look at this?
This issue will prevent the flag flip planed for Bazel 4.0.

[laurentlb]

@ahumesky

This breaks the Examples repository (see https://buildkite.com/bazel/bazelisk-plus-incompatible-flags/builds/571#58b644b5-222b-4768-93d3-f23f9397d6ca).

[laurentlb]

ping?

[laurentlb]

@jin @ahumesky @timpeut
Do you know anyone who could look at this?

[ahumesky]

#8195 has been moved to 5.0, so we can look at this for next quarter

[limdor]

The problem is still present and it is in this line https://github.com/bazelbuild/bazel/pull/15330/files#diff-ac45a531edff6b8e6694f669184fc3625c869c84710142175e2caae08e159a9dL197

[ahumesky]

--incompatible_disallow_empty_glob has been rolled out