opendev/gerrit
Code Issues Proposed changes

Merge "Bazel: Fix invocation of mktemp for OSX pre El Capitan"

Browse Source
This commit is contained in:
David Ostrovsky
2016-10-27 08:25:58 +00:00
committed by Gerrit Code Review
parent c3bf9587d5 2b64c44f1a
commit 79f16bc27a
3 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
polygerrit-ui/app/BUILD
View File

@@ -43,7 +43,7 @@ filegroup(
genrule( genrule(
name = "polygerrit_ui", name = "polygerrit_ui",
cmd = " && ".join([ cmd = " && ".join([
"t=$$(mktemp -d)", "t=$$(mktemp -d || mktemp -d -t bazel-tmp)",
"p=$$PWD", "p=$$PWD",
"mkdir -p $$t/polygerrit_ui/{styles,fonts,bower_components/{highlightjs,webcomponentsjs},elements}", "mkdir -p $$t/polygerrit_ui/{styles,fonts,bower_components/{highlightjs,webcomponentsjs},elements}",
"cp $(locations :gr-app) $$t/polygerrit_ui/elements/", "cp $(locations :gr-app) $$t/polygerrit_ui/elements/",

2
tools/bzl/genrule2.bzl
View File

@@ -20,7 +20,7 @@
def genrule2(out, cmd, **kwargs): def genrule2(out, cmd, **kwargs):
cmd = ' && '.join([ cmd = ' && '.join([
'ROOT=$$PWD', 'ROOT=$$PWD',
'TMP=$$(mktemp -d)', 'TMP=$$(mktemp -d || mktemp -d -t bazel-tmp)',
'(' + cmd + ')', '(' + cmd + ')',
]) ])
native.genrule( native.genrule(

2
tools/bzl/js.bzl
View File

@@ -87,7 +87,7 @@ def _bower_archive(ctx):
fail("failed %s: %s" % (" ".join(cmd), out.stderr)) fail("failed %s: %s" % (" ".join(cmd), out.stderr))
_bash(ctx, " && " .join([ _bash(ctx, " && " .join([
"TMP=$(mktemp -d )", "TMP=$(mktemp -d || mkdir -d -t bazel-tmp)",
"cd $TMP", "cd $TMP",
"mkdir bower_components", "mkdir bower_components",
"cd bower_components", "cd bower_components",