Wireshark-commits: [Wireshark-commits] master ea515c5: Don't assume Bash.
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Tue, 21 Aug 2018 09:38:31 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=ea515c5f2da5995c37dd7fdc9d8a81aa2515c847
Submitter: "Guy Harris <guy@xxxxxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

ea515c5 by Guy Harris (guy@xxxxxxxxxxxx):

    Don't assume Bash.
    
    At least on my OpenBSD 6.3 VM, there's no Bash installed; just use
    /bin/sh.
    
    Fix two places that use csh-style ">& /dev/null" to use
    "> /dev/null 2>&1" instead.
    
    Change-Id: I48656c47e89b3ad09f3c2d9e3c90dfad7afabb71
    Reviewed-on: https://code.wireshark.org/review/29214
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  5673a58   AT: Fix suffix length checks, add no suffix case
     add  ea515c5   Don't assume Bash.


Summary of changes:
 tools/bsd-setup.sh | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)