diff options
author | Renato Botelho <renato@netgate.com> | 2016-02-05 09:53:46 -0200 |
---|---|---|
committer | Renato Botelho <renato@netgate.com> | 2016-02-05 09:53:46 -0200 |
commit | f1d01c3ee747a7392cf3df8175aa362c4a24e675 (patch) | |
tree | e6b3a97381232878d31cec252319781815ec861c /tools | |
parent | 23400c8b92a026c343b6aef6bc4f0fef5a4b48b2 (diff) | |
download | pfsense-f1d01c3ee747a7392cf3df8175aa362c4a24e675.zip pfsense-f1d01c3ee747a7392cf3df8175aa362c4a24e675.tar.gz |
Fix #5837
Since the introduction of pkg_with_pb(), direct calls to this function
were added using the following format:
if ! pkg_with_pb ... | tee -a $logfile; then
What happened in this case is if was always getting a successfull
return code from tee call, and pkg_with_pb return was always being
ignored leading pfSense-upgrade to keep running when a fatal error
happened.
Since we need to print pkg output and also send it to log, _exec() call
is the best option here.
Added necessary flags to make sure necessary actions (unlock additional
packages or cleaning up kernel package annotation) are executed on
_exit call in fatal cases and replaced all direct calls by _exec()
Diffstat (limited to 'tools')
0 files changed, 0 insertions, 0 deletions