Skip to content

Commit 662ee4e

Browse files
committed
iceberg-rest-catalog: fix genrule
local = True was forcing local, unsandboxed execution and making the build bypass the remote cache. This commit: - drops local = True so the action runs with a sandbox and can use remote cache, - sets an explicit GRADLE_USER_HOME, otherwise would fallback on $HOME/.gradle which is read-only from the sandbox, - drops the `chmod +x ./gradlew` which was unneeded, plus it breaks in the sandbox because it is read-only
1 parent 09a9edb commit 662ee4e

1 file changed

Lines changed: 1 addition & 2 deletions

File tree

  • tests/java/iceberg-rest-catalog

tests/java/iceberg-rest-catalog/BUILD

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,11 +14,10 @@ genrule(
1414
OUT_PATH=$$(realpath $(OUTS))
1515
SRC_DIR=$$(dirname $(location gradlew))
1616
cd $$SRC_DIR
17-
chmod +x ./gradlew
17+
export GRADLE_USER_HOME="$$(pwd)/.gradle-home"
1818
./gradlew --no-daemon shadowJar
1919
cp build/libs/iceberg-rest-catalog-all.jar $$OUT_PATH
2020
""",
21-
local = True,
2221
tags = ["requires-network"],
2322
visibility = ["//visibility:public"],
2423
)

0 commit comments

Comments
 (0)