]> git.ipfire.org Git - thirdparty/git.git/blobdiff - remote.c
Merge branch 'jk/multi-line-indent-style-fix'
[thirdparty/git.git] / remote.c
index 76838ed4d488af262ddd6170d1d9bc91c9b2b43c..8b9d090ae5c7bed16ff6753c8871d8a1568f2a4c 100644 (file)
--- a/remote.c
+++ b/remote.c
@@ -11,7 +11,7 @@
 #include "tag.h"
 #include "string-list.h"
 #include "mergesort.h"
-#include "argv-array.h"
+#include "strvec.h"
 #include "commit-reach.h"
 #include "advice.h"
 
@@ -174,54 +174,43 @@ static void add_merge(struct branch *branch, const char *name)
        branch->merge_name[branch->merge_nr++] = name;
 }
 
-static struct branch *make_branch(const char *name, int len)
+static struct branch *make_branch(const char *name, size_t len)
 {
        struct branch *ret;
        int i;
 
        for (i = 0; i < branches_nr; i++) {
-               if (len ? (!strncmp(name, branches[i]->name, len) &&
-                          !branches[i]->name[len]) :
-                   !strcmp(name, branches[i]->name))
+               if (!strncmp(name, branches[i]->name, len) &&
+                   !branches[i]->name[len])
                        return branches[i];
        }
 
        ALLOC_GROW(branches, branches_nr + 1, branches_alloc);
        ret = xcalloc(1, sizeof(struct branch));
        branches[branches_nr++] = ret;
-       if (len)
-               ret->name = xstrndup(name, len);
-       else
-               ret->name = xstrdup(name);
+       ret->name = xstrndup(name, len);
        ret->refname = xstrfmt("refs/heads/%s", ret->name);
 
        return ret;
 }
 
-static struct rewrite *make_rewrite(struct rewrites *r, const char *base, int len)
+static struct rewrite *make_rewrite(struct rewrites *r,
+                                   const char *base, size_t len)
 {
        struct rewrite *ret;
        int i;
 
        for (i = 0; i < r->rewrite_nr; i++) {
-               if (len
-                   ? (len == r->rewrite[i]->baselen &&
-                      !strncmp(base, r->rewrite[i]->base, len))
-                   : !strcmp(base, r->rewrite[i]->base))
+               if (len == r->rewrite[i]->baselen &&
+                   !strncmp(base, r->rewrite[i]->base, len))
                        return r->rewrite[i];
        }
 
        ALLOC_GROW(r->rewrite, r->rewrite_nr + 1, r->rewrite_alloc);
        ret = xcalloc(1, sizeof(struct rewrite));
        r->rewrite[r->rewrite_nr++] = ret;
-       if (len) {
-               ret->base = xstrndup(base, len);
-               ret->baselen = len;
-       }
-       else {
-               ret->base = xstrdup(base);
-               ret->baselen = strlen(base);
-       }
+       ret->base = xstrndup(base, len);
+       ret->baselen = len;
        return ret;
 }
 
@@ -287,7 +276,7 @@ static void read_branches_file(struct remote *remote)
 
        /*
         * The branches file would have URL and optionally
-        * #branch specified.  The "master" (or specified) branch is
+        * #branch specified.  The default (or specified) branch is
         * fetched and stored in the local branch matching the
         * remote name.
         */
@@ -295,28 +284,24 @@ static void read_branches_file(struct remote *remote)
        if (frag)
                *(frag++) = '\0';
        else
-               frag = "master";
+               frag = (char *)git_default_branch_name();
 
        add_url_alias(remote, strbuf_detach(&buf, NULL));
-       strbuf_addf(&buf, "refs/heads/%s:refs/heads/%s",
-                   frag, remote->name);
-       refspec_append(&remote->fetch, buf.buf);
+       refspec_appendf(&remote->fetch, "refs/heads/%s:refs/heads/%s",
+                       frag, remote->name);
 
        /*
         * Cogito compatible push: push current HEAD to remote #branch
         * (master if missing)
         */
