Skip to content

Commit

Permalink
🐛 Fix issue where mach proxy was not inheriting the exit code
Browse files Browse the repository at this point in the history
  • Loading branch information
kierandrewett committed Feb 16, 2023
1 parent 40673b2 commit 1d3ebda
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 10 deletions.
9 changes: 0 additions & 9 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -50,15 +50,6 @@ jobs:
working-directory: gecko/dot
run: ./mach import-patches

- name: Assert mozconfig is setup
working-directory: gecko
shell: bash
run: |
if !grep -q $topsrcdir/dot/config/mozconfigs/common mozconfig; then
echo Assertion failed! mozconfig is not setup.
exit 1
fi
- name: Build browser
working-directory: gecko/dot
run: ./mach build
4 changes: 3 additions & 1 deletion mach
Original file line number Diff line number Diff line change
Expand Up @@ -13,4 +13,6 @@ os.chdir("..")

print(f"Executing mach from cwd {os.getcwd()}.")

os.system(" ".join(sys.argv))
sys.exit(
os.WEXITSTATUS(os.system(" ".join(sys.argv)))
)

0 comments on commit 1d3ebda

Please sign in to comment.