Ted Pavlic <ted@xxxxxxxxxxxxx> wrote: > In the single case of: > > if [...] || [...] > > changed to: > > if test ... || test ... NAK. This script only runs in bash. bash supports [...]. The prevailing convention in the script is to use [...]. Only 4 tests inside of __git_ps1 use "test", the rest of the code is using [...]. I would agree to a test->[...] conversion patch as its fairly small, but not this one. Too large, too much code churn, no benefit. -- Shawn. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
