bazelbuild / bazelisk

A user-friendly launcher for Bazel.

Apache License 2.0

1.95k stars 300 forks source link

Support `Tools/` in case-sensitive file systems #577

Closed luispadron closed 1 month ago

luispadron commented 1 month ago

We've been keeping our wrapper in a uppercased Tools/ directory at Tools/bazel. This is because we have existing stuff in Tools/. This works fine for case-insensitive file systems but fails silently on a case-sensitive file system as bazelisk only checks for tools/bazel.

Could we support both/or any casing here?

keith commented 1 month ago

maybe best solved by https://github.com/bazelbuild/bazelisk/pull/567

luispadron commented 1 month ago

yes that would be perfect thank you @keith. Any idea if theres a desire to merge that from the maintainers? We can keep this open and close it if that merges

keith commented 1 month ago

haven't heard anything

luispadron commented 1 month ago

567 solves this for us thanks Keith!