Hello Noam,
The problem is that in the absence of explicit argument we set the list
of files to special path ":/" which means repo root:
if ((0 < addremove_explicit || take_worktree_changes) && !argc) {
static const char *whole[2] = { ":/", NULL };
argc = 1;
argv = whole;
}
And after that we treat it as regular file
if (!seen[i] && path[0] &&
((pathspec.items[i].magic &
(PATHSPEC_GLOB | PATHSPEC_ICASE)) ||
!file_exists(path))) { /* <========
file_exists() here just checks lstat() result */
Maybe it'll make sense to modify file_exists() to treat ":/" specially.
Something like that:
diff --git a/dir.c b/dir.c
index 109ceea..6cae3b9 100644
--- a/dir.c
+++ b/dir.c
@@ -2103,6 +2103,10 @@ int read_directory(struct dir_struct *dir, const
char *path, int len, const stru
int file_exists(const char *f)
{
struct stat sb;
+ if (!strcmp(f, ":/")) {
+ /* ":/" - root dir, always exists */
+ return 1;
+ }
return lstat(f, &sb) == 0;
}
--
Victor
On 10/24/2015 02:39 AM, Noam Postavsky wrote:
~/tmp/tmprepo$ git init
Initialized empty Git repository in /home/npostavs/tmp/tmprepo/.git/
~/tmp/tmprepo$ git --literal-pathspecs add -u
fatal: pathspec ':/' did not match any files
~/tmp/tmprepo$ git --version
git version 2.6.1
It was reported[1] that 2.0.2 and several following versions also fail
with the same error; I found that version 1.9.5 succeeds.
Adding a "." argument:
git --literal-pathspecs add -u .
succeeds in all versions.
[1]: https://github.com/magit/magit/issues/2354#issuecomment-150665961
--
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
--
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