From: Jim Hill <gjthill@gmail.com>
Date: Sun, 21 Dec 2014 19:12:45 GMT
Subject: Re: [PATCH w/signoff] pre-push.sample: Remove unwanted `IFS=' '`.
Message-ID: <CAEE75_0Ff7NfQYUPrA414N9E0AE6LsS2zs0kL=BJ25bjPgom_w@mail.gmail.com>
In-Reply-To: <xmqqtx0obzwm.fsf@gitster.dls.corp.google.com>

I call it unwanted because the default works fine with the actual
input and explicitly limiting whitespace this way breaks most command
substitution.  For instance, attempting to check only new commits with

    range="$local_sha $(
            git for-each-ref --format='^%(refname)' refs/remotes/$remote
    )"
    # ...
    commit=`git rev-list -n 1 --grep '\bbad string\b' $range`


fails, because the newlines are passed verbatim instead of being
treated as whitespace.

See http://stackoverflow.com/a/27392839/1290731 for the motivation.

Sorry for the capsing, and also for presuming ESP.