Skip to content

Commit 40c6272

Browse files
authored
Always use |& with output::indent (#1917)
To ensure any stderr is intended too. (All other call sites currently use `|&`) See: https://www.gnu.org/software/bash/manual/html_node/Pipelines.html
1 parent 0576746 commit 40c6272

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

bin/steps/collectstatic

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ export PYTHONPATH
6767
COLLECTSTATIC_LOG=$(mktemp)
6868

6969
set +e
70-
python "${MANAGE_FILE}" collectstatic --noinput --traceback 2>&1 | tee "${COLLECTSTATIC_LOG}" | sed --unbuffered '/^Post-processed/d;/^Copying/d;/^$/d' | output::indent
70+
python "${MANAGE_FILE}" collectstatic --noinput --traceback |& tee "${COLLECTSTATIC_LOG}" |& sed --unbuffered '/^Post-processed/d;/^Copying/d;/^$/d' |& output::indent
7171
COLLECTSTATIC_STATUS="${PIPESTATUS[0]}"
7272
set -e
7373

lib/output.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ function output::step() {
2323
#
2424
# Usage:
2525
# ```
26-
# pip install ... | output::indent
26+
# pip install ... |& output::indent
2727
# ```
2828
function output::indent() {
2929
sed --unbuffered "s/^/ /"

0 commit comments

Comments
 (0)