diff options
| author |  Tim Henigan <tim.henigan@gmail.com> | 2012-02-24 18:12:58 -0500 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2012-02-27 15:10:23 -0800 |
| commit | 860f70f9f4206379361fd5af84c8fde31984b418 (patch) | |
| tree | 74806c7f14b85fa2e1075722b49e3c7e75377a1e /t/t7700-repack.sh | |
| parent | CodingGuidelines: Add a note about spaces after redirection (diff) | |
| download | tgif-860f70f9f4206379361fd5af84c8fde31984b418.tar.xz | |
CodingGuidelines: do not use 'which' in shell scripts
During the code review of a recent patch, it was noted that shell scripts
must not use 'which $cmd' to check the availability of the command $cmd.
The output of the command is not machine parseable and its exit code is
not reliable across platforms.
It is better to use 'type' to accomplish this task.
Signed-off-by: Tim Henigan <tim.henigan@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t7700-repack.sh')
0 files changed, 0 insertions, 0 deletions