admin/ci-emsdk
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Ensure temporary files have proper unique names for bazel builds. (#1415)

Browse Source 
This attempts to fix #1386.
This commit is contained in:
Daniel Kalmar
2024-06-27 19:50:03 +02:00
committed by GitHub
parent 8caaa25afb
commit a33daf378c
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
bazel/emscripten_toolchain/link_wrapper.py
View File

@@ -139,7 +139,7 @@ if os.path.exists(wasm_base + '.debug.wasm') and os.path.exists(wasm_base):
final_bytes.extend((base_name + '.wasm.debug.wasm').encode()) final_bytes.extend((base_name + '.wasm.debug.wasm').encode())
# Write our length + filename bytes to a temp file. # Write our length + filename bytes to a temp file.
with open('debugsection.tmp', 'wb+') as f: with open(base_name + '_debugsection.tmp', 'wb+') as f:
f.write(final_bytes) f.write(final_bytes)
f.close() f.close()
@@ -152,7 +152,7 @@ if os.path.exists(wasm_base + '.debug.wasm') and os.path.exists(wasm_base):
subprocess.check_call([ subprocess.check_call([
llvm_objcopy, llvm_objcopy,
wasm_base, wasm_base,
'--add-section=external_debug_info=debugsection.tmp']) '--add-section=external_debug_info=' + base_name + '_debugsection.tmp'])
# Make sure we have at least one output file. # Make sure we have at least one output file.
if not len(files): if not len(files):
@@ -160,8 +160,8 @@ if not len(files):
sys.exit(1) sys.exit(1)
# cc_binary must output exactly one file; put all the output files in a tarball. # cc_binary must output exactly one file; put all the output files in a tarball.
cmd = ['tar', 'cf', 'tmp.tar'] + files cmd = ['tar', 'cf', base_name + '.tar'] + files
subprocess.check_call(cmd, cwd=outdir) subprocess.check_call(cmd, cwd=outdir)
os.replace(os.path.join(outdir, 'tmp.tar'), output_file) os.replace(os.path.join(outdir, base_name + '.tar'), output_file)
sys.exit(0) sys.exit(0)