Apply expand_locations_and_make_variables to cache_entries

bazel-contrib / rules_foreign_cc

Build rules for interfacing with "foreign" (non-Bazel) build systems (CMake, configure-make, GNU Make, boost, ninja, Meson)

https://bazel-contrib.github.io/rules_foreign_cc

Apache License 2.0

679 stars 249 forks source link

Apply expand_locations_and_make_variables to cache_entries #1240

Closed froody closed 3 months ago

froody commented 4 months ago

When building proj I need to pass the sqlite3 binary via a cache entry, like {"EXE_SQLITE3": "$(execpath @sqlite3//:shell)"} but that requires that cache entries are also expanded.

jsharpe commented 3 months ago

Apologies for having to revert this @froody - I should have checked CI, but the already broken state meant that I missed the failures this introduced. Feel free to resubmit this as I should soon have CI back to a green state again.