Opened 10 years ago

Closed 10 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 10 years ago.

Download all attachments as: .zip

Change History (3)

comment:1 by mmadia, 10 years ago

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

in reply to:  1 comment:2 by bonefish, 10 years ago

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.