2007-05-12 23:45:53 +08:00
|
|
|
#include "cache.h"
|
|
|
|
#include "remote.h"
|
|
|
|
#include "refs.h"
|
|
|
|
|
2008-02-25 14:25:04 +08:00
|
|
|
struct counted_string {
|
|
|
|
size_t len;
|
|
|
|
const char *s;
|
|
|
|
};
|
2008-02-21 02:43:53 +08:00
|
|
|
struct rewrite {
|
|
|
|
const char *base;
|
2008-02-25 14:25:04 +08:00
|
|
|
size_t baselen;
|
|
|
|
struct counted_string *instead_of;
|
2008-02-21 02:43:53 +08:00
|
|
|
int instead_of_nr;
|
|
|
|
int instead_of_alloc;
|
|
|
|
};
|
|
|
|
|
2007-05-12 23:45:53 +08:00
|
|
|
static struct remote **remotes;
|
2008-02-19 12:41:41 +08:00
|
|
|
static int remotes_alloc;
|
|
|
|
static int remotes_nr;
|
2007-05-12 23:45:53 +08:00
|
|
|
|
2007-09-11 11:02:56 +08:00
|
|
|
static struct branch **branches;
|
2008-02-19 12:41:41 +08:00
|
|
|
static int branches_alloc;
|
|
|
|
static int branches_nr;
|
2007-09-11 11:02:56 +08:00
|
|
|
|
|
|
|
static struct branch *current_branch;
|
|
|
|
static const char *default_remote_name;
|
|
|
|
|
2008-02-21 02:43:53 +08:00
|
|
|
static struct rewrite **rewrite;
|
|
|
|
static int rewrite_alloc;
|
|
|
|
static int rewrite_nr;
|
|
|
|
|
2007-05-12 23:45:53 +08:00
|
|
|
#define BUF_SIZE (2048)
|
|
|
|
static char buffer[BUF_SIZE];
|
|
|
|
|
2008-02-21 02:43:53 +08:00
|
|
|
static const char *alias_url(const char *url)
|
|
|
|
{
|
|
|
|
int i, j;
|
2008-02-25 14:25:04 +08:00
|
|
|
char *ret;
|
|
|
|
struct counted_string *longest;
|
|
|
|
int longest_i;
|
|
|
|
|
|
|
|
longest = NULL;
|
|
|
|
longest_i = -1;
|
2008-02-21 02:43:53 +08:00
|
|
|
for (i = 0; i < rewrite_nr; i++) {
|
|
|
|
if (!rewrite[i])
|
|
|
|
continue;
|
|
|
|
for (j = 0; j < rewrite[i]->instead_of_nr; j++) {
|
2008-02-25 14:25:04 +08:00
|
|
|
if (!prefixcmp(url, rewrite[i]->instead_of[j].s) &&
|
|
|
|
(!longest ||
|
|
|
|
longest->len < rewrite[i]->instead_of[j].len)) {
|
|
|
|
longest = &(rewrite[i]->instead_of[j]);
|
|
|
|
longest_i = i;
|
2008-02-21 02:43:53 +08:00
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
2008-02-25 14:25:04 +08:00
|
|
|
if (!longest)
|
|
|
|
return url;
|
|
|
|
|
|
|
|
ret = malloc(rewrite[longest_i]->baselen +
|
|
|
|
(strlen(url) - longest->len) + 1);
|
|
|
|
strcpy(ret, rewrite[longest_i]->base);
|
|
|
|
strcpy(ret + rewrite[longest_i]->baselen, url + longest->len);
|
|
|
|
return ret;
|
2008-02-21 02:43:53 +08:00
|
|
|
}
|
|
|
|
|
2007-05-12 23:45:53 +08:00
|
|
|
static void add_push_refspec(struct remote *remote, const char *ref)
|
|
|
|
{
|
2008-02-19 12:41:41 +08:00
|
|
|
ALLOC_GROW(remote->push_refspec,
|
|
|
|
remote->push_refspec_nr + 1,
|
|
|
|
remote->push_refspec_alloc);
|
|
|
|
remote->push_refspec[remote->push_refspec_nr++] = ref;
|
2007-05-12 23:45:53 +08:00
|
|
|
}
|
|
|
|
|
2007-05-12 23:46:03 +08:00
|
|
|
static void add_fetch_refspec(struct remote *remote, const char *ref)
|
|
|
|
{
|
2008-02-19 12:41:41 +08:00
|
|
|
ALLOC_GROW(remote->fetch_refspec,
|
|
|
|
remote->fetch_refspec_nr + 1,
|
|
|
|
remote->fetch_refspec_alloc);
|
|
|
|
remote->fetch_refspec[remote->fetch_refspec_nr++] = ref;
|
2007-05-12 23:46:03 +08:00
|
|
|
}
|
|
|
|
|
2007-09-19 12:49:27 +08:00
|
|
|
static void add_url(struct remote *remote, const char *url)
|
2007-05-12 23:45:53 +08:00
|
|
|
{
|
2008-02-19 12:41:41 +08:00
|
|
|
ALLOC_GROW(remote->url, remote->url_nr + 1, remote->url_alloc);
|
|
|
|
remote->url[remote->url_nr++] = url;
|
2007-05-12 23:45:53 +08:00
|
|
|
}
|
|
|
|
|
2008-02-21 02:43:53 +08:00
|
|
|
static void add_url_alias(struct remote *remote, const char *url)
|
|
|
|
{
|
|
|
|
add_url(remote, alias_url(url));
|
|
|
|
}
|
|
|
|
|
2007-05-12 23:45:53 +08:00
|
|
|
static struct remote *make_remote(const char *name, int len)
|
|
|
|
{
|
2008-02-19 12:41:41 +08:00
|
|
|
struct remote *ret;
|
|
|
|
int i;
|
2007-05-12 23:45:53 +08:00
|
|
|
|
2008-02-19 12:41:41 +08:00
|
|
|
for (i = 0; i < remotes_nr; i++) {
|
|
|
|
if (len ? (!strncmp(name, remotes[i]->name, len) &&
|
|
|
|
!remotes[i]->name[len]) :
|
|
|
|
!strcmp(name, remotes[i]->name))
|
|
|
|
return remotes[i];
|
2007-05-12 23:45:53 +08:00
|
|
|
}
|
|
|
|
|
2008-02-19 12:41:41 +08:00
|
|
|
ret = xcalloc(1, sizeof(struct remote));
|
|
|
|
ALLOC_GROW(remotes, remotes_nr + 1, remotes_alloc);
|
|
|
|
remotes[remotes_nr++] = ret;
|
2007-05-12 23:45:53 +08:00
|
|
|
if (len)
|
2008-02-19 12:41:41 +08:00
|
|
|
ret->name = xstrndup(name, len);
|
2007-05-12 23:45:53 +08:00
|
|
|
else
|
2008-02-19 12:41:41 +08:00
|
|
|
ret->name = xstrdup(name);
|
|
|
|
return ret;
|
2007-05-12 23:45:53 +08:00
|
|
|
}
|
|
|
|
|
2007-09-11 11:02:56 +08:00
|
|
|
static void add_merge(struct branch *branch, const char *name)
|
|
|
|
{
|
2008-02-19 12:41:41 +08:00
|
|
|
ALLOC_GROW(branch->merge_name, branch->merge_nr + 1,
|
|
|
|
branch->merge_alloc);
|
|
|
|
branch->merge_name[branch->merge_nr++] = name;
|
2007-09-11 11:02:56 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
static struct branch *make_branch(const char *name, int len)
|
|
|
|
{
|
2008-02-19 12:41:41 +08:00
|
|
|
struct branch *ret;
|
|
|
|
int i;
|
2007-09-11 11:02:56 +08:00
|
|
|
char *refname;
|
|
|
|
|
2008-02-19 12:41:41 +08:00
|
|
|
for (i = 0; i < branches_nr; i++) {
|
|
|
|
if (len ? (!strncmp(name, branches[i]->name, len) &&
|
|
|
|
!branches[i]->name[len]) :
|
|
|
|
!strcmp(name, branches[i]->name))
|
|
|
|
return branches[i];
|
2007-09-11 11:02:56 +08:00
|
|
|
}
|
|
|
|
|
2008-02-19 12:41:41 +08:00
|
|
|
ALLOC_GROW(branches, branches_nr + 1, branches_alloc);
|
|
|
|
ret = xcalloc(1, sizeof(struct branch));
|
|
|
|
branches[branches_nr++] = ret;
|
2007-09-11 11:02:56 +08:00
|
|
|
if (len)
|
2008-02-19 12:41:41 +08:00
|
|
|
ret->name = xstrndup(name, len);
|
2007-09-11 11:02:56 +08:00
|
|
|
else
|
2008-02-19 12:41:41 +08:00
|
|
|
ret->name = xstrdup(name);
|
2007-09-11 11:02:56 +08:00
|
|
|
refname = malloc(strlen(name) + strlen("refs/heads/") + 1);
|
|
|
|
strcpy(refname, "refs/heads/");
|
2008-02-19 12:41:41 +08:00
|
|
|
strcpy(refname + strlen("refs/heads/"), ret->name);
|
|
|
|
ret->refname = refname;
|
2007-09-11 11:02:56 +08:00
|
|
|
|
2008-02-19 12:41:41 +08:00
|
|
|
return ret;
|
2007-09-11 11:02:56 +08:00
|
|
|
}
|
|
|
|
|
2008-02-21 02:43:53 +08:00
|
|
|
static struct rewrite *make_rewrite(const char *base, int len)
|
|
|
|
{
|
|
|
|
struct rewrite *ret;
|
|
|
|
int i;
|
|
|
|
|
|
|
|
for (i = 0; i < rewrite_nr; i++) {
|
2008-02-25 14:25:04 +08:00
|
|
|
if (len
|
|
|
|
? (len == rewrite[i]->baselen &&
|
|
|
|
!strncmp(base, rewrite[i]->base, len))
|
|
|
|
: !strcmp(base, rewrite[i]->base))
|
2008-02-21 02:43:53 +08:00
|
|
|
return rewrite[i];
|
|
|
|
}
|
|
|
|
|
|
|
|
ALLOC_GROW(rewrite, rewrite_nr + 1, rewrite_alloc);
|
|
|
|
ret = xcalloc(1, sizeof(struct rewrite));
|
|
|
|
rewrite[rewrite_nr++] = ret;
|
2008-02-25 14:25:04 +08:00
|
|
|
if (len) {
|
2008-02-21 02:43:53 +08:00
|
|
|
ret->base = xstrndup(base, len);
|
2008-02-25 14:25:04 +08:00
|
|
|
ret->baselen = len;
|
|
|
|
}
|
|
|
|
else {
|
2008-02-21 02:43:53 +08:00
|
|
|
ret->base = xstrdup(base);
|
2008-02-25 14:25:04 +08:00
|
|
|
ret->baselen = strlen(base);
|
|
|
|
}
|
2008-02-21 02:43:53 +08:00
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
|
|
|
static void add_instead_of(struct rewrite *rewrite, const char *instead_of)
|
|
|
|
{
|
|
|
|
ALLOC_GROW(rewrite->instead_of, rewrite->instead_of_nr + 1, rewrite->instead_of_alloc);
|
2008-02-25 14:25:04 +08:00
|
|
|
rewrite->instead_of[rewrite->instead_of_nr].s = instead_of;
|
|
|
|
rewrite->instead_of[rewrite->instead_of_nr].len = strlen(instead_of);
|
|
|
|
rewrite->instead_of_nr++;
|
2008-02-21 02:43:53 +08:00
|
|
|
}
|
|
|
|
|
2007-05-12 23:45:53 +08:00
|
|
|
static void read_remotes_file(struct remote *remote)
|
|
|
|
{
|
|
|
|
FILE *f = fopen(git_path("remotes/%s", remote->name), "r");
|
|
|
|
|
|
|
|
if (!f)
|
|
|
|
return;
|
|
|
|
while (fgets(buffer, BUF_SIZE, f)) {
|
|
|
|
int value_list;
|
|
|
|
char *s, *p;
|
|
|
|
|
|
|
|
if (!prefixcmp(buffer, "URL:")) {
|
|
|
|
value_list = 0;
|
|
|
|
s = buffer + 4;
|
|
|
|
} else if (!prefixcmp(buffer, "Push:")) {
|
|
|
|
value_list = 1;
|
|
|
|
s = buffer + 5;
|
2007-05-12 23:46:03 +08:00
|
|
|
} else if (!prefixcmp(buffer, "Pull:")) {
|
|
|
|
value_list = 2;
|
|
|
|
s = buffer + 5;
|
2007-05-12 23:45:53 +08:00
|
|
|
} else
|
|
|
|
continue;
|
|
|
|
|
|
|
|
while (isspace(*s))
|
|
|
|
s++;
|
|
|
|
if (!*s)
|
|
|
|
continue;
|
|
|
|
|
|
|
|
p = s + strlen(s);
|
|
|
|
while (isspace(p[-1]))
|
|
|
|
*--p = 0;
|
|
|
|
|
|
|
|
switch (value_list) {
|
|
|
|
case 0:
|
2008-02-21 02:43:53 +08:00
|
|
|
add_url_alias(remote, xstrdup(s));
|
2007-05-12 23:45:53 +08:00
|
|
|
break;
|
|
|
|
case 1:
|
|
|
|
add_push_refspec(remote, xstrdup(s));
|
|
|
|
break;
|
2007-05-12 23:46:03 +08:00
|
|
|
case 2:
|
|
|
|
add_fetch_refspec(remote, xstrdup(s));
|
|
|
|
break;
|
2007-05-12 23:45:53 +08:00
|
|
|
}
|
|
|
|
}
|
|
|
|
fclose(f);
|
|
|
|
}
|
|
|
|
|
|
|
|
static void read_branches_file(struct remote *remote)
|
|
|
|
{
|
|
|
|
const char *slash = strchr(remote->name, '/');
|
2007-09-11 11:02:56 +08:00
|
|
|
char *frag;
|
2008-03-26 07:35:28 +08:00
|
|
|
struct strbuf branch;
|
2007-05-12 23:45:53 +08:00
|
|
|
int n = slash ? slash - remote->name : 1000;
|
|
|
|
FILE *f = fopen(git_path("branches/%.*s", n, remote->name), "r");
|
|
|
|
char *s, *p;
|
|
|
|
int len;
|
|
|
|
|
|
|
|
if (!f)
|
|
|
|
return;
|
|
|
|
s = fgets(buffer, BUF_SIZE, f);
|
|
|
|
fclose(f);
|
|
|
|
if (!s)
|
|
|
|
return;
|
|
|
|
while (isspace(*s))
|
|
|
|
s++;
|
|
|
|
if (!*s)
|
|
|
|
return;
|
|
|
|
p = s + strlen(s);
|
|
|
|
while (isspace(p[-1]))
|
|
|
|
*--p = 0;
|
|
|
|
len = p - s;
|
|
|
|
if (slash)
|
|
|
|
len += strlen(slash);
|
|
|
|
p = xmalloc(len + 1);
|
|
|
|
strcpy(p, s);
|
|
|
|
if (slash)
|
|
|
|
strcat(p, slash);
|
2008-03-26 07:35:28 +08:00
|
|
|
|
|
|
|
/*
|
|
|
|
* With "slash", e.g. "git fetch jgarzik/netdev-2.6" when
|
|
|
|
* reading from $GIT_DIR/branches/jgarzik fetches "HEAD" from
|
|
|
|
* the partial URL obtained from the branches file plus
|
|
|
|
* "/netdev-2.6" and does not store it in any tracking ref.
|
|
|
|
* #branch specifier in the file is ignored.
|
|
|
|
*
|
|
|
|
* Otherwise, the branches file would have URL and optionally
|
|
|
|
* #branch specified. The "master" (or specified) branch is
|
|
|
|
* fetched and stored in the local branch of the same name.
|
|
|
|
*/
|
|
|
|
strbuf_init(&branch, 0);
|
2007-09-11 11:02:56 +08:00
|
|
|
frag = strchr(p, '#');
|
|
|
|
if (frag) {
|
|
|
|
*(frag++) = '\0';
|
2008-03-26 07:35:28 +08:00
|
|
|
strbuf_addf(&branch, "refs/heads/%s", frag);
|
|
|
|
} else
|
|
|
|
strbuf_addstr(&branch, "refs/heads/master");
|
|
|
|
if (!slash) {
|
|
|
|
strbuf_addf(&branch, ":refs/heads/%s", remote->name);
|
2007-09-11 11:02:56 +08:00
|
|
|
} else {
|
2008-03-26 07:35:28 +08:00
|
|
|
strbuf_reset(&branch);
|
|
|
|
strbuf_addstr(&branch, "HEAD:");
|
2007-09-11 11:02:56 +08:00
|
|
|
}
|
2008-02-21 02:43:53 +08:00
|
|
|
add_url_alias(remote, p);
|
2008-03-26 07:35:28 +08:00
|
|
|
add_fetch_refspec(remote, strbuf_detach(&branch, 0));
|
2007-09-11 11:03:08 +08:00
|
|
|
remote->fetch_tags = 1; /* always auto-follow */
|
2007-05-12 23:45:53 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
static int handle_config(const char *key, const char *value)
|
|
|
|
{
|
|
|
|
const char *name;
|
|
|
|
const char *subkey;
|
|
|
|
struct remote *remote;
|
2007-09-11 11:02:56 +08:00
|
|
|
struct branch *branch;
|
|
|
|
if (!prefixcmp(key, "branch.")) {
|
|
|
|
name = key + 7;
|
|
|
|
subkey = strrchr(name, '.');
|
|
|
|
if (!subkey)
|
|
|
|
return 0;
|
2007-12-15 12:34:56 +08:00
|
|
|
branch = make_branch(name, subkey - name);
|
2007-09-11 11:02:56 +08:00
|
|
|
if (!strcmp(subkey, ".remote")) {
|
2008-02-12 03:00:10 +08:00
|
|
|
if (!value)
|
|
|
|
return config_error_nonbool(key);
|
2007-09-11 11:02:56 +08:00
|
|
|
branch->remote_name = xstrdup(value);
|
|
|
|
if (branch == current_branch)
|
|
|
|
default_remote_name = branch->remote_name;
|
2008-02-12 03:00:10 +08:00
|
|
|
} else if (!strcmp(subkey, ".merge")) {
|
|
|
|
if (!value)
|
|
|
|
return config_error_nonbool(key);
|
2007-09-11 11:02:56 +08:00
|
|
|
add_merge(branch, xstrdup(value));
|
2008-02-12 03:00:10 +08:00
|
|
|
}
|
2007-09-11 11:02:56 +08:00
|
|
|
return 0;
|
2007-05-12 23:45:53 +08:00
|
|
|
}
|
2008-02-21 02:43:53 +08:00
|
|
|
if (!prefixcmp(key, "url.")) {
|
|
|
|
struct rewrite *rewrite;
|
2008-04-13 03:32:00 +08:00
|
|
|
name = key + 4;
|
2008-02-21 02:43:53 +08:00
|
|
|
subkey = strrchr(name, '.');
|
|
|
|
if (!subkey)
|
|
|
|
return 0;
|
|
|
|
rewrite = make_rewrite(name, subkey - name);
|
|
|
|
if (!strcmp(subkey, ".insteadof")) {
|
|
|
|
if (!value)
|
|
|
|
return config_error_nonbool(key);
|
|
|
|
add_instead_of(rewrite, xstrdup(value));
|
|
|
|
}
|
|
|
|
}
|
2007-05-12 23:45:53 +08:00
|
|
|
if (prefixcmp(key, "remote."))
|
|
|
|
return 0;
|
|
|
|
name = key + 7;
|
|
|
|
subkey = strrchr(name, '.');
|
|
|
|
if (!subkey)
|
|
|
|
return error("Config with no key for remote %s", name);
|
|
|
|
if (*subkey == '/') {
|
|
|
|
warning("Config remote shorthand cannot begin with '/': %s", name);
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
remote = make_remote(name, subkey - name);
|
|
|
|
if (!value) {
|
|
|
|
/* if we ever have a boolean variable, e.g. "remote.*.disabled"
|
|
|
|
* [remote "frotz"]
|
|
|
|
* disabled
|
|
|
|
* is a valid way to set it to true; we get NULL in value so
|
|
|
|
* we need to handle it here.
|
|
|
|
*
|
|
|
|
* if (!strcmp(subkey, ".disabled")) {
|
|
|
|
* val = git_config_bool(key, value);
|
|
|
|
* return 0;
|
|
|
|
* } else
|
|
|
|
*
|
|
|
|
*/
|
|
|
|
return 0; /* ignore unknown booleans */
|
|
|
|
}
|
|
|
|
if (!strcmp(subkey, ".url")) {
|
2007-09-19 12:49:27 +08:00
|
|
|
add_url(remote, xstrdup(value));
|
2007-05-12 23:45:53 +08:00
|
|
|
} else if (!strcmp(subkey, ".push")) {
|
|
|
|
add_push_refspec(remote, xstrdup(value));
|
2007-05-12 23:46:03 +08:00
|
|
|
} else if (!strcmp(subkey, ".fetch")) {
|
|
|
|
add_fetch_refspec(remote, xstrdup(value));
|
2007-05-12 23:45:53 +08:00
|
|
|
} else if (!strcmp(subkey, ".receivepack")) {
|
|
|
|
if (!remote->receivepack)
|
|
|
|
remote->receivepack = xstrdup(value);
|
|
|
|
else
|
|
|
|
error("more than one receivepack given, using the first");
|
2007-09-11 11:02:51 +08:00
|
|
|
} else if (!strcmp(subkey, ".uploadpack")) {
|
|
|
|
if (!remote->uploadpack)
|
|
|
|
remote->uploadpack = xstrdup(value);
|
|
|
|
else
|
|
|
|
error("more than one uploadpack given, using the first");
|
2007-09-11 11:03:08 +08:00
|
|
|
} else if (!strcmp(subkey, ".tagopt")) {
|
|
|
|
if (!strcmp(value, "--no-tags"))
|
|
|
|
remote->fetch_tags = -1;
|
2007-12-04 05:48:54 +08:00
|
|
|
} else if (!strcmp(subkey, ".proxy")) {
|
|
|
|
remote->http_proxy = xstrdup(value);
|
2008-02-29 09:45:45 +08:00
|
|
|
} else if (!strcmp(subkey, ".skipdefaultupdate"))
|
|
|
|
remote->skip_default_update = 1;
|
2007-05-12 23:45:53 +08:00
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2008-02-21 02:43:53 +08:00
|
|
|
static void alias_all_urls(void)
|
|
|
|
{
|
|
|
|
int i, j;
|
|
|
|
for (i = 0; i < remotes_nr; i++) {
|
|
|
|
if (!remotes[i])
|
|
|
|
continue;
|
|
|
|
for (j = 0; j < remotes[i]->url_nr; j++) {
|
|
|
|
remotes[i]->url[j] = alias_url(remotes[i]->url[j]);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2007-05-12 23:45:53 +08:00
|
|
|
static void read_config(void)
|
|
|
|
{
|
|
|
|
unsigned char sha1[20];
|
|
|
|
const char *head_ref;
|
|
|
|
int flag;
|
|
|
|
if (default_remote_name) // did this already
|
|
|
|
return;
|
|
|
|
default_remote_name = xstrdup("origin");
|
|
|
|
current_branch = NULL;
|
|
|
|
head_ref = resolve_ref("HEAD", sha1, 0, &flag);
|
|
|
|
if (head_ref && (flag & REF_ISSYMREF) &&
|
|
|
|
!prefixcmp(head_ref, "refs/heads/")) {
|
2007-09-11 11:02:56 +08:00
|
|
|
current_branch =
|
|
|
|
make_branch(head_ref + strlen("refs/heads/"), 0);
|
2007-05-12 23:45:53 +08:00
|
|
|
}
|
|
|
|
git_config(handle_config);
|
2008-02-21 02:43:53 +08:00
|
|
|
alias_all_urls();
|
2007-05-12 23:45:53 +08:00
|
|
|
}
|
|
|
|
|
2008-04-13 17:56:54 +08:00
|
|
|
static struct refspec *parse_refspec_internal(int nr_refspec, const char **refspec, int fetch, int verify)
|
2007-05-12 23:45:59 +08:00
|
|
|
{
|
|
|
|
int i;
|
2008-03-18 10:05:23 +08:00
|
|
|
int st;
|
2007-05-12 23:45:59 +08:00
|
|
|
struct refspec *rs = xcalloc(sizeof(*rs), nr_refspec);
|
remote.c: Fix overtight refspec validation
We tightened the refspec validation code in an earlier commit ef00d15
(Tighten refspec processing, 2008-03-17) per my suggestion, but the
suggestion was misguided to begin with and it broke this usage:
$ git push origin HEAD~12:master
The syntax of push refspecs and fetch refspecs are similar in that they
are both colon separated LHS and RHS (possibly prefixed with a + to
force), but the similarity ends there. For example, LHS in a push refspec
can be anything that evaluates to a valid object name at runtime (except
when colon and RHS is missing, or it is a glob), while it must be a
valid-looking refname in a fetch refspec. To validate them correctly, the
caller needs to be able to say which kind of refspecs they are. It is
unreasonable to keep a single interface that cannot tell which kind it is
dealing with, and ask it to behave sensibly.
This commit separates the parsing of the two into different functions, and
clarifies the code to implement the parsing proper (i.e. splitting into
two parts, making sure both sides are wildcard or neither side is).
This happens to also allow pushing a commit named with the esoteric "look
for that string" syntax:
$ git push ../test.git ':/remote.c: Fix overtight refspec:master'
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-03-21 14:34:37 +08:00
|
|
|
|
2007-05-12 23:45:59 +08:00
|
|
|
for (i = 0; i < nr_refspec; i++) {
|
remote.c: Fix overtight refspec validation
We tightened the refspec validation code in an earlier commit ef00d15
(Tighten refspec processing, 2008-03-17) per my suggestion, but the
suggestion was misguided to begin with and it broke this usage:
$ git push origin HEAD~12:master
The syntax of push refspecs and fetch refspecs are similar in that they
are both colon separated LHS and RHS (possibly prefixed with a + to
force), but the similarity ends there. For example, LHS in a push refspec
can be anything that evaluates to a valid object name at runtime (except
when colon and RHS is missing, or it is a glob), while it must be a
valid-looking refname in a fetch refspec. To validate them correctly, the
caller needs to be able to say which kind of refspecs they are. It is
unreasonable to keep a single interface that cannot tell which kind it is
dealing with, and ask it to behave sensibly.
This commit separates the parsing of the two into different functions, and
clarifies the code to implement the parsing proper (i.e. splitting into
two parts, making sure both sides are wildcard or neither side is).
This happens to also allow pushing a commit named with the esoteric "look
for that string" syntax:
$ git push ../test.git ':/remote.c: Fix overtight refspec:master'
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-03-21 14:34:37 +08:00
|
|
|
size_t llen, rlen;
|
|
|
|
int is_glob;
|
|
|
|
const char *lhs, *rhs;
|
|
|
|
|
|
|
|
llen = rlen = is_glob = 0;
|
|
|
|
|
|
|
|
lhs = refspec[i];
|
|
|
|
if (*lhs == '+') {
|
2007-05-12 23:45:59 +08:00
|
|
|
rs[i].force = 1;
|
remote.c: Fix overtight refspec validation
We tightened the refspec validation code in an earlier commit ef00d15
(Tighten refspec processing, 2008-03-17) per my suggestion, but the
suggestion was misguided to begin with and it broke this usage:
$ git push origin HEAD~12:master
The syntax of push refspecs and fetch refspecs are similar in that they
are both colon separated LHS and RHS (possibly prefixed with a + to
force), but the similarity ends there. For example, LHS in a push refspec
can be anything that evaluates to a valid object name at runtime (except
when colon and RHS is missing, or it is a glob), while it must be a
valid-looking refname in a fetch refspec. To validate them correctly, the
caller needs to be able to say which kind of refspecs they are. It is
unreasonable to keep a single interface that cannot tell which kind it is
dealing with, and ask it to behave sensibly.
This commit separates the parsing of the two into different functions, and
clarifies the code to implement the parsing proper (i.e. splitting into
two parts, making sure both sides are wildcard or neither side is).
This happens to also allow pushing a commit named with the esoteric "look
for that string" syntax:
$ git push ../test.git ':/remote.c: Fix overtight refspec:master'
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-03-21 14:34:37 +08:00
|
|
|
lhs++;
|
2007-05-12 23:45:59 +08:00
|
|
|
}
|
remote.c: Fix overtight refspec validation
We tightened the refspec validation code in an earlier commit ef00d15
(Tighten refspec processing, 2008-03-17) per my suggestion, but the
suggestion was misguided to begin with and it broke this usage:
$ git push origin HEAD~12:master
The syntax of push refspecs and fetch refspecs are similar in that they
are both colon separated LHS and RHS (possibly prefixed with a + to
force), but the similarity ends there. For example, LHS in a push refspec
can be anything that evaluates to a valid object name at runtime (except
when colon and RHS is missing, or it is a glob), while it must be a
valid-looking refname in a fetch refspec. To validate them correctly, the
caller needs to be able to say which kind of refspecs they are. It is
unreasonable to keep a single interface that cannot tell which kind it is
dealing with, and ask it to behave sensibly.
This commit separates the parsing of the two into different functions, and
clarifies the code to implement the parsing proper (i.e. splitting into
two parts, making sure both sides are wildcard or neither side is).
This happens to also allow pushing a commit named with the esoteric "look
for that string" syntax:
$ git push ../test.git ':/remote.c: Fix overtight refspec:master'
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-03-21 14:34:37 +08:00
|
|
|
|
|
|
|
rhs = strrchr(lhs, ':');
|
|
|
|
if (rhs) {
|
|
|
|
rhs++;
|
|
|
|
rlen = strlen(rhs);
|
|
|
|
is_glob = (2 <= rlen && !strcmp(rhs + rlen - 2, "/*"));
|
2008-03-26 12:15:52 +08:00
|
|
|
if (is_glob)
|
|
|
|
rlen -= 2;
|
|
|
|
rs[i].dst = xstrndup(rhs, rlen);
|
2007-05-12 23:45:59 +08:00
|
|
|
}
|
2008-03-18 10:05:23 +08:00
|
|
|
|
remote.c: Fix overtight refspec validation
We tightened the refspec validation code in an earlier commit ef00d15
(Tighten refspec processing, 2008-03-17) per my suggestion, but the
suggestion was misguided to begin with and it broke this usage:
$ git push origin HEAD~12:master
The syntax of push refspecs and fetch refspecs are similar in that they
are both colon separated LHS and RHS (possibly prefixed with a + to
force), but the similarity ends there. For example, LHS in a push refspec
can be anything that evaluates to a valid object name at runtime (except
when colon and RHS is missing, or it is a glob), while it must be a
valid-looking refname in a fetch refspec. To validate them correctly, the
caller needs to be able to say which kind of refspecs they are. It is
unreasonable to keep a single interface that cannot tell which kind it is
dealing with, and ask it to behave sensibly.
This commit separates the parsing of the two into different functions, and
clarifies the code to implement the parsing proper (i.e. splitting into
two parts, making sure both sides are wildcard or neither side is).
This happens to also allow pushing a commit named with the esoteric "look
for that string" syntax:
$ git push ../test.git ':/remote.c: Fix overtight refspec:master'
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-03-21 14:34:37 +08:00
|
|
|
llen = (rhs ? (rhs - lhs - 1) : strlen(lhs));
|
2008-03-26 12:15:52 +08:00
|
|
|
if (2 <= llen && !memcmp(lhs + llen - 2, "/*", 2)) {
|
|
|
|
if ((rhs && !is_glob) || (!rhs && fetch))
|
|
|
|
goto invalid;
|
|
|
|
is_glob = 1;
|
remote.c: Fix overtight refspec validation
We tightened the refspec validation code in an earlier commit ef00d15
(Tighten refspec processing, 2008-03-17) per my suggestion, but the
suggestion was misguided to begin with and it broke this usage:
$ git push origin HEAD~12:master
The syntax of push refspecs and fetch refspecs are similar in that they
are both colon separated LHS and RHS (possibly prefixed with a + to
force), but the similarity ends there. For example, LHS in a push refspec
can be anything that evaluates to a valid object name at runtime (except
when colon and RHS is missing, or it is a glob), while it must be a
valid-looking refname in a fetch refspec. To validate them correctly, the
caller needs to be able to say which kind of refspecs they are. It is
unreasonable to keep a single interface that cannot tell which kind it is
dealing with, and ask it to behave sensibly.
This commit separates the parsing of the two into different functions, and
clarifies the code to implement the parsing proper (i.e. splitting into
two parts, making sure both sides are wildcard or neither side is).
This happens to also allow pushing a commit named with the esoteric "look
for that string" syntax:
$ git push ../test.git ':/remote.c: Fix overtight refspec:master'
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-03-21 14:34:37 +08:00
|
|
|
llen -= 2;
|
2008-03-26 12:15:52 +08:00
|
|
|
} else if (rhs && is_glob) {
|
|
|
|
goto invalid;
|
2008-03-18 10:05:23 +08:00
|
|
|
}
|
2008-03-26 12:15:52 +08:00
|
|
|
|
remote.c: Fix overtight refspec validation
We tightened the refspec validation code in an earlier commit ef00d15
(Tighten refspec processing, 2008-03-17) per my suggestion, but the
suggestion was misguided to begin with and it broke this usage:
$ git push origin HEAD~12:master
The syntax of push refspecs and fetch refspecs are similar in that they
are both colon separated LHS and RHS (possibly prefixed with a + to
force), but the similarity ends there. For example, LHS in a push refspec
can be anything that evaluates to a valid object name at runtime (except
when colon and RHS is missing, or it is a glob), while it must be a
valid-looking refname in a fetch refspec. To validate them correctly, the
caller needs to be able to say which kind of refspecs they are. It is
unreasonable to keep a single interface that cannot tell which kind it is
dealing with, and ask it to behave sensibly.
This commit separates the parsing of the two into different functions, and
clarifies the code to implement the parsing proper (i.e. splitting into
two parts, making sure both sides are wildcard or neither side is).
This happens to also allow pushing a commit named with the esoteric "look
for that string" syntax:
$ git push ../test.git ':/remote.c: Fix overtight refspec:master'
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-03-21 14:34:37 +08:00
|
|
|
rs[i].pattern = is_glob;
|
|
|
|
rs[i].src = xstrndup(lhs, llen);
|
|
|
|
|
|
|
|
if (fetch) {
|
|
|
|
/*
|
|
|
|
* LHS
|
|
|
|
* - empty is allowed; it means HEAD.
|
|
|
|
* - otherwise it must be a valid looking ref.
|
|
|
|
*/
|
|
|
|
if (!*rs[i].src)
|
|
|
|
; /* empty is ok */
|
|
|
|
else {
|
|
|
|
st = check_ref_format(rs[i].src);
|
|
|
|
if (st && st != CHECK_REF_FORMAT_ONELEVEL)
|
|
|
|
goto invalid;
|
|
|
|
}
|
|
|
|
/*
|
|
|
|
* RHS
|
2008-03-26 12:15:52 +08:00
|
|
|
* - missing is ok, and is same as empty.
|
remote.c: Fix overtight refspec validation
We tightened the refspec validation code in an earlier commit ef00d15
(Tighten refspec processing, 2008-03-17) per my suggestion, but the
suggestion was misguided to begin with and it broke this usage:
$ git push origin HEAD~12:master
The syntax of push refspecs and fetch refspecs are similar in that they
are both colon separated LHS and RHS (possibly prefixed with a + to
force), but the similarity ends there. For example, LHS in a push refspec
can be anything that evaluates to a valid object name at runtime (except
when colon and RHS is missing, or it is a glob), while it must be a
valid-looking refname in a fetch refspec. To validate them correctly, the
caller needs to be able to say which kind of refspecs they are. It is
unreasonable to keep a single interface that cannot tell which kind it is
dealing with, and ask it to behave sensibly.
This commit separates the parsing of the two into different functions, and
clarifies the code to implement the parsing proper (i.e. splitting into
two parts, making sure both sides are wildcard or neither side is).
This happens to also allow pushing a commit named with the esoteric "look
for that string" syntax:
$ git push ../test.git ':/remote.c: Fix overtight refspec:master'
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-03-21 14:34:37 +08:00
|
|
|
* - empty is ok; it means not to store.
|
|
|
|
* - otherwise it must be a valid looking ref.
|
|
|
|
*/
|
|
|
|
if (!rs[i].dst) {
|
|
|
|
; /* ok */
|
|
|
|
} else if (!*rs[i].dst) {
|
|
|
|
; /* ok */
|
|
|
|
} else {
|
|
|
|
st = check_ref_format(rs[i].dst);
|
|
|
|
if (st && st != CHECK_REF_FORMAT_ONELEVEL)
|
|
|
|
goto invalid;
|
|
|
|
}
|
|
|
|
} else {
|
|
|
|
/*
|
|
|
|
* LHS
|
|
|
|
* - empty is allowed; it means delete.
|
|
|
|
* - when wildcarded, it must be a valid looking ref.
|
|
|
|
* - otherwise, it must be an extended SHA-1, but
|
|
|
|
* there is no existing way to validate this.
|
|
|
|
*/
|
|
|
|
if (!*rs[i].src)
|
|
|
|
; /* empty is ok */
|
|
|
|
else if (is_glob) {
|
|
|
|
st = check_ref_format(rs[i].src);
|
|
|
|
if (st && st != CHECK_REF_FORMAT_ONELEVEL)
|
|
|
|
goto invalid;
|
|
|
|
}
|
|
|
|
else
|
|
|
|
; /* anything goes, for now */
|
|
|
|
/*
|
|
|
|
* RHS
|
|
|
|
* - missing is allowed, but LHS then must be a
|
|
|
|
* valid looking ref.
|
|
|
|
* - empty is not allowed.
|
|
|
|
* - otherwise it must be a valid looking ref.
|
|
|
|
*/
|
|
|
|
if (!rs[i].dst) {
|
|
|
|
st = check_ref_format(rs[i].src);
|
|
|
|
if (st && st != CHECK_REF_FORMAT_ONELEVEL)
|
|
|
|
goto invalid;
|
|
|
|
} else if (!*rs[i].dst) {
|
|
|
|
goto invalid;
|
|
|
|
} else {
|
|
|
|
st = check_ref_format(rs[i].dst);
|
|
|
|
if (st && st != CHECK_REF_FORMAT_ONELEVEL)
|
|
|
|
goto invalid;
|
|
|
|
}
|
2008-03-18 10:05:23 +08:00
|
|
|
}
|
2007-05-12 23:45:59 +08:00
|
|
|
}
|
|
|
|
return rs;
|
remote.c: Fix overtight refspec validation
We tightened the refspec validation code in an earlier commit ef00d15
(Tighten refspec processing, 2008-03-17) per my suggestion, but the
suggestion was misguided to begin with and it broke this usage:
$ git push origin HEAD~12:master
The syntax of push refspecs and fetch refspecs are similar in that they
are both colon separated LHS and RHS (possibly prefixed with a + to
force), but the similarity ends there. For example, LHS in a push refspec
can be anything that evaluates to a valid object name at runtime (except
when colon and RHS is missing, or it is a glob), while it must be a
valid-looking refname in a fetch refspec. To validate them correctly, the
caller needs to be able to say which kind of refspecs they are. It is
unreasonable to keep a single interface that cannot tell which kind it is
dealing with, and ask it to behave sensibly.
This commit separates the parsing of the two into different functions, and
clarifies the code to implement the parsing proper (i.e. splitting into
two parts, making sure both sides are wildcard or neither side is).
This happens to also allow pushing a commit named with the esoteric "look
for that string" syntax:
$ git push ../test.git ':/remote.c: Fix overtight refspec:master'
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-03-21 14:34:37 +08:00
|
|
|
|
|
|
|
invalid:
|
2008-04-13 17:56:54 +08:00
|
|
|
if (verify) {
|
|
|
|
free(rs);
|
|
|
|
return NULL;
|
|
|
|
}
|
remote.c: Fix overtight refspec validation
We tightened the refspec validation code in an earlier commit ef00d15
(Tighten refspec processing, 2008-03-17) per my suggestion, but the
suggestion was misguided to begin with and it broke this usage:
$ git push origin HEAD~12:master
The syntax of push refspecs and fetch refspecs are similar in that they
are both colon separated LHS and RHS (possibly prefixed with a + to
force), but the similarity ends there. For example, LHS in a push refspec
can be anything that evaluates to a valid object name at runtime (except
when colon and RHS is missing, or it is a glob), while it must be a
valid-looking refname in a fetch refspec. To validate them correctly, the
caller needs to be able to say which kind of refspecs they are. It is
unreasonable to keep a single interface that cannot tell which kind it is
dealing with, and ask it to behave sensibly.
This commit separates the parsing of the two into different functions, and
clarifies the code to implement the parsing proper (i.e. splitting into
two parts, making sure both sides are wildcard or neither side is).
This happens to also allow pushing a commit named with the esoteric "look
for that string" syntax:
$ git push ../test.git ':/remote.c: Fix overtight refspec:master'
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-03-21 14:34:37 +08:00
|
|
|
die("Invalid refspec '%s'", refspec[i]);
|
|
|
|
}
|
|
|
|
|
2008-04-13 17:56:54 +08:00
|
|
|
int valid_fetch_refspec(const char *fetch_refspec_str)
|
|
|
|
{
|
|
|
|
const char *fetch_refspec[] = { fetch_refspec_str };
|
|
|
|
struct refspec *refspec;
|
|
|
|
|
|
|
|
refspec = parse_refspec_internal(1, fetch_refspec, 1, 1);
|
|
|
|
if (refspec)
|
|
|
|
free(refspec);
|
|
|
|
return !!refspec;
|
|
|
|
}
|
|
|
|
|
remote.c: Fix overtight refspec validation
We tightened the refspec validation code in an earlier commit ef00d15
(Tighten refspec processing, 2008-03-17) per my suggestion, but the
suggestion was misguided to begin with and it broke this usage:
$ git push origin HEAD~12:master
The syntax of push refspecs and fetch refspecs are similar in that they
are both colon separated LHS and RHS (possibly prefixed with a + to
force), but the similarity ends there. For example, LHS in a push refspec
can be anything that evaluates to a valid object name at runtime (except
when colon and RHS is missing, or it is a glob), while it must be a
valid-looking refname in a fetch refspec. To validate them correctly, the
caller needs to be able to say which kind of refspecs they are. It is
unreasonable to keep a single interface that cannot tell which kind it is
dealing with, and ask it to behave sensibly.
This commit separates the parsing of the two into different functions, and
clarifies the code to implement the parsing proper (i.e. splitting into
two parts, making sure both sides are wildcard or neither side is).
This happens to also allow pushing a commit named with the esoteric "look
for that string" syntax:
$ git push ../test.git ':/remote.c: Fix overtight refspec:master'
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-03-21 14:34:37 +08:00
|
|
|
struct refspec *parse_fetch_refspec(int nr_refspec, const char **refspec)
|
|
|
|
{
|
2008-04-13 17:56:54 +08:00
|
|
|
return parse_refspec_internal(nr_refspec, refspec, 1, 0);
|
remote.c: Fix overtight refspec validation
We tightened the refspec validation code in an earlier commit ef00d15
(Tighten refspec processing, 2008-03-17) per my suggestion, but the
suggestion was misguided to begin with and it broke this usage:
$ git push origin HEAD~12:master
The syntax of push refspecs and fetch refspecs are similar in that they
are both colon separated LHS and RHS (possibly prefixed with a + to
force), but the similarity ends there. For example, LHS in a push refspec
can be anything that evaluates to a valid object name at runtime (except
when colon and RHS is missing, or it is a glob), while it must be a
valid-looking refname in a fetch refspec. To validate them correctly, the
caller needs to be able to say which kind of refspecs they are. It is
unreasonable to keep a single interface that cannot tell which kind it is
dealing with, and ask it to behave sensibly.
This commit separates the parsing of the two into different functions, and
clarifies the code to implement the parsing proper (i.e. splitting into
two parts, making sure both sides are wildcard or neither side is).
This happens to also allow pushing a commit named with the esoteric "look
for that string" syntax:
$ git push ../test.git ':/remote.c: Fix overtight refspec:master'
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-03-21 14:34:37 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
struct refspec *parse_push_refspec(int nr_refspec, const char **refspec)
|
|
|
|
{
|
2008-04-13 17:56:54 +08:00
|
|
|
return parse_refspec_internal(nr_refspec, refspec, 0, 0);
|
2007-05-12 23:45:59 +08:00
|
|
|
}
|
|
|
|
|
2008-02-16 03:14:18 +08:00
|
|
|
static int valid_remote_nick(const char *name)
|
|
|
|
{
|
|
|
|
if (!name[0] || /* not empty */
|
|
|
|
(name[0] == '.' && /* not "." */
|
|
|
|
(!name[1] || /* not ".." */
|
|
|
|
(name[1] == '.' && !name[2]))))
|
|
|
|
return 0;
|
|
|
|
return !strchr(name, '/'); /* no slash */
|
|
|
|
}
|
|
|
|
|
2007-05-12 23:45:53 +08:00
|
|
|
struct remote *remote_get(const char *name)
|
|
|
|
{
|
|
|
|
struct remote *ret;
|
|
|
|
|
|
|
|
read_config();
|
|
|
|
if (!name)
|
|
|
|
name = default_remote_name;
|
|
|
|
ret = make_remote(name, 0);
|
2008-02-16 03:14:18 +08:00
|
|
|
if (valid_remote_nick(name)) {
|
2007-09-19 12:49:27 +08:00
|
|
|
if (!ret->url)
|
2007-05-12 23:45:53 +08:00
|
|
|
read_remotes_file(ret);
|
2007-09-19 12:49:27 +08:00
|
|
|
if (!ret->url)
|
2007-05-12 23:45:53 +08:00
|
|
|
read_branches_file(ret);
|
|
|
|
}
|
2007-09-19 12:49:27 +08:00
|
|
|
if (!ret->url)
|
2008-02-21 02:43:53 +08:00
|
|
|
add_url_alias(ret, name);
|
2007-09-19 12:49:27 +08:00
|
|
|
if (!ret->url)
|
2007-05-12 23:45:53 +08:00
|
|
|
return NULL;
|
remote.c: Fix overtight refspec validation
We tightened the refspec validation code in an earlier commit ef00d15
(Tighten refspec processing, 2008-03-17) per my suggestion, but the
suggestion was misguided to begin with and it broke this usage:
$ git push origin HEAD~12:master
The syntax of push refspecs and fetch refspecs are similar in that they
are both colon separated LHS and RHS (possibly prefixed with a + to
force), but the similarity ends there. For example, LHS in a push refspec
can be anything that evaluates to a valid object name at runtime (except
when colon and RHS is missing, or it is a glob), while it must be a
valid-looking refname in a fetch refspec. To validate them correctly, the
caller needs to be able to say which kind of refspecs they are. It is
unreasonable to keep a single interface that cannot tell which kind it is
dealing with, and ask it to behave sensibly.
This commit separates the parsing of the two into different functions, and
clarifies the code to implement the parsing proper (i.e. splitting into
two parts, making sure both sides are wildcard or neither side is).
This happens to also allow pushing a commit named with the esoteric "look
for that string" syntax:
$ git push ../test.git ':/remote.c: Fix overtight refspec:master'
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-03-21 14:34:37 +08:00
|
|
|
ret->fetch = parse_fetch_refspec(ret->fetch_refspec_nr, ret->fetch_refspec);
|
|
|
|
ret->push = parse_push_refspec(ret->push_refspec_nr, ret->push_refspec);
|
2007-05-12 23:45:53 +08:00
|
|
|
return ret;
|
|
|
|
}
|
2007-05-12 23:45:59 +08:00
|
|
|
|
2007-07-11 01:48:40 +08:00
|
|
|
int for_each_remote(each_remote_fn fn, void *priv)
|
|
|
|
{
|
|
|
|
int i, result = 0;
|
|
|
|
read_config();
|
2008-02-19 12:41:41 +08:00
|
|
|
for (i = 0; i < remotes_nr && !result; i++) {
|
2007-07-11 01:48:40 +08:00
|
|
|
struct remote *r = remotes[i];
|
|
|
|
if (!r)
|
|
|
|
continue;
|
|
|
|
if (!r->fetch)
|
remote.c: Fix overtight refspec validation
We tightened the refspec validation code in an earlier commit ef00d15
(Tighten refspec processing, 2008-03-17) per my suggestion, but the
suggestion was misguided to begin with and it broke this usage:
$ git push origin HEAD~12:master
The syntax of push refspecs and fetch refspecs are similar in that they
are both colon separated LHS and RHS (possibly prefixed with a + to
force), but the similarity ends there. For example, LHS in a push refspec
can be anything that evaluates to a valid object name at runtime (except
when colon and RHS is missing, or it is a glob), while it must be a
valid-looking refname in a fetch refspec. To validate them correctly, the
caller needs to be able to say which kind of refspecs they are. It is
unreasonable to keep a single interface that cannot tell which kind it is
dealing with, and ask it to behave sensibly.
This commit separates the parsing of the two into different functions, and
clarifies the code to implement the parsing proper (i.e. splitting into
two parts, making sure both sides are wildcard or neither side is).
This happens to also allow pushing a commit named with the esoteric "look
for that string" syntax:
$ git push ../test.git ':/remote.c: Fix overtight refspec:master'
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-03-21 14:34:37 +08:00
|
|
|
r->fetch = parse_fetch_refspec(r->fetch_refspec_nr,
|
|
|
|
r->fetch_refspec);
|
2007-07-11 01:48:40 +08:00
|
|
|
if (!r->push)
|
remote.c: Fix overtight refspec validation
We tightened the refspec validation code in an earlier commit ef00d15
(Tighten refspec processing, 2008-03-17) per my suggestion, but the
suggestion was misguided to begin with and it broke this usage:
$ git push origin HEAD~12:master
The syntax of push refspecs and fetch refspecs are similar in that they
are both colon separated LHS and RHS (possibly prefixed with a + to
force), but the similarity ends there. For example, LHS in a push refspec
can be anything that evaluates to a valid object name at runtime (except
when colon and RHS is missing, or it is a glob), while it must be a
valid-looking refname in a fetch refspec. To validate them correctly, the
caller needs to be able to say which kind of refspecs they are. It is
unreasonable to keep a single interface that cannot tell which kind it is
dealing with, and ask it to behave sensibly.
This commit separates the parsing of the two into different functions, and
clarifies the code to implement the parsing proper (i.e. splitting into
two parts, making sure both sides are wildcard or neither side is).
This happens to also allow pushing a commit named with the esoteric "look
for that string" syntax:
$ git push ../test.git ':/remote.c: Fix overtight refspec:master'
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-03-21 14:34:37 +08:00
|
|
|
r->push = parse_push_refspec(r->push_refspec_nr,
|
|
|
|
r->push_refspec);
|
2007-07-11 01:48:40 +08:00
|
|
|
result = fn(r, priv);
|
|
|
|
}
|
|
|
|
return result;
|
|
|
|
}
|
|
|
|
|
2007-10-08 12:25:07 +08:00
|
|
|
void ref_remove_duplicates(struct ref *ref_map)
|
|
|
|
{
|
|
|
|
struct ref **posn;
|
|
|
|
struct ref *next;
|
|
|
|
for (; ref_map; ref_map = ref_map->next) {
|
|
|
|
if (!ref_map->peer_ref)
|
|
|
|
continue;
|
|
|
|
posn = &ref_map->next;
|
|
|
|
while (*posn) {
|
|
|
|
if ((*posn)->peer_ref &&
|
|
|
|
!strcmp((*posn)->peer_ref->name,
|
|
|
|
ref_map->peer_ref->name)) {
|
|
|
|
if (strcmp((*posn)->name, ref_map->name))
|
|
|
|
die("%s tracks both %s and %s",
|
|
|
|
ref_map->peer_ref->name,
|
|
|
|
(*posn)->name, ref_map->name);
|
|
|
|
next = (*posn)->next;
|
|
|
|
free((*posn)->peer_ref);
|
|
|
|
free(*posn);
|
|
|
|
*posn = next;
|
|
|
|
} else {
|
|
|
|
posn = &(*posn)->next;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2007-09-19 12:49:27 +08:00
|
|
|
int remote_has_url(struct remote *remote, const char *url)
|
2007-05-12 23:46:03 +08:00
|
|
|
{
|
|
|
|
int i;
|
2007-09-19 12:49:27 +08:00
|
|
|
for (i = 0; i < remote->url_nr; i++) {
|
|
|
|
if (!strcmp(remote->url[i], url))
|
2007-05-12 23:46:03 +08:00
|
|
|
return 1;
|
|
|
|
}
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
int remote_find_tracking(struct remote *remote, struct refspec *refspec)
|
|
|
|
{
|
2007-07-11 01:48:40 +08:00
|
|
|
int find_src = refspec->src == NULL;
|
|
|
|
char *needle, **result;
|
2007-05-12 23:46:03 +08:00
|
|
|
int i;
|
2007-07-11 01:48:40 +08:00
|
|
|
|
|
|
|
if (find_src) {
|
2007-09-25 12:13:14 +08:00
|
|
|
if (!refspec->dst)
|
2007-07-11 01:48:40 +08:00
|
|
|
return error("find_tracking: need either src or dst");
|
|
|
|
needle = refspec->dst;
|
|
|
|
result = &refspec->src;
|
|
|
|
} else {
|
|
|
|
needle = refspec->src;
|
|
|
|
result = &refspec->dst;
|
|
|
|
}
|
|
|
|
|
2007-05-12 23:46:03 +08:00
|
|
|
for (i = 0; i < remote->fetch_refspec_nr; i++) {
|
|
|
|
struct refspec *fetch = &remote->fetch[i];
|
2007-07-11 01:48:40 +08:00
|
|
|
const char *key = find_src ? fetch->dst : fetch->src;
|
|
|
|
const char *value = find_src ? fetch->src : fetch->dst;
|
2007-05-12 23:46:03 +08:00
|
|
|
if (!fetch->dst)
|
|
|
|
continue;
|
|
|
|
if (fetch->pattern) {
|
2008-03-18 10:05:23 +08:00
|
|
|
if (!prefixcmp(needle, key) &&
|
|
|
|
needle[strlen(key)] == '/') {
|
2007-07-11 01:48:40 +08:00
|
|
|
*result = xmalloc(strlen(value) +
|
|
|
|
strlen(needle) -
|
|
|
|
strlen(key) + 1);
|
|
|
|
strcpy(*result, value);
|
|
|
|
strcpy(*result + strlen(value),
|
|
|
|
needle + strlen(key));
|
2007-05-12 23:46:03 +08:00
|
|
|
refspec->force = fetch->force;
|
|
|
|
return 0;
|
|
|
|
}
|
2007-07-11 01:48:40 +08:00
|
|
|
} else if (!strcmp(needle, key)) {
|
|
|
|
*result = xstrdup(value);
|
|
|
|
refspec->force = fetch->force;
|
|
|
|
return 0;
|
2007-05-12 23:46:03 +08:00
|
|
|
}
|
|
|
|
}
|
|
|
|
return -1;
|
|
|
|
}
|
|
|
|
|
2007-07-10 12:47:23 +08:00
|
|
|
struct ref *alloc_ref(unsigned namelen)
|
|
|
|
{
|
|
|
|
struct ref *ret = xmalloc(sizeof(struct ref) + namelen);
|
|
|
|
memset(ret, 0, sizeof(struct ref) + namelen);
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
2007-10-30 09:05:40 +08:00
|
|
|
static struct ref *copy_ref(const struct ref *ref)
|
2007-09-11 11:03:08 +08:00
|
|
|
{
|
|
|
|
struct ref *ret = xmalloc(sizeof(struct ref) + strlen(ref->name) + 1);
|
|
|
|
memcpy(ret, ref, sizeof(struct ref) + strlen(ref->name) + 1);
|
|
|
|
ret->next = NULL;
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
2007-10-30 09:05:40 +08:00
|
|
|
struct ref *copy_ref_list(const struct ref *ref)
|
|
|
|
{
|
|
|
|
struct ref *ret = NULL;
|
|
|
|
struct ref **tail = &ret;
|
|
|
|
while (ref) {
|
|
|
|
*tail = copy_ref(ref);
|
|
|
|
ref = ref->next;
|
|
|
|
tail = &((*tail)->next);
|
|
|
|
}
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
2007-07-10 12:47:23 +08:00
|
|
|
void free_refs(struct ref *ref)
|
|
|
|
{
|
|
|
|
struct ref *next;
|
|
|
|
while (ref) {
|
|
|
|
next = ref->next;
|
Avoid unnecessary "if-before-free" tests.
This change removes all obvious useless if-before-free tests.
E.g., it replaces code like this:
if (some_expression)
free (some_expression);
with the now-equivalent:
free (some_expression);
It is equivalent not just because POSIX has required free(NULL)
to work for a long time, but simply because it has worked for
so long that no reasonable porting target fails the test.
Here's some evidence from nearly 1.5 years ago:
http://www.winehq.org/pipermail/wine-patches/2006-October/031544.html
FYI, the change below was prepared by running the following:
git ls-files -z | xargs -0 \
perl -0x3b -pi -e \
's/\bif\s*\(\s*(\S+?)(?:\s*!=\s*NULL)?\s*\)\s+(free\s*\(\s*\1\s*\))/$2/s'
Note however, that it doesn't handle brace-enclosed blocks like
"if (x) { free (x); }". But that's ok, since there were none like
that in git sources.
Beware: if you do use the above snippet, note that it can
produce syntactically invalid C code. That happens when the
affected "if"-statement has a matching "else".
E.g., it would transform this
if (x)
free (x);
else
foo ();
into this:
free (x);
else
foo ();
There were none of those here, either.
If you're interested in automating detection of the useless
tests, you might like the useless-if-before-free script in gnulib:
[it *does* detect brace-enclosed free statements, and has a --name=S
option to make it detect free-like functions with different names]
http://git.sv.gnu.org/gitweb/?p=gnulib.git;a=blob;f=build-aux/useless-if-before-free
Addendum:
Remove one more (in imap-send.c), spotted by Jean-Luc Herren <jlh@gmx.ch>.
Signed-off-by: Jim Meyering <meyering@redhat.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-02-01 01:26:32 +08:00
|
|
|
free(ref->peer_ref);
|
2007-07-10 12:47:23 +08:00
|
|
|
free(ref);
|
|
|
|
ref = next;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2007-05-12 23:45:59 +08:00
|
|
|
static int count_refspec_match(const char *pattern,
|
|
|
|
struct ref *refs,
|
|
|
|
struct ref **matched_ref)
|
|
|
|
{
|
|
|
|
int patlen = strlen(pattern);
|
|
|
|
struct ref *matched_weak = NULL;
|
|
|
|
struct ref *matched = NULL;
|
|
|
|
int weak_match = 0;
|
|
|
|
int match = 0;
|
|
|
|
|
|
|
|
for (weak_match = match = 0; refs; refs = refs->next) {
|
|
|
|
char *name = refs->name;
|
|
|
|
int namelen = strlen(name);
|
|
|
|
|
2007-11-11 22:01:47 +08:00
|
|
|
if (!refname_match(pattern, name, ref_rev_parse_rules))
|
2007-05-12 23:45:59 +08:00
|
|
|
continue;
|
|
|
|
|
|
|
|
/* A match is "weak" if it is with refs outside
|
|
|
|
* heads or tags, and did not specify the pattern
|
|
|
|
* in full (e.g. "refs/remotes/origin/master") or at
|
|
|
|
* least from the toplevel (e.g. "remotes/origin/master");
|
|
|
|
* otherwise "git push $URL master" would result in
|
|
|
|
* ambiguity between remotes/origin/master and heads/master
|
|
|
|
* at the remote site.
|
|
|
|
*/
|
|
|
|
if (namelen != patlen &&
|
|
|
|
patlen != namelen - 5 &&
|
|
|
|
prefixcmp(name, "refs/heads/") &&
|
|
|
|
prefixcmp(name, "refs/tags/")) {
|
|
|
|
/* We want to catch the case where only weak
|
|
|
|
* matches are found and there are multiple
|
|
|
|
* matches, and where more than one strong
|
|
|
|
* matches are found, as ambiguous. One
|
|
|
|
* strong match with zero or more weak matches
|
|
|
|
* are acceptable as a unique match.
|
|
|
|
*/
|
|
|
|
matched_weak = refs;
|
|
|
|
weak_match++;
|
|
|
|
}
|
|
|
|
else {
|
|
|
|
matched = refs;
|
|
|
|
match++;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
if (!matched) {
|
|
|
|
*matched_ref = matched_weak;
|
|
|
|
return weak_match;
|
|
|
|
}
|
|
|
|
else {
|
|
|
|
*matched_ref = matched;
|
|
|
|
return match;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2007-07-10 12:47:26 +08:00
|
|
|
static void tail_link_ref(struct ref *ref, struct ref ***tail)
|
2007-05-12 23:45:59 +08:00
|
|
|
{
|
|
|
|
**tail = ref;
|
2007-07-10 12:47:26 +08:00
|
|
|
while (ref->next)
|
|
|
|
ref = ref->next;
|
2007-05-12 23:45:59 +08:00
|
|
|
*tail = &ref->next;
|
|
|
|
}
|
|
|
|
|
|
|
|
static struct ref *try_explicit_object_name(const char *name)
|
|
|
|
{
|
|
|
|
unsigned char sha1[20];
|
|
|
|
struct ref *ref;
|
|
|
|
int len;
|
|
|
|
|
|
|
|
if (!*name) {
|
2007-07-10 12:47:23 +08:00
|
|
|
ref = alloc_ref(20);
|
2007-05-12 23:45:59 +08:00
|
|
|
strcpy(ref->name, "(delete)");
|
|
|
|
hashclr(ref->new_sha1);
|
|
|
|
return ref;
|
|
|
|
}
|
|
|
|
if (get_sha1(name, sha1))
|
|
|
|
return NULL;
|
|
|
|
len = strlen(name) + 1;
|
2007-07-10 12:47:23 +08:00
|
|
|
ref = alloc_ref(len);
|
2007-05-12 23:45:59 +08:00
|
|
|
memcpy(ref->name, name, len);
|
|
|
|
hashcpy(ref->new_sha1, sha1);
|
|
|
|
return ref;
|
|
|
|
}
|
|
|
|
|
2007-07-10 12:47:26 +08:00
|
|
|
static struct ref *make_linked_ref(const char *name, struct ref ***tail)
|
2007-05-12 23:45:59 +08:00
|
|
|
{
|
2007-07-10 12:47:26 +08:00
|
|
|
struct ref *ret;
|
2007-06-09 15:07:34 +08:00
|
|
|
size_t len;
|
2007-05-12 23:45:59 +08:00
|
|
|
|
2007-06-09 15:07:34 +08:00
|
|
|
len = strlen(name) + 1;
|
2007-07-10 12:47:26 +08:00
|
|
|
ret = alloc_ref(len);
|
|
|
|
memcpy(ret->name, name, len);
|
|
|
|
tail_link_ref(ret, tail);
|
|
|
|
return ret;
|
2007-06-09 15:07:34 +08:00
|
|
|
}
|
2007-05-25 13:20:56 +08:00
|
|
|
|
push: allow unqualified dest refspecs to DWIM
Previously, a push like:
git push remote src:dst
would go through the following steps:
1. check for an unambiguous 'dst' on the remote; if it
exists, then push to that ref
2. otherwise, check if 'dst' begins with 'refs/'; if it
does, create a new ref
3. otherwise, complain because we don't know where in the
refs hierarchy to put 'dst'
However, in some cases, we can guess about the ref type of
'dst' based on the ref type of 'src'. Specifically, before
complaining we now check:
2.5. if 'src' resolves to a ref starting with refs/heads
or refs/tags, then prepend that to 'dst'
So now this creates a new branch on the remote, whereas it
previously failed with an error message:
git push master:newbranch
Note that, by design, we limit this DWIM behavior only to
source refs which resolve exactly (including symrefs which
resolve to existing refs). We still complain on a partial
destination refspec if the source is a raw sha1, or a ref
expression such as 'master~10'.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-04-23 17:16:06 +08:00
|
|
|
static char *guess_ref(const char *name, struct ref *peer)
|
|
|
|
{
|
|
|
|
struct strbuf buf = STRBUF_INIT;
|
|
|
|
unsigned char sha1[20];
|
|
|
|
|
|
|
|
const char *r = resolve_ref(peer->name, sha1, 1, NULL);
|
|
|
|
if (!r)
|
|
|
|
return NULL;
|
|
|
|
|
|
|
|
if (!prefixcmp(r, "refs/heads/"))
|
|
|
|
strbuf_addstr(&buf, "refs/heads/");
|
|
|
|
else if (!prefixcmp(r, "refs/tags/"))
|
|
|
|
strbuf_addstr(&buf, "refs/tags/");
|
|
|
|
else
|
|
|
|
return NULL;
|
|
|
|
|
|
|
|
strbuf_addstr(&buf, name);
|
|
|
|
return strbuf_detach(&buf, NULL);
|
|
|
|
}
|
|
|
|
|
2007-06-09 14:22:58 +08:00
|
|
|
static int match_explicit(struct ref *src, struct ref *dst,
|
|
|
|
struct ref ***dst_tail,
|
|
|
|
struct refspec *rs,
|
|
|
|
int errs)
|
2007-05-12 23:45:59 +08:00
|
|
|
{
|
2007-06-09 14:22:58 +08:00
|
|
|
struct ref *matched_src, *matched_dst;
|
2007-05-25 13:20:56 +08:00
|
|
|
|
2007-06-09 14:22:58 +08:00
|
|
|
const char *dst_value = rs->dst;
|
push: allow unqualified dest refspecs to DWIM
Previously, a push like:
git push remote src:dst
would go through the following steps:
1. check for an unambiguous 'dst' on the remote; if it
exists, then push to that ref
2. otherwise, check if 'dst' begins with 'refs/'; if it
does, create a new ref
3. otherwise, complain because we don't know where in the
refs hierarchy to put 'dst'
However, in some cases, we can guess about the ref type of
'dst' based on the ref type of 'src'. Specifically, before
complaining we now check:
2.5. if 'src' resolves to a ref starting with refs/heads
or refs/tags, then prepend that to 'dst'
So now this creates a new branch on the remote, whereas it
previously failed with an error message:
git push master:newbranch
Note that, by design, we limit this DWIM behavior only to
source refs which resolve exactly (including symrefs which
resolve to existing refs). We still complain on a partial
destination refspec if the source is a raw sha1, or a ref
expression such as 'master~10'.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-04-23 17:16:06 +08:00
|
|
|
char *dst_guess;
|
2007-05-12 23:45:59 +08:00
|
|
|
|
2007-06-09 14:22:58 +08:00
|
|
|
if (rs->pattern)
|
|
|
|
return errs;
|
2007-05-25 13:20:56 +08:00
|
|
|
|
2007-06-09 14:22:58 +08:00
|
|
|
matched_src = matched_dst = NULL;
|
|
|
|
switch (count_refspec_match(rs->src, src, &matched_src)) {
|
|
|
|
case 1:
|
|
|
|
break;
|
|
|
|
case 0:
|
|
|
|
/* The source could be in the get_sha1() format
|
|
|
|
* not a reference name. :refs/other is a
|
|
|
|
* way to delete 'other' ref at the remote end.
|
|
|
|
*/
|
|
|
|
matched_src = try_explicit_object_name(rs->src);
|
2007-09-25 12:13:19 +08:00
|
|
|
if (!matched_src)
|
|
|
|
error("src refspec %s does not match any.", rs->src);
|
2007-06-09 14:22:58 +08:00
|
|
|
break;
|
|
|
|
default:
|
2007-06-09 15:14:04 +08:00
|
|
|
matched_src = NULL;
|
2007-09-25 12:13:19 +08:00
|
|
|
error("src refspec %s matches more than one.", rs->src);
|
2007-06-09 14:22:58 +08:00
|
|
|
break;
|
|
|
|
}
|
2007-06-09 15:14:04 +08:00
|
|
|
|
|
|
|
if (!matched_src)
|
|
|
|
errs = 1;
|
|
|
|
|
2007-09-25 12:13:25 +08:00
|
|
|
if (!dst_value) {
|
2008-02-21 01:54:05 +08:00
|
|
|
unsigned char sha1[20];
|
|
|
|
int flag;
|
|
|
|
|
2007-09-25 12:13:25 +08:00
|
|
|
if (!matched_src)
|
|
|
|
return errs;
|
2008-02-21 01:54:05 +08:00
|
|
|
dst_value = resolve_ref(matched_src->name, sha1, 1, &flag);
|
|
|
|
if (!dst_value ||
|
|
|
|
((flag & REF_ISSYMREF) &&
|
|
|
|
prefixcmp(dst_value, "refs/heads/")))
|
|
|
|
die("%s cannot be resolved to branch.",
|
|
|
|
matched_src->name);
|
2007-09-25 12:13:25 +08:00
|
|
|
}
|
2007-06-09 15:14:04 +08:00
|
|
|
|
2007-06-09 14:22:58 +08:00
|
|
|
switch (count_refspec_match(dst_value, dst, &matched_dst)) {
|
|
|
|
case 1:
|
|
|
|
break;
|
|
|
|
case 0:
|
2007-06-09 15:07:34 +08:00
|
|
|
if (!memcmp(dst_value, "refs/", 5))
|
2007-07-10 12:47:26 +08:00
|
|
|
matched_dst = make_linked_ref(dst_value, dst_tail);
|
push: allow unqualified dest refspecs to DWIM
Previously, a push like:
git push remote src:dst
would go through the following steps:
1. check for an unambiguous 'dst' on the remote; if it
exists, then push to that ref
2. otherwise, check if 'dst' begins with 'refs/'; if it
does, create a new ref
3. otherwise, complain because we don't know where in the
refs hierarchy to put 'dst'
However, in some cases, we can guess about the ref type of
'dst' based on the ref type of 'src'. Specifically, before
complaining we now check:
2.5. if 'src' resolves to a ref starting with refs/heads
or refs/tags, then prepend that to 'dst'
So now this creates a new branch on the remote, whereas it
previously failed with an error message:
git push master:newbranch
Note that, by design, we limit this DWIM behavior only to
source refs which resolve exactly (including symrefs which
resolve to existing refs). We still complain on a partial
destination refspec if the source is a raw sha1, or a ref
expression such as 'master~10'.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-04-23 17:16:06 +08:00
|
|
|
else if((dst_guess = guess_ref(dst_value, matched_src)))
|
|
|
|
matched_dst = make_linked_ref(dst_guess, dst_tail);
|
2007-06-09 15:14:04 +08:00
|
|
|
else
|
push: allow unqualified dest refspecs to DWIM
Previously, a push like:
git push remote src:dst
would go through the following steps:
1. check for an unambiguous 'dst' on the remote; if it
exists, then push to that ref
2. otherwise, check if 'dst' begins with 'refs/'; if it
does, create a new ref
3. otherwise, complain because we don't know where in the
refs hierarchy to put 'dst'
However, in some cases, we can guess about the ref type of
'dst' based on the ref type of 'src'. Specifically, before
complaining we now check:
2.5. if 'src' resolves to a ref starting with refs/heads
or refs/tags, then prepend that to 'dst'
So now this creates a new branch on the remote, whereas it
previously failed with an error message:
git push master:newbranch
Note that, by design, we limit this DWIM behavior only to
source refs which resolve exactly (including symrefs which
resolve to existing refs). We still complain on a partial
destination refspec if the source is a raw sha1, or a ref
expression such as 'master~10'.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-04-23 17:16:06 +08:00
|
|
|
error("unable to push to unqualified destination: %s\n"
|
|
|
|
"The destination refspec neither matches an "
|
|
|
|
"existing ref on the remote nor\n"
|
|
|
|
"begins with refs/, and we are unable to "
|
|
|
|
"guess a prefix based on the source ref.",
|
|
|
|
dst_value);
|
2007-06-09 14:22:58 +08:00
|
|
|
break;
|
|
|
|
default:
|
2007-06-09 15:14:04 +08:00
|
|
|
matched_dst = NULL;
|
2007-06-09 14:22:58 +08:00
|
|
|
error("dst refspec %s matches more than one.",
|
|
|
|
dst_value);
|
|
|
|
break;
|
|
|
|
}
|
2007-09-25 12:13:14 +08:00
|
|
|
if (errs || !matched_dst)
|
2007-06-09 15:14:04 +08:00
|
|
|
return 1;
|
2007-06-09 14:22:58 +08:00
|
|
|
if (matched_dst->peer_ref) {
|
|
|
|
errs = 1;
|
|
|
|
error("dst ref %s receives from more than one src.",
|
|
|
|
matched_dst->name);
|
2007-05-12 23:45:59 +08:00
|
|
|
}
|
2007-06-09 14:22:58 +08:00
|
|
|
else {
|
|
|
|
matched_dst->peer_ref = matched_src;
|
|
|
|
matched_dst->force = rs->force;
|
2007-05-12 23:45:59 +08:00
|
|
|
}
|
2007-06-09 14:22:58 +08:00
|
|
|
return errs;
|
|
|
|
}
|
|
|
|
|
|
|
|
static int match_explicit_refs(struct ref *src, struct ref *dst,
|
|
|
|
struct ref ***dst_tail, struct refspec *rs,
|
|
|
|
int rs_nr)
|
|
|
|
{
|
|
|
|
int i, errs;
|
|
|
|
for (i = errs = 0; i < rs_nr; i++)
|
|
|
|
errs |= match_explicit(src, dst, dst_tail, &rs[i], errs);
|
2007-05-12 23:45:59 +08:00
|
|
|
return -errs;
|
|
|
|
}
|
|
|
|
|
2007-06-08 07:43:05 +08:00
|
|
|
static const struct refspec *check_pattern_match(const struct refspec *rs,
|
|
|
|
int rs_nr,
|
|
|
|
const struct ref *src)
|
2007-05-25 13:20:56 +08:00
|
|
|
{
|
|
|
|
int i;
|
|
|
|
for (i = 0; i < rs_nr; i++) {
|
2008-03-18 10:05:23 +08:00
|
|
|
if (rs[i].pattern &&
|
|
|
|
!prefixcmp(src->name, rs[i].src) &&
|
|
|
|
src->name[strlen(rs[i].src)] == '/')
|
2007-06-08 07:43:05 +08:00
|
|
|
return rs + i;
|
2007-05-25 13:20:56 +08:00
|
|
|
}
|
2007-06-08 07:43:05 +08:00
|
|
|
return NULL;
|
2007-05-25 13:20:56 +08:00
|
|
|
}
|
|
|
|
|
2007-06-09 14:22:58 +08:00
|
|
|
/*
|
|
|
|
* Note. This is used only by "push"; refspec matching rules for
|
|
|
|
* push and fetch are subtly different, so do not try to reuse it
|
|
|
|
* without thinking.
|
|
|
|
*/
|
2007-05-12 23:45:59 +08:00
|
|
|
int match_refs(struct ref *src, struct ref *dst, struct ref ***dst_tail,
|
2007-11-10 07:32:10 +08:00
|
|
|
int nr_refspec, const char **refspec, int flags)
|
2007-05-12 23:45:59 +08:00
|
|
|
{
|
|
|
|
struct refspec *rs =
|
remote.c: Fix overtight refspec validation
We tightened the refspec validation code in an earlier commit ef00d15
(Tighten refspec processing, 2008-03-17) per my suggestion, but the
suggestion was misguided to begin with and it broke this usage:
$ git push origin HEAD~12:master
The syntax of push refspecs and fetch refspecs are similar in that they
are both colon separated LHS and RHS (possibly prefixed with a + to
force), but the similarity ends there. For example, LHS in a push refspec
can be anything that evaluates to a valid object name at runtime (except
when colon and RHS is missing, or it is a glob), while it must be a
valid-looking refname in a fetch refspec. To validate them correctly, the
caller needs to be able to say which kind of refspecs they are. It is
unreasonable to keep a single interface that cannot tell which kind it is
dealing with, and ask it to behave sensibly.
This commit separates the parsing of the two into different functions, and
clarifies the code to implement the parsing proper (i.e. splitting into
two parts, making sure both sides are wildcard or neither side is).
This happens to also allow pushing a commit named with the esoteric "look
for that string" syntax:
$ git push ../test.git ':/remote.c: Fix overtight refspec:master'
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-03-21 14:34:37 +08:00
|
|
|
parse_push_refspec(nr_refspec, (const char **) refspec);
|
2007-11-10 07:32:10 +08:00
|
|
|
int send_all = flags & MATCH_REFS_ALL;
|
|
|
|
int send_mirror = flags & MATCH_REFS_MIRROR;
|
2007-05-12 23:45:59 +08:00
|
|
|
|
2007-05-25 13:20:56 +08:00
|
|
|
if (match_explicit_refs(src, dst, dst_tail, rs, nr_refspec))
|
|
|
|
return -1;
|
2007-05-12 23:45:59 +08:00
|
|
|
|
|
|
|
/* pick the remainder */
|
|
|
|
for ( ; src; src = src->next) {
|
|
|
|
struct ref *dst_peer;
|
2007-06-08 07:43:05 +08:00
|
|
|
const struct refspec *pat = NULL;
|
|
|
|
char *dst_name;
|
2007-05-12 23:45:59 +08:00
|
|
|
if (src->peer_ref)
|
|
|
|
continue;
|
2007-06-08 07:43:05 +08:00
|
|
|
if (nr_refspec) {
|
|
|
|
pat = check_pattern_match(rs, nr_refspec, src);
|
|
|
|
if (!pat)
|
|
|
|
continue;
|
|
|
|
}
|
2007-11-10 07:32:10 +08:00
|
|
|
else if (!send_mirror && prefixcmp(src->name, "refs/heads/"))
|
2007-07-02 10:00:08 +08:00
|
|
|
/*
|
|
|
|
* "matching refs"; traditionally we pushed everything
|
|
|
|
* including refs outside refs/heads/ hierarchy, but
|
|
|
|
* that does not make much sense these days.
|
|
|
|
*/
|
|
|
|
continue;
|
2007-05-25 13:20:56 +08:00
|
|
|
|
2007-06-08 07:43:05 +08:00
|
|
|
if (pat) {
|
2007-06-15 22:22:37 +08:00
|
|
|
const char *dst_side = pat->dst ? pat->dst : pat->src;
|
|
|
|
dst_name = xmalloc(strlen(dst_side) +
|
2007-06-08 07:43:05 +08:00
|
|
|
strlen(src->name) -
|
|
|
|
strlen(pat->src) + 2);
|
2007-06-15 22:22:37 +08:00
|
|
|
strcpy(dst_name, dst_side);
|
2007-06-08 07:43:05 +08:00
|
|
|
strcat(dst_name, src->name + strlen(pat->src));
|
|
|
|
} else
|
2007-06-13 04:59:21 +08:00
|
|
|
dst_name = xstrdup(src->name);
|
2007-06-08 07:43:05 +08:00
|
|
|
dst_peer = find_ref_by_name(dst, dst_name);
|
2007-05-25 13:20:56 +08:00
|
|
|
if (dst_peer && dst_peer->peer_ref)
|
|
|
|
/* We're already sending something to this ref. */
|
2007-06-08 07:43:05 +08:00
|
|
|
goto free_name;
|
2007-11-10 07:32:10 +08:00
|
|
|
|
|
|
|
if (!dst_peer && !nr_refspec && !(send_all || send_mirror))
|
|
|
|
/*
|
|
|
|
* Remote doesn't have it, and we have no
|
2007-05-25 13:20:56 +08:00
|
|
|
* explicit pattern, and we don't have
|
2007-11-10 07:32:10 +08:00
|
|
|
* --all nor --mirror.
|
|
|
|
*/
|
2007-06-08 07:43:05 +08:00
|
|
|
goto free_name;
|
2007-05-12 23:45:59 +08:00
|
|
|
if (!dst_peer) {
|
|
|
|
/* Create a new one and link it */
|
2007-07-10 12:47:26 +08:00
|
|
|
dst_peer = make_linked_ref(dst_name, dst_tail);
|
2007-05-12 23:45:59 +08:00
|
|
|
hashcpy(dst_peer->new_sha1, src->new_sha1);
|
|
|
|
}
|
|
|
|
dst_peer->peer_ref = src;
|
2007-10-19 17:04:00 +08:00
|
|
|
if (pat)
|
|
|
|
dst_peer->force = pat->force;
|
2007-06-08 07:43:05 +08:00
|
|
|
free_name:
|
|
|
|
free(dst_name);
|
2007-05-12 23:45:59 +08:00
|
|
|
}
|
|
|
|
return 0;
|
|
|
|
}
|
2007-09-11 11:02:56 +08:00
|
|
|
|
|
|
|
struct branch *branch_get(const char *name)
|
|
|
|
{
|
|
|
|
struct branch *ret;
|
|
|
|
|
|
|
|
read_config();
|
|
|
|
if (!name || !*name || !strcmp(name, "HEAD"))
|
|
|
|
ret = current_branch;
|
|
|
|
else
|
|
|
|
ret = make_branch(name, 0);
|
|
|
|
if (ret && ret->remote_name) {
|
|
|
|
ret->remote = remote_get(ret->remote_name);
|
|
|
|
if (ret->merge_nr) {
|
|
|
|
int i;
|
|
|
|
ret->merge = xcalloc(sizeof(*ret->merge),
|
|
|
|
ret->merge_nr);
|
|
|
|
for (i = 0; i < ret->merge_nr; i++) {
|
|
|
|
ret->merge[i] = xcalloc(1, sizeof(**ret->merge));
|
|
|
|
ret->merge[i]->src = xstrdup(ret->merge_name[i]);
|
|
|
|
remote_find_tracking(ret->remote,
|
|
|
|
ret->merge[i]);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
|
|
|
int branch_has_merge_config(struct branch *branch)
|
|
|
|
{
|
|
|
|
return branch && !!branch->merge;
|
|
|
|
}
|
|
|
|
|
2007-09-18 16:54:53 +08:00
|
|
|
int branch_merge_matches(struct branch *branch,
|
|
|
|
int i,
|
|
|
|
const char *refname)
|
2007-09-11 11:02:56 +08:00
|
|
|
{
|
2007-09-18 16:54:53 +08:00
|
|
|
if (!branch || i < 0 || i >= branch->merge_nr)
|
2007-09-11 11:02:56 +08:00
|
|
|
return 0;
|
2007-11-11 22:01:48 +08:00
|
|
|
return refname_match(branch->merge[i]->src, refname, ref_fetch_rules);
|
2007-09-11 11:02:56 +08:00
|
|
|
}
|
2007-09-11 11:03:08 +08:00
|
|
|
|
2007-10-30 09:05:40 +08:00
|
|
|
static struct ref *get_expanded_map(const struct ref *remote_refs,
|
2007-09-11 11:03:08 +08:00
|
|
|
const struct refspec *refspec)
|
|
|
|
{
|
2007-10-30 09:05:40 +08:00
|
|
|
const struct ref *ref;
|
2007-09-11 11:03:08 +08:00
|
|
|
struct ref *ret = NULL;
|
|
|
|
struct ref **tail = &ret;
|
|
|
|
|
|
|
|
int remote_prefix_len = strlen(refspec->src);
|
|
|
|
int local_prefix_len = strlen(refspec->dst);
|
|
|
|
|
|
|
|
for (ref = remote_refs; ref; ref = ref->next) {
|
|
|
|
if (strchr(ref->name, '^'))
|
|
|
|
continue; /* a dereference item */
|
|
|
|
if (!prefixcmp(ref->name, refspec->src)) {
|
2007-10-30 09:05:40 +08:00
|
|
|
const char *match;
|
2007-09-11 11:03:08 +08:00
|
|
|
struct ref *cpy = copy_ref(ref);
|
|
|
|
match = ref->name + remote_prefix_len;
|
|
|
|
|
|
|
|
cpy->peer_ref = alloc_ref(local_prefix_len +
|
|
|
|
strlen(match) + 1);
|
|
|
|
sprintf(cpy->peer_ref->name, "%s%s",
|
|
|
|
refspec->dst, match);
|
|
|
|
if (refspec->force)
|
|
|
|
cpy->peer_ref->force = 1;
|
|
|
|
*tail = cpy;
|
|
|
|
tail = &cpy->next;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
2007-10-30 09:05:40 +08:00
|
|
|
static const struct ref *find_ref_by_name_abbrev(const struct ref *refs, const char *name)
|
2007-09-11 11:03:08 +08:00
|
|
|
{
|
2007-10-30 09:05:40 +08:00
|
|
|
const struct ref *ref;
|
2007-09-11 11:03:08 +08:00
|
|
|
for (ref = refs; ref; ref = ref->next) {
|
2007-11-11 22:01:48 +08:00
|
|
|
if (refname_match(name, ref->name, ref_fetch_rules))
|
2007-09-11 11:03:08 +08:00
|
|
|
return ref;
|
|
|
|
}
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
2007-10-30 09:05:40 +08:00
|
|
|
struct ref *get_remote_ref(const struct ref *remote_refs, const char *name)
|
2007-09-11 11:03:08 +08:00
|
|
|
{
|
2007-10-30 09:05:40 +08:00
|
|
|
const struct ref *ref = find_ref_by_name_abbrev(remote_refs, name);
|
2007-09-11 11:03:08 +08:00
|
|
|
|
|
|
|
if (!ref)
|
2007-10-27 14:09:48 +08:00
|
|
|
return NULL;
|
2007-09-11 11:03:08 +08:00
|
|
|
|
|
|
|
return copy_ref(ref);
|
|
|
|
}
|
|
|
|
|
|
|
|
static struct ref *get_local_ref(const char *name)
|
|
|
|
{
|
|
|
|
struct ref *ret;
|
|
|
|
if (!name)
|
|
|
|
return NULL;
|
|
|
|
|
|
|
|
if (!prefixcmp(name, "refs/")) {
|
|
|
|
ret = alloc_ref(strlen(name) + 1);
|
|
|
|
strcpy(ret->name, name);
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (!prefixcmp(name, "heads/") ||
|
|
|
|
!prefixcmp(name, "tags/") ||
|
|
|
|
!prefixcmp(name, "remotes/")) {
|
|
|
|
ret = alloc_ref(strlen(name) + 6);
|
|
|
|
sprintf(ret->name, "refs/%s", name);
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
|
|
|
ret = alloc_ref(strlen(name) + 12);
|
|
|
|
sprintf(ret->name, "refs/heads/%s", name);
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
2007-10-30 09:05:40 +08:00
|
|
|
int get_fetch_map(const struct ref *remote_refs,
|
2007-09-11 11:03:08 +08:00
|
|
|
const struct refspec *refspec,
|
2007-10-27 14:09:48 +08:00
|
|
|
struct ref ***tail,
|
|
|
|
int missing_ok)
|
2007-09-11 11:03:08 +08:00
|
|
|
{
|
2008-03-18 10:05:23 +08:00
|
|
|
struct ref *ref_map, **rmp;
|
2007-09-11 11:03:08 +08:00
|
|
|
|
|
|
|
if (refspec->pattern) {
|
|
|
|
ref_map = get_expanded_map(remote_refs, refspec);
|
|
|
|
} else {
|
2007-10-27 14:09:48 +08:00
|
|
|
const char *name = refspec->src[0] ? refspec->src : "HEAD";
|
|
|
|
|
|
|
|
ref_map = get_remote_ref(remote_refs, name);
|
|
|
|
if (!missing_ok && !ref_map)
|
|
|
|
die("Couldn't find remote ref %s", name);
|
|
|
|
if (ref_map) {
|
|
|
|
ref_map->peer_ref = get_local_ref(refspec->dst);
|
|
|
|
if (ref_map->peer_ref && refspec->force)
|
|
|
|
ref_map->peer_ref->force = 1;
|
|
|
|
}
|
2007-09-11 11:03:08 +08:00
|
|
|
}
|
|
|
|
|
2008-03-18 10:05:23 +08:00
|
|
|
for (rmp = &ref_map; *rmp; ) {
|
|
|
|
if ((*rmp)->peer_ref) {
|
|
|
|
int st = check_ref_format((*rmp)->peer_ref->name + 5);
|
|
|
|
if (st && st != CHECK_REF_FORMAT_ONELEVEL) {
|
|
|
|
struct ref *ignore = *rmp;
|
|
|
|
error("* Ignoring funny ref '%s' locally",
|
|
|
|
(*rmp)->peer_ref->name);
|
|
|
|
*rmp = (*rmp)->next;
|
|
|
|
free(ignore->peer_ref);
|
|
|
|
free(ignore);
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
rmp = &((*rmp)->next);
|
2007-09-11 11:03:08 +08:00
|
|
|
}
|
|
|
|
|
2007-10-13 04:40:04 +08:00
|
|
|
if (ref_map)
|
|
|
|
tail_link_ref(ref_map, tail);
|
2007-09-11 11:03:08 +08:00
|
|
|
|
|
|
|
return 0;
|
|
|
|
}
|