Re: git --literal-pathspecs add -u says "fatal: pathspec ':/' did not match any files"

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]