Am 10/29/2012 0:28, schrieb Jens Lehmann:
> +     /* Remove trailing '/' from directories to find submodules in the index 
> */
> +     for (i = 0; i < argc; i++) {
> +             size_t pathlen = strlen(argv[i]);
> +             if (pathlen && is_directory(argv[i]) && (argv[i][pathlen - 1] 
> == '/'))
> +                     argv[i] = xmemdupz(argv[i], pathlen - 1);
> +     }
> +
>       pathspec = get_pathspec(prefix, argv);
>       refresh_index(&the_index, REFRESH_QUIET, pathspec, NULL, NULL);

That's wrong: Either move the check below get_pathspec() (which normalizes
backslashes to forward-slashes on Windows) or use is_dir_sep().

But isn't it somewhat dangerous to check pathspec for existance in the
worktree without interpreting them? Think of magic pathspec syntax (that
we do not have yet, but which may materialize sometime in the future).

-- Hannes
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to