Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | t9901-git-web--browse.sh: use the $( ... ) construct for command substitutionep/shell-command-substitution-style | Elia Pinto | 2016-01-12 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | | | | | | | The Git CodingGuidelines prefer the $(...) construct for command substitution instead of using the backquotes `...`. The backquoted form is the traditional method for command substitution, and is supported by POSIX. However, all but the simplest uses become complicated quickly. In particular, embedded command substitutions and/or the use of double quotes require careful escaping with the backslash character. The patch was generated by: for _f in $(find . -name "*.sh") do perl -i -pe 'BEGIN{undef $/;} s/`(.+?)`/\$(\1)/smg' "${_f}" done and then carefully proof-read. Signed-off-by: Elia Pinto <gitter.spiros@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | ||||
* | t9901: fix line-ending dependency on windows | Pat Thoyts | 2011-10-15 | 1 | -15/+12 |
| | | | | | Signed-off-by: Pat Thoyts <patthoyts@users.sourceforge.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | ||||
* | git-web--browse: avoid the use of evalcp/git-web-browse-browsers | Chris Packham | 2011-10-03 | 1 | -0/+66 |
Using eval causes problems when the URL contains an appropriately escaped ampersand (\&). Dropping eval from the built-in browser invocation avoids the problem. Helped-by: Jeff King <peff@peff.net> (test case) Signed-off-by: Chris Packham <judge.packham@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> |