diff options
author | Klaus Aehlig <aehlig@google.com> | 2016-09-20 14:45:06 +0000 |
---|---|---|
committer | Laszlo Csomor <laszlocsomor@google.com> | 2016-09-21 07:05:55 +0000 |
commit | 96d46280bc5a4803ba2242a4ad16939f85a3b212 (patch) | |
tree | ba4be058b0032956ef8d8e37d25fcd042297fb8b | |
parent | 6e2142af1c069a35a942b257e368ab05d04401f1 (diff) |
Reintroduce .bazelrc and set --action_env defaults
Shipping a .bazelrc was removed in 009b48bcfaf4ebb17f9ab03bea9036cfa3c12ff0.
However, as per our "Specifying environment variables for actions" design
document, we want to ship a global rc file specifying the common environment
variables to be inherited. So revert that change and add the desired entries
to the global bazelrc file we're shipping.
--
Change-Id: Ib763677a951e1264ae92954df9b831374594d8b7
Reviewed-on: https://bazel-review.googlesource.com/#/c/6113
MOS_MIGRATED_REVID=133704608
-rw-r--r-- | scripts/packages/BUILD | 9 | ||||
-rw-r--r-- | scripts/packages/debian/bazel.bazelrc | 3 | ||||
-rwxr-xr-x | scripts/packages/template_bin.sh | 22 |
3 files changed, 30 insertions, 4 deletions
diff --git a/scripts/packages/BUILD b/scripts/packages/BUILD index ef3fe22f90..5bce69d16f 100644 --- a/scripts/packages/BUILD +++ b/scripts/packages/BUILD @@ -92,6 +92,14 @@ pkg_tar( strip_prefix = ".", ) +pkg_tar( + name = "bazel-rc", + files = ["debian/bazel.bazelrc"], + mode = "0644", + package_dir = "/etc", + strip_prefix = "debian", +) + genrule( name = "rename-bazel-completion", srcs = ["//scripts:bash_completion"], @@ -112,6 +120,7 @@ pkg_tar( deps = [ ":bazel-bin", ":bazel-completion", + ":bazel-rc", ], ) diff --git a/scripts/packages/debian/bazel.bazelrc b/scripts/packages/debian/bazel.bazelrc index e69de29bb2..36742468a2 100644 --- a/scripts/packages/debian/bazel.bazelrc +++ b/scripts/packages/debian/bazel.bazelrc @@ -0,0 +1,3 @@ +common --action_env=PATH +common --action_env=LD_LIBRARY_PATH +common --action_env=TMPDIR diff --git a/scripts/packages/template_bin.sh b/scripts/packages/template_bin.sh index ceb58c83de..69837f4ad7 100755 --- a/scripts/packages/template_bin.sh +++ b/scripts/packages/template_bin.sh @@ -41,14 +41,16 @@ usage() { echo " --prefix=/some/path set the prefix path (default=/usr/local)." >&2 echo " --bin= set the binary folder path (default=%prefix%/bin)." >&2 echo " --base= set the base install path (default=%prefix%/lib/bazel)." >&2 + echo " --bazelrc= set the path to bazelrc (default=/etc/bazel.bazelrc)." >&2 echo " --user configure for user install, expands to:" >&2 - echo ' --bin=$HOME/bin --base=$HOME/.bazel' >&2 + echo ' --bin=$HOME/bin --base=$HOME/.bazel --bazelrc=$HOME/.bazelrc' >&2 exit 1 } prefix="/usr/local" bin="%prefix%/bin" base="%prefix%/lib/bazel" +bazelrc="/etc/bazel.bazelrc" for opt in "${@}"; do case $opt in @@ -56,8 +58,7 @@ for opt in "${@}"; do prefix="$(echo "$opt" | cut -d '=' -f 2-)" ;; --bazelrc=*) - # TODO(kchodorow): remove once everything we're testing on CI contains - # this comment. + bazelrc="$(echo "$opt" | cut -d '=' -f 2-)" ;; --bin=*) bin="$(echo "$opt" | cut -d '=' -f 2-)" @@ -68,6 +69,7 @@ for opt in "${@}"; do --user) bin="$HOME/bin" base="$HOME/.bazel" + bazelrc="$HOME/.bazelrc" ;; *) usage @@ -77,6 +79,7 @@ done bin="${bin//%prefix%/${prefix}}" base="${base//%prefix%/${prefix}}" +bazelrc="${bazelrc//%prefix%/${prefix}}" test_write() { local file="$1" @@ -130,6 +133,7 @@ fi # Test for write access test_write "${bin}" test_write "${base}" +test_write "${bazelrc}" # Do the actual installation echo -n "Uncompressing." @@ -155,7 +159,17 @@ echo -n . ln -s "${base}/bin/bazel" "${bin}/bazel" echo -n . -if [ "${UID}" -ne 0 ]; then +if [ -f "${bazelrc}" ]; then + echo + echo "${bazelrc} already exists, moving it to ${bazelrc}.bak." + mv "${bazelrc}" "${bazelrc}.bak" +fi + +# Not necessary, but this way it matches the Debian package. +touch "${bazelrc}" +if [ "${UID}" -eq 0 ]; then + chmod 0644 "${bazelrc}" +else # Uncompress the bazel base install for faster startup time "${bin}/bazel" help >/dev/null fi |