revi-archive/llvm-premerge-checks
1
0
Fork 0
Code Activity

fix excessive environment copying

Browse source
This commit is contained in:
Mikhail Goncharov 2021-10-04 16:58:52 +02:00
parent b69eb6f364
commit 77cbb15b64
3 changed files with 5 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
scripts/llvm-dependencies.yaml
View file

@ -63,7 +63,7 @@ allprojects:
mlir: ["check-mlir"]
openmp: ["check-openmp"]
parallel-libs: ["check-all"]
polly: ["check-polly-tests", "check-polly"]
polly: ["check-polly-tests", "check-polly"] # TODO: redundant?
pstl: ["check-all"] # There does not seem to be a more specific target.
llvm: ["check-llvm"]

5
scripts/pipeline_main.py
View file

@ -50,12 +50,11 @@ if __name__ == '__main__':
steps.extend(generic_windows(os.getenv('ph_projects', ';'.join(windows_projects))))
steps.extend(bazel([], force=True))
if os.getenv('ph_skip_generated') is None:
gen_env = os.environ.copy()
# BUILDKITE_COMMIT might be an alias, e.g. "HEAD". Resolve it to make the build hermetic.
if ('BUILDKITE_COMMIT' not in env) or (env['BUILDKITE_COMMIT'] == "HEAD"):
if os.getenv('BUILDKITE_COMMIT', 'HEAD') == "HEAD":
env['BUILDKITE_COMMIT'] = repo.head.commit.hexsha
for gen in steps_generators:
steps.extend(from_shell_output(gen, env=gen_env))
steps.extend(from_shell_output(gen, env=env))
notify = []
for e in notify_emails:

5
scripts/pipeline_premerge.py
View file

@ -68,9 +68,8 @@ if __name__ == '__main__':
# Add custom checks.
if os.getenv('ph_skip_generated') is None:
env = os.environ.copy()
# BUILDKITE_COMMIT might be an alias, e.g. "HEAD". Resolve it to make the build hermetic.
env["BUILDKITE_COMMIT"] = repo.head.commit.hexsha
if os.getenv('BUILDKITE_COMMIT', 'HEAD') == "HEAD":
env['BUILDKITE_COMMIT'] = repo.head.commit.hexsha
for gen in steps_generators:
steps.extend(from_shell_output(gen, env=env))
modified_files = cp.get_changed_files(patch)