Pavel Roskin <proski@xxxxxxx> writes:
> Ignore configuration data other that "url" and "fetch" for the remote.
> We cannot process it to extract the remote name from it reliably.
> Besides, a remote without "url" is currently invalid, so we are not
> missing anything.
>
> Signed-off-by: Pavel Roskin <proski@xxxxxxx>
I do not think we ever officially "supported" remotes with a dot
in their names since $GIT_DIR/remotes/ or $GIT_DIR/branches
days.
> for (@remotes) {
> - if (/^remote\.([^.]*)\.(\S*)\s+(.*)$/) {
> + if (/^remote\.(\S*)\.(fetch|url)\s+(.*)$/) {
> add_remote_config(\%seen, $1, $2, $3);
> }
I do not strongly oppose to allowing it now, but I suspect this
is probably less impact:
if (/^remote\.(\S+?)\.([^.\s]+)\s+(.*)$/) {
...
}
With this, we disallow whitespaces in remote names, but we leave
the door open for supporting variables other than fetch and url
by accepting the third token that matches any sequence of
non-dot, non-whitespace letters.
-
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