as it gives the absolute (aka "physical") path of the toplevel directory and 'cd -P' is not supported by all shell implementations. See NetBSD PR/42168. http://www.netbsd.org/cgi-bin/query-pr-single.pl?number=42168 --- This is a revision of my original patch after the input from Jonathan. --- git-sh-setup.sh | 18 ++++-------------- 1 files changed, 4 insertions(+), 14 deletions(-) diff --git a/git-sh-setup.sh b/git-sh-setup.sh index dfcb807..4ecc578 100755 --- a/git-sh-setup.sh +++ b/git-sh-setup.sh @@ -120,20 +120,10 @@ is_bare_repository () { } cd_to_toplevel () { - cdup=$(git rev-parse --show-cdup) - if test ! -z "$cdup" - then - # The "-P" option says to follow "physical" directory - # structure instead of following symbolic links. When cdup is - # "../", this means following the ".." entry in the current - # directory instead textually removing a symlink path element - # from the PWD shell variable. The "-P" behavior is more - # consistent with the C-style chdir used by most of Git. - cd -P "$cdup" || { - echo >&2 "Cannot chdir to $cdup, the toplevel of the working tree" - exit 1 - } - fi + cdup=$(git rev-parse --show-toplevel) && cd "$cdup" || { + echo >&2 "Cannot chdir to $cdup, the toplevel of the working tree" + exit 1 + } } require_work_tree () { -- 1.6.4 -- 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