Skip to content

build script exit status ignored by tools/build.sh #72

Closed
donald opened this issue Jul 1, 2019 · 1 comment · Fixed by #77
Closed

build script exit status ignored by tools/build.sh #72

donald opened this issue Jul 1, 2019 · 1 comment · Fixed by #77

Comments

@donald
Copy link
Contributor

donald commented Jul 1, 2019

Since #66 the exist status of the build script is lost, because script dosn't forward the exist status:

buczek@theinternet:~/git/pkg-scripts (fix-build-log)$ script -q -c "false" x.log ; echo $?
0

So tools/build.sh now continues even if the build scripts fails. Bad.

The fact, that the logfile is now written by the build user instead of the calling user is another unintentional change of #66.

@thomas
Copy link
Contributor

thomas commented Jul 15, 2019

Completely agree on this. Changing a failed build to bin.bin is not clever.

thomas added a commit that referenced this issue Aug 1, 2019
'script -e' returns the exit code of the child process.

This yields again to the intended behaviour (Closes #72).
@donald donald closed this as completed in #77 Aug 1, 2019
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants