Opened 9 years ago

Closed 9 years ago

#5370 closed bug (fixed)

[patch] swap order of shell stderr, stdout redirection

Reported by: mmadia Owned by: bonefish
Priority: normal Milestone: R1
Component: Build System Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: no Platform: All

Description

Following chanegset [35412] , this patch swaps the remaining instances of 2>&1 > /dev/null with > /dev/null 2>&1

Attachments (1)

SwapOrderOfShellRedirection-dev_null-2_1.patch (1.5 KB) - added by mmadia 9 years ago.

Download all attachments as: .zip

Change History (3)

comment:1 Changed 9 years ago by mmadia

To note, build_haiku_image#L138 has $bfsShell -n $imageOffsetFlags "$imagePath" > /dev/null & . I'm not sure if this should be updated to > /dev/null 2>&1

comment:2 in reply to:  1 Changed 9 years ago by bonefish

Resolution: fixed
Status: newclosed

Replying to mmadia:

To note, build_haiku_image#L138 has $bfsShell -n $imageOffsetFlags "$imagePath" > /dev/null & . I'm not sure if this should be updated to > /dev/null 2>&1

No, we actually want to see the errors that are printed.

Patch applied in hrev35595. Thanks!

Note: See TracTickets for help on using tickets.