tree d161554cd9b4e15b8e0f5560f8b39a3ca14125b1
parent 3ad30c608bfcacc597c950cfa63bff050aa1a8c2
author Grant Grundler <grundler@chromium.org> 1526338406 -0700
committer chrome-bot <chrome-bot@chromium.org> 1526456468 -0700

check_ethernet: switch to /bin/sh (dash)

There are technical reasons why /bin/sh (currently dash) is the preferred
shell for ChromeOS, not bash.  Since checkbashism is reporting no errors
or warnings, should be simple to change right?

I expect so but wouldn't not be offended if sheriff or
trooper has to revert this change.

BUG=none
TEST=sudo apt-get install devscripts # outside of chroot
    checkbashism check_ethernet.hook # returns no output
    scp recover_duts/hooks/check_ethernet.hook  kefka.lab:/usr/local/bin/hooks/check_ethernet.hook
    ssh kefka.lab
    touch /mnt/stateful_partition/.labmachine
    # unplug link, wait for machine to reboot after ~2 minutes
    # log in again to review /var/log/messages and verify no
    #     shell errors are reported by check_ethernet.hook.
    #     (just the usually "attempting method ..." etc)
    rm /mnt/stateful_partition/.labmachine

Change-Id: I1ace53b339671caf3ba3a6359b5626b429459259
Reviewed-on: https://chromium-review.googlesource.com/1058443
Commit-Ready: Kirtika Ruchandani <kirtika@chromium.org>
Tested-by: Grant Grundler <grundler@chromium.org>
Reviewed-by: Kirtika Ruchandani <kirtika@chromium.org>
Reviewed-by: Grant Grundler <grundler@chromium.org>