-       strbuf_reset(&buf);
-       strbuf_addf(&buf, "HEAD:refs/heads/%s", frag);
-       refspec_append(&remote->push, buf.buf);
+       refspec_appendf(&remote->push, "HEAD:refs/heads/%s", frag);
        remote->fetch_tags = 1; /* always auto-follow */
-       strbuf_release(&buf);
 }
 
 static int handle_config(const char *key, const char *value, void *cb)
 {
        const char *name;
-       int namelen;
+       size_t namelen;
        const char *subkey;
        struct remote *remote;
        struct branch *branch;
@@ -470,7 +455,7 @@ static void read_config(void)
                const char *head_ref = resolve_ref_unsafe("HEAD", 0, NULL, &flag);
                if (head_ref && (flag & REF_ISSYMREF) &&
                    skip_prefix(head_ref, "refs/heads/", &head_ref)) {
-                       current_branch = make_branch(head_ref, 0);
+                       current_branch = make_branch(head_ref, strlen(head_ref));
                }
        }
        git_config(handle_config, NULL);
@@ -516,14 +501,11 @@ const char *pushremote_for_branch(struct branch *branch, int *explicit)
        return remote_for_branch(branch, explicit);
 }
 
-const char *remote_ref_for_branch(struct branch *branch, int for_push,
-                                 int *explicit)
+const char *remote_ref_for_branch(struct branch *branch, int for_push)
 {
        if (branch) {
                if (!for_push) {
                        if (branch->merge_nr) {
-                               if (explicit)
-                                       *explicit = 1;
                                return branch->merge_name[0];
                        }
                } else {
@@ -534,15 +516,11 @@ const char *remote_ref_for_branch(struct branch *branch, int for_push,
                        if (remote && remote->push.nr &&
                            (dst = apply_refspecs(&remote->push,
                                                  branch->refname))) {
-                               if (explicit)
-                                       *explicit = 1;
                                return dst;
                        }
                }
        }
-       if (explicit)
-               *explicit = 0;
-       return "";
+       return NULL;
 }
 
 static struct remote *remote_get_1(const char *name,
@@ -704,6 +682,91 @@ static int match_name_with_pattern(const char *key, const char *name,
        return ret;
 }
 
+static int refspec_match(const struct refspec_item *refspec,
+                        const char *name)
+{
+       if (refspec->pattern)
+               return match_name_with_pattern(refspec->src, name, NULL, NULL);
+
+       return !strcmp(refspec->src, name);
+}
+
+static int omit_name_by_refspec(const char *name, struct refspec *rs)
+{
+       int i;
+
+       for (i = 0; i < rs->nr; i++) {
+               if (rs->items[i].negative && refspec_match(&rs->items[i], name))
+                       return 1;
+       }
+       return 0;
+}
+
+struct ref *apply_negative_refspecs(struct ref *ref_map, struct refspec *rs)
+{
+       struct ref **tail;
+
+       for (tail = &ref_map; *tail; ) {
+               struct ref *ref = *tail;
+
+               if (omit_name_by_refspec(ref->name, rs)) {
+                       *tail = ref->next;
+                       free(ref->peer_ref);
+                       free(ref);
+               } else
+                       tail = &ref->next;
+       }
+
+       return ref_map;
+}
+
+static int query_matches_negative_refspec(struct refspec *rs, struct refspec_item *query)
+{
+       int i, matched_negative = 0;
+       int find_src = !query->src;
+       struct string_list reversed = STRING_LIST_INIT_NODUP;
+       const char *needle = find_src ? query->dst : query->src;
+
+       /*
+        * Check whether the queried ref matches any negative refpsec. If so,
+        * then we should ultimately treat this as not matching the query at
+        * all.
+        *
+        * Note that negative refspecs always match the source, but the query
+        * item uses the destination. To handle this, we apply pattern
+        * refspecs in reverse to figure out if the query source matches any
+        * of the negative refspecs.
+        */
+       for (i = 0; i < rs->nr; i++) {
+               struct refspec_item *refspec = &rs->items[i];
+               char *expn_name;
+
+               if (refspec->negative)
+                       continue;
+
+               /* Note the reversal of src and dst */
+               if (refspec->pattern) {
+                       const char *key = refspec->dst ? refspec->dst : refspec->src;
+                       const char *value = refspec->src;
+
+                       if (match_name_with_pattern(key, needle, value, &expn_name))
+                               string_list_append_nodup(&reversed, expn_name);
+               } else {
+                       if (!strcmp(needle, refspec->src))
+                               string_list_append(&reversed, refspec->src);
+               }
+       }
+
+       for (i = 0; !matched_negative && i < reversed.nr; i++) {
+               if (omit_name_by_refspec(reversed.items[i].string, rs))
+                       matched_negative = 1;
+       }
+
+       string_list_clear(&reversed, 0);
+
+       return matched_negative;
+}
+
 static void query_refspecs_multiple(struct refspec *rs,
                                    struct refspec_item *query,
                                    struct string_list *results)
@@ -714,6 +777,9 @@ static void query_refspecs_multiple(struct refspec *rs,
        if (find_src && !query->dst)
                BUG("query_refspecs_multiple: need either src or dst");
 
+       if (query_matches_negative_refspec(rs, query))
+               return;
+
        for (i = 0; i < rs->nr; i++) {
                struct refspec_item *refspec = &rs->items[i];
                const char *key = find_src ? refspec->dst : refspec->src;
@@ -721,7 +787,7 @@ static void query_refspecs_multiple(struct refspec *rs,
                const char *needle = find_src ? query->dst : query->src;
                char **result = find_src ? &query->src : &query->dst;
 
-               if (!refspec->dst)
+               if (!refspec->dst || refspec->negative)
                        continue;
                if (refspec->pattern) {
                        if (match_name_with_pattern(key, needle, value, result))
@@ -742,12 +808,15 @@ int query_refspecs(struct refspec *rs, struct refspec_item *query)
        if (find_src && !query->dst)
                BUG("query_refspecs: need either src or dst");
 
+       if (query_matches_negative_refspec(rs, query))
+               return -1;
+
        for (i = 0; i < rs->nr; i++) {
                struct refspec_item *refspec = &rs->items[i];
                const char *key = find_src ? refspec->dst : refspec->src;
                const char *value = find_src ? refspec->src : refspec->dst;
 
-               if (!refspec->dst)
+               if (!refspec->dst || refspec->negative)
                        continue;
                if (refspec->pattern) {
                        if (match_name_with_pattern(key, needle, value, result)) {
@@ -1076,7 +1145,7 @@ static int match_explicit(struct ref *src, struct ref *dst,
        const char *dst_value = rs->dst;
        char *dst_guess;
 
-       if (rs->pattern || rs->matching)
+       if (rs->pattern || rs->matching || rs->negative)
                return 0;
 
        matched_src = matched_dst = NULL;
@@ -1152,6 +1221,10 @@ static char *get_ref_match(const struct refspec *rs, const struct ref *ref,
        int matching_refs = -1;
        for (i = 0; i < rs->nr; i++) {
                const struct refspec_item *item = &rs->items[i];
+
+               if (item->negative)
+                       continue;
+
                if (item->matching &&
                    (matching_refs == -1 || item->force)) {
                        matching_refs = i;
@@ -1357,7 +1430,7 @@ int check_push_refs(struct ref *src, struct refspec *rs)
        for (i = 0; i < rs->nr; i++) {
                struct refspec_item *item = &rs->items[i];
 
-               if (item->pattern || item->matching)
+               if (item->pattern || item->matching || item->negative)
                        continue;
 
                ret |= match_explicit_lhs(src, item, NULL, NULL);
@@ -1459,6 +1532,8 @@ int match_push_refs(struct ref *src, struct ref **dst,
                string_list_clear(&src_ref_index, 0);
        }
 
+       *dst = apply_negative_refspecs(*dst, rs);
+
        if (errs)
                return -1;
        return 0;
@@ -1493,12 +1568,23 @@ void set_ref_status_for_push(struct ref *remote_refs, int send_mirror,
                 * with the remote-tracking branch to find the value
                 * to expect, but we did not have such a tracking
                 * branch.
+                *
+                * If the tip of the remote-tracking ref is unreachable
+                * from any reflog entry of its local ref indicating a
+                * possible update since checkout; reject the push.
                 */
                if (ref->expect_old_sha1) {
                        if (!oideq(&ref->old_oid, &ref->old_oid_expect))
                                reject_reason = REF_STATUS_REJECT_STALE;
+                       else if (ref->check_reachable && ref->unreachable)
+                               reject_reason =
+                                       REF_STATUS_REJECT_REMOTE_UPDATED;
                        else
-                               /* If the ref isn't stale then force the update. */
+                               /*
+                                * If the ref isn't stale, and is reachable
+                                * from from one of the reflog entries of
+                                * the local branch, force the update.
+                                */
                                force_ref_update = 1;
                }
 
@@ -1576,7 +1662,7 @@ static void set_merge(struct branch *ret)
                    strcmp(ret->remote_name, "."))
                        continue;
                if (dwim_ref(ret->merge_name[i], strlen(ret->merge_name[i]),
-                            &oid, &ref) == 1)
+                            &oid, &ref, 0) == 1)
                        ret->merge[i]->dst = ref;
                else
                        ret->merge[i]->dst = xstrdup(ret->merge_name[i]);
@@ -1591,7 +1677,7 @@ struct branch *branch_get(const char *name)
        if (!name || !*name || !strcmp(name, "HEAD"))
                ret = current_branch;
        else
-               ret = make_branch(name, 0);
+               ret = make_branch(name, strlen(name));
        set_merge(ret);
        return ret;
 }
@@ -1828,6 +1914,9 @@ int get_fetch_map(const struct ref *remote_refs,
 {
        struct ref *ref_map, **rmp;
 
+       if (refspec->negative)
+               return 0;
+
        if (refspec->pattern) {
                ref_map = get_expanded_map(remote_refs, refspec);
        } else {
@@ -1903,7 +1992,7 @@ static int stat_branch_pair(const char *branch_name, const char *base,
        struct object_id oid;
        struct commit *ours, *theirs;
        struct rev_info revs;
-       struct argv_array argv = ARGV_ARRAY_INIT;
+       struct strvec argv = STRVEC_INIT;
 
        /* Cannot stat if what we used to build on no longer exists */
        if (read_ref(base, &oid))
@@ -1929,15 +2018,15 @@ static int stat_branch_pair(const char *branch_name, const char *base,
                BUG("stat_branch_pair: invalid abf '%d'", abf);
 
        /* Run "rev-list --left-right ours...theirs" internally... */
-       argv_array_push(&argv, ""); /* ignored */
-       argv_array_push(&argv, "--left-right");
-       argv_array_pushf(&argv, "%s...%s",
-                        oid_to_hex(&ours->object.oid),
-                        oid_to_hex(&theirs->object.oid));
-       argv_array_push(&argv, "--");
+       strvec_push(&argv, ""); /* ignored */
+       strvec_push(&argv, "--left-right");
+       strvec_pushf(&argv, "%s...%s",
+                    oid_to_hex(&ours->object.oid),
+                    oid_to_hex(&theirs->object.oid));
+       strvec_push(&argv, "--");
 
        repo_init_revisions(the_repository, &revs, NULL);
-       setup_revisions(argv.argc, argv.argv, &revs, NULL);
+       setup_revisions(argv.nr, argv.v, &revs, NULL);
        if (prepare_revision_walk(&revs))
                die(_("revision walk setup failed"));
 
@@ -1956,7 +2045,7 @@ static int stat_branch_pair(const char *branch_name, const char *base,
        clear_commit_marks(ours, ALL_REV_FLAGS);
        clear_commit_marks(theirs, ALL_REV_FLAGS);
 
-       argv_array_clear(&argv);
+       strvec_clear(&argv);
        return 1;
 }
 
@@ -2115,8 +2204,16 @@ struct ref *guess_remote_head(const struct ref *head,
        if (head->symref)
                return copy_ref(find_ref_by_name(refs, head->symref));
 
-       /* If refs/heads/master could be right, it is. */
+       /* If a remote branch exists with the default branch name, let's use it. */
        if (!all) {
+               char *ref = xstrfmt("refs/heads/%s", git_default_branch_name());
+
+               r = find_ref_by_name(refs, ref);
+               free(ref);
+               if (r && oideq(&r->old_oid, &head->old_oid))
+                       return copy_ref(r);
+
+               /* Fall back to the hard-coded historical default */
                r = find_ref_by_name(refs, "refs/heads/master");
                if (r && oideq(&r->old_oid, &head->old_oid))
                        return copy_ref(r);
@@ -2265,12 +2362,13 @@ int is_empty_cas(const struct push_cas_option *cas)
 
 /*
  * Look at remote.fetch refspec and see if we have a remote
- * tracking branch for the refname there.  Fill its current
- * value in sha1[].
+ * tracking branch for the refname there. Fill the name of
+ * the remote-tracking branch in *dst_refname, and the name
+ * of the commit object at its tip in oid[].
  * If we cannot do so, return negative to signal an error.
  */
 static int remote_tracking(struct remote *remote, const char *refname,
-                          struct object_id *oid)
+                          struct object_id *oid, char **dst_refname)
 {
        char *dst;
 
@@ -2279,9 +2377,150 @@ static int remote_tracking(struct remote *remote, const char *refname,
                return -1; /* no tracking ref for refname at remote */
        if (read_ref(dst, oid))
                return -1; /* we know what the tracking ref is but we cannot read it */
+
+       *dst_refname = dst;
+       return 0;
+}
+
+/*
+ * The struct "reflog_commit_array" and related helper functions
+ * are used for collecting commits into an array during reflog
+ * traversals in "check_and_collect_until()".
+ */
+struct reflog_commit_array {
+       struct commit **item;
+       size_t nr, alloc;
+};
+
+#define REFLOG_COMMIT_ARRAY_INIT { NULL, 0, 0 }
+
+/* Append a commit to the array. */
+static void append_commit(struct reflog_commit_array *arr,
+                         struct commit *commit)
+{
+       ALLOC_GROW(arr->item, arr->nr + 1, arr->alloc);
+       arr->item[arr->nr++] = commit;
+}
+
+/* Free and reset the array. */
+static void free_commit_array(struct reflog_commit_array *arr)
+{
+       FREE_AND_NULL(arr->item);
+       arr->nr = arr->alloc = 0;
+}
+
+struct check_and_collect_until_cb_data {
+       struct commit *remote_commit;
+       struct reflog_commit_array *local_commits;
+       timestamp_t remote_reflog_timestamp;
+};
+
+/* Get the timestamp of the latest entry. */
+static int peek_reflog(struct object_id *o_oid, struct object_id *n_oid,
+                      const char *ident, timestamp_t timestamp,
+                      int tz, const char *message, void *cb_data)
+{
+       timestamp_t *ts = cb_data;
+       *ts = timestamp;
+       return 1;
+}
+
+static int check_and_collect_until(struct object_id *o_oid,
+                                  struct object_id *n_oid,
+                                  const char *ident, timestamp_t timestamp,
+                                  int tz, const char *message, void *cb_data)
+{
+       struct commit *commit;
+       struct check_and_collect_until_cb_data *cb = cb_data;
+
+       /* An entry was found. */
+       if (oideq(n_oid, &cb->remote_commit->object.oid))
+               return 1;
+
+       if ((commit = lookup_commit_reference(the_repository, n_oid)))
+               append_commit(cb->local_commits, commit);
+
+       /*
+        * If the reflog entry timestamp is older than the remote ref's
+        * latest reflog entry, there is no need to check or collect
+        * entries older than this one.
+        */
+       if (timestamp < cb->remote_reflog_timestamp)
+               return -1;
+
        return 0;
 }
 
+#define MERGE_BASES_BATCH_SIZE 8
+
+/*
+ * Iterate through the reflog of the local ref to check if there is an entry
+ * for the given remote-tracking ref; runs until the timestamp of an entry is
+ * older than latest timestamp of remote-tracking ref's reflog. Any commits
+ * are that seen along the way are collected into an array to check if the
+ * remote-tracking ref is reachable from any of them.
+ */
+static int is_reachable_in_reflog(const char *local, const struct ref *remote)
+{
+       timestamp_t date;
+       struct commit *commit;
+       struct commit **chunk;
+       struct check_and_collect_until_cb_data cb;
+       struct reflog_commit_array arr = REFLOG_COMMIT_ARRAY_INIT;
+       size_t size = 0;
+       int ret = 0;
+
+       commit = lookup_commit_reference(the_repository, &remote->old_oid);
+       if (!commit)
+               goto cleanup_return;
+
+       /*
+        * Get the timestamp from the latest entry
+        * of the remote-tracking ref's reflog.
+        */
+       for_each_reflog_ent_reverse(remote->tracking_ref, peek_reflog, &date);
+
+       cb.remote_commit = commit;
+       cb.local_commits = &arr;
+       cb.remote_reflog_timestamp = date;
+       ret = for_each_reflog_ent_reverse(local, check_and_collect_until, &cb);
+
+       /* We found an entry in the reflog. */
+       if (ret > 0)
+               goto cleanup_return;
+
+       /*
+        * Check if the remote commit is reachable from any
+        * of the commits in the collected array, in batches.
+        */
+       for (chunk = arr.item; chunk < arr.item + arr.nr; chunk += size) {
+               size = arr.item + arr.nr - chunk;
+               if (MERGE_BASES_BATCH_SIZE < size)
+                       size = MERGE_BASES_BATCH_SIZE;
+
+               if ((ret = in_merge_bases_many(commit, size, chunk)))
+                       break;
+       }
+
+cleanup_return:
+       free_commit_array(&arr);
+       return ret;
+}
+
+/*
+ * Check for reachability of a remote-tracking
+ * ref in the reflog entries of its local ref.
+ */
+static void check_if_includes_upstream(struct ref *remote)
+{
+       struct ref *local = get_local_ref(remote->name);
+       if (!local)
+               return;
+
+       if (is_reachable_in_reflog(local->name, remote) <= 0)
+               remote->unreachable = 1;
+}
+
 static void apply_cas(struct push_cas_option *cas,
                      struct remote *remote,
                      struct ref *ref)
@@ -2296,8 +2535,12 @@ static void apply_cas(struct push_cas_option *cas,
                ref->expect_old_sha1 = 1;
                if (!entry->use_tracking)
                        oidcpy(&ref->old_oid_expect, &entry->expect);
-               else if (remote_tracking(remote, ref->name, &ref->old_oid_expect))
+               else if (remote_tracking(remote, ref->name,
+                                        &ref->old_oid_expect,
+                                        &ref->tracking_ref))
                        oidclr(&ref->old_oid_expect);
+               else
+                       ref->check_reachable = cas->use_force_if_includes;
                return;
        }
 
@@ -2306,8 +2549,12 @@ static void apply_cas(struct push_cas_option *cas,
                return;
 
        ref->expect_old_sha1 = 1;
-       if (remote_tracking(remote, ref->name, &ref->old_oid_expect))
+       if (remote_tracking(remote, ref->name,
+                           &ref->old_oid_expect,
+                           &ref->tracking_ref))
                oidclr(&ref->old_oid_expect);
+       else
+               ref->check_reachable = cas->use_force_if_includes;
 }
 
 void apply_push_cas(struct push_cas_option *cas,
@@ -2315,6 +2562,15 @@ void apply_push_cas(struct push_cas_option *cas,
                    struct ref *remote_refs)
 {
        struct ref *ref;
-       for (ref = remote_refs; ref; ref = ref->next)
+       for (ref = remote_refs; ref; ref = ref->next) {
                apply_cas(cas, remote, ref);
+
+               /*
+                * If "compare-and-swap" is in "use_tracking[_for_rest]"
+                * mode, and if "--force-if-includes" was specified, run
+                * the check.
+                */
+               if (ref->check_reachable)
+                       check_if_includes_upstream(ref);
+       }
 }