Thread (2 messages) flat view 2 messages, 2 authors, 2016-06-15

Re: [PATCH w/signoff] pre-push.sample: Remove unwanted `IFS=' '`.

From: Jim Hill <hidden>
Date: 2016-06-15 23:03:19

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.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help