more jobs that use tee but not pipefail discovered
Bug #1096948 reported by
Jeff Lane
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Checkbox |
Fix Released
|
Medium
|
Jeff Lane |
Bug Description
Jeffrey Chang pointed out two more jobs that use tee, but don't properly use pipefail to ensure checkbox gets the correct exit code.
He suggests:
bluetooth/
suspend/
looking at bluetooth/
with suspend/
Related branches
lp://staging/~bladernr/checkbox/1096948-pipefail-on-bluetooth-detect
- Zygmunt Krynicki (community): Approve
-
Diff: 39 lines (+5/-2)3 files modifieddebian/changelog (+3/-0)
jobs/bluetooth.txt.in (+1/-1)
jobs/suspend.txt.in (+1/-1)
Changed in checkbox: | |
status: | New → In Progress |
assignee: | nobody → Jeff Lane (bladernr) |
importance: | Undecided → Medium |
Changed in checkbox: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
actually, after looking further, the original intent was to have suspend/ network_ after_suspend only run on success of suspend/ network_ before_ suspend, so I have also applied the pipefail fix to network_ before_ suspend as well.