Re: [PATCH 3/3] completion: fix bugs with slashes in remote names

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

 



David Mandelberg <david@xxxxxxxxxxxxxx> writes:

>> As a remote name, we still do use "is that a
>> file on the filesystem?" to see if it is a local file:// URL
>> (i.e. "git fetch github/dseomn" may be fetching from a subdirectory
>> two levels down)
>
> Btw, I just tested it, and I think this is an issue for remote names
> without slashes too:

Yes, and that is an intended way local repositories are discovered
and used.

The problem with slashes is that the mistake surface becomes a lot
larger.  "ls<RETURN>" would show you "origin" immediately in the
current working directory, but you wouldn't know if origin has
origin/foo or origin/bar unless you look.  Limiting the local
filesystem reference to something like "only the ones that has no
slashes, or begin with ../ or /" would make it less likely that you
meant to push to somewhere and instead push to a local directory.





[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]

  Powered by Linux