]> git.ipfire.org Git - thirdparty/git.git/commit - setup.c
longest_ancestor_length(): require prefix list entries to be normalized
authorMichael Haggerty <mhagger@alum.mit.edu>
Sun, 28 Oct 2012 16:16:25 +0000 (17:16 +0100)
committerJeff King <peff@peff.net>
Mon, 29 Oct 2012 06:34:58 +0000 (02:34 -0400)
commit9e2326c7e1efb1ae42b17e3fa38c16711a8d0bd8
tree7d5bbc7f00411aa7b77d80cc50ec43a2a210b336
parent31171d9e454d71144685866cfd6476b8ac69d314
longest_ancestor_length(): require prefix list entries to be normalized

Move the responsibility for normalizing prefixes from
longest_ancestor_length() to its callers. Use slightly different
normalizations at the two callers:

In setup_git_directory_gently_1(), use the old normalization, which
ignores paths that are not usable.  In the next commit we will change
this caller to also resolve symlinks in the paths from
GIT_CEILING_DIRECTORIES as part of the normalization.

In "test-path-utils longest_ancestor_length", use the old
normalization, but die() if any paths are unusable.  Also change t0060
to only pass normalized paths to the test program (no empty entries or
non-absolute paths, strip trailing slashes from the paths, and remove
tests that thereby become redundant).

The point of this change is to reduce the scope of the ancestor_length
tests in t0060 from testing normalization+longest_prefix to testing
only mostly longest_prefix.  This is necessary because when
setup_git_directory_gently_1() starts resolving symlinks as part of
its normalization, it will not be reasonable to do the same in the
test suite, because that would make the test results depend on the
contents of the root directory of the filesystem on which the test is
run.  HOWEVER: under Windows, bash mangles arguments that look like
absolute POSIX paths into DOS paths.  So we have to retain the level
of normalization done by normalize_path_copy() to convert the
bash-mangled DOS paths (which contain backslashes) into paths that use
forward slashes.

Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu>
Signed-off-by: Jeff King <peff@peff.net>
path.c
setup.c
t/t0060-path-utils.sh
test-path-utils.c