2007-09-11 05:03:15 +02:00
|
|
|
#include "cache.h"
|
2014-10-01 12:28:42 +02:00
|
|
|
#include "lockfile.h"
|
2007-09-11 05:03:15 +02:00
|
|
|
#include "bundle.h"
|
2018-05-16 01:42:15 +02:00
|
|
|
#include "object-store.h"
|
2018-06-29 03:21:51 +02:00
|
|
|
#include "repository.h"
|
2007-09-11 05:03:15 +02:00
|
|
|
#include "object.h"
|
|
|
|
#include "commit.h"
|
|
|
|
#include "diff.h"
|
|
|
|
#include "revision.h"
|
|
|
|
#include "list-objects.h"
|
|
|
|
#include "run-command.h"
|
2007-11-23 01:51:18 +01:00
|
|
|
#include "refs.h"
|
2020-07-28 22:23:39 +02:00
|
|
|
#include "strvec.h"
|
2007-09-11 05:03:15 +02:00
|
|
|
|
2020-07-30 01:14:20 +02:00
|
|
|
|
|
|
|
static const char v2_bundle_signature[] = "# v2 git bundle\n";
|
|
|
|
static const char v3_bundle_signature[] = "# v3 git bundle\n";
|
|
|
|
static struct {
|
|
|
|
int version;
|
|
|
|
const char *signature;
|
|
|
|
} bundle_sigs[] = {
|
|
|
|
{ 2, v2_bundle_signature },
|
|
|
|
{ 3, v3_bundle_signature },
|
|
|
|
};
|
2007-09-11 05:03:15 +02:00
|
|
|
|
bundle: remove "ref_list" in favor of string-list.c API
Move away from the "struct ref_list" in bundle.c in favor of the
almost identical string-list.c API.
That API fits this use-case perfectly, but did not exist in its
current form when this code was added in 2e0afafebd (Add git-bundle:
move objects and references by archive, 2007-02-22), with hindsight we
could have used the path-list API, which later got renamed to
string-list. See 8fd2cb4069 (Extract helper bits from
c-merge-recursive work, 2006-07-25)
We need to change "name" to "string" and "oid" to "util" to make this
conversion, but other than that the APIs are pretty much identical for
what bundle.c made use of.
Let's also replace the memset(..,0,...) pattern with a more idiomatic
"INIT" macro, and finally add a *_release() function so to free the
allocated memory.
Before this the add_to_ref_list() would leak memory, now e.g. "bundle
list-heads" reports no memory leaks at all under valgrind.
In the bundle_header_init() function we're using a clever trick to
memcpy() what we'd get from the corresponding
BUNDLE_HEADER_INIT. There is a concurrent series to make use of that
pattern more generally, see [1].
1. https://lore.kernel.org/git/cover-0.5-00000000000-20210701T104855Z-avarab@gmail.com/
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Acked-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2021-07-02 11:57:32 +02:00
|
|
|
void bundle_header_init(struct bundle_header *header)
|
2007-09-11 05:03:15 +02:00
|
|
|
{
|
bundle: remove "ref_list" in favor of string-list.c API
Move away from the "struct ref_list" in bundle.c in favor of the
almost identical string-list.c API.
That API fits this use-case perfectly, but did not exist in its
current form when this code was added in 2e0afafebd (Add git-bundle:
move objects and references by archive, 2007-02-22), with hindsight we
could have used the path-list API, which later got renamed to
string-list. See 8fd2cb4069 (Extract helper bits from
c-merge-recursive work, 2006-07-25)
We need to change "name" to "string" and "oid" to "util" to make this
conversion, but other than that the APIs are pretty much identical for
what bundle.c made use of.
Let's also replace the memset(..,0,...) pattern with a more idiomatic
"INIT" macro, and finally add a *_release() function so to free the
allocated memory.
Before this the add_to_ref_list() would leak memory, now e.g. "bundle
list-heads" reports no memory leaks at all under valgrind.
In the bundle_header_init() function we're using a clever trick to
memcpy() what we'd get from the corresponding
BUNDLE_HEADER_INIT. There is a concurrent series to make use of that
pattern more generally, see [1].
1. https://lore.kernel.org/git/cover-0.5-00000000000-20210701T104855Z-avarab@gmail.com/
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Acked-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2021-07-02 11:57:32 +02:00
|
|
|
struct bundle_header blank = BUNDLE_HEADER_INIT;
|
|
|
|
memcpy(header, &blank, sizeof(*header));
|
|
|
|
}
|
|
|
|
|
|
|
|
void bundle_header_release(struct bundle_header *header)
|
|
|
|
{
|
|
|
|
string_list_clear(&header->prerequisites, 1);
|
|
|
|
string_list_clear(&header->references, 1);
|
2007-09-11 05:03:15 +02:00
|
|
|
}
|
|
|
|
|
2020-07-30 01:14:20 +02:00
|
|
|
static int parse_capability(struct bundle_header *header, const char *capability)
|
2020-06-19 19:56:00 +02:00
|
|
|
{
|
2020-07-30 01:14:20 +02:00
|
|
|
const char *arg;
|
|
|
|
if (skip_prefix(capability, "object-format=", &arg)) {
|
|
|
|
int algo = hash_algo_by_name(arg);
|
|
|
|
if (algo == GIT_HASH_UNKNOWN)
|
|
|
|
return error(_("unrecognized bundle hash algorithm: %s"), arg);
|
|
|
|
header->hash_algo = &hash_algos[algo];
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
return error(_("unknown capability '%s'"), capability);
|
|
|
|
}
|
2020-06-19 19:56:00 +02:00
|
|
|
|
2020-07-30 01:14:20 +02:00
|
|
|
static int parse_bundle_signature(struct bundle_header *header, const char *line)
|
|
|
|
{
|
|
|
|
int i;
|
|
|
|
|
|
|
|
for (i = 0; i < ARRAY_SIZE(bundle_sigs); i++) {
|
|
|
|
if (!strcmp(line, bundle_sigs[i].signature)) {
|
|
|
|
header->version = bundle_sigs[i].version;
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
return -1;
|
2020-06-19 19:56:00 +02:00
|
|
|
}
|
|
|
|
|
2011-10-14 00:19:31 +02:00
|
|
|
static int parse_bundle_header(int fd, struct bundle_header *header,
|
|
|
|
const char *report_path)
|
2007-11-09 00:35:32 +01:00
|
|
|
{
|
2011-10-14 00:12:02 +02:00
|
|
|
struct strbuf buf = STRBUF_INIT;
|
|
|
|
int status = 0;
|
2007-09-11 05:03:15 +02:00
|
|
|
|
2011-10-14 00:12:02 +02:00
|
|
|
/* The bundle header begins with the signature */
|
2012-02-22 20:34:22 +01:00
|
|
|
if (strbuf_getwholeline_fd(&buf, fd, '\n') ||
|
2020-07-30 01:14:20 +02:00
|
|
|
parse_bundle_signature(header, buf.buf)) {
|
2011-10-14 00:19:31 +02:00
|
|
|
if (report_path)
|
2020-07-30 01:14:20 +02:00
|
|
|
error(_("'%s' does not look like a v2 or v3 bundle file"),
|
2011-10-14 00:19:31 +02:00
|
|
|
report_path);
|
2011-10-14 00:12:02 +02:00
|
|
|
status = -1;
|
|
|
|
goto abort;
|
2007-09-11 05:03:15 +02:00
|
|
|
}
|
2011-10-14 00:12:02 +02:00
|
|
|
|
2020-07-30 01:14:20 +02:00
|
|
|
header->hash_algo = the_hash_algo;
|
|
|
|
|
2011-10-14 00:12:02 +02:00
|
|
|
/* The bundle header ends with an empty line */
|
2012-02-22 20:34:22 +01:00
|
|
|
while (!strbuf_getwholeline_fd(&buf, fd, '\n') &&
|
2011-10-14 00:12:02 +02:00
|
|
|
buf.len && buf.buf[0] != '\n') {
|
2017-05-01 04:28:59 +02:00
|
|
|
struct object_id oid;
|
2011-10-14 00:12:02 +02:00
|
|
|
int is_prereq = 0;
|
2017-05-01 04:28:59 +02:00
|
|
|
const char *p;
|
2011-10-14 00:12:02 +02:00
|
|
|
|
|
|
|
strbuf_rtrim(&buf);
|
2007-09-11 05:03:15 +02:00
|
|
|
|
2020-07-30 01:14:20 +02:00
|
|
|
if (header->version == 3 && *buf.buf == '@') {
|
|
|
|
if (parse_capability(header, buf.buf + 1)) {
|
2020-06-19 19:56:00 +02:00
|
|
|
status = -1;
|
|
|
|
break;
|
|
|
|
}
|
2020-07-30 01:14:20 +02:00
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (*buf.buf == '-') {
|
|
|
|
is_prereq = 1;
|
|
|
|
strbuf_remove(&buf, 0, 1);
|
2020-06-19 19:56:00 +02:00
|
|
|
}
|
|
|
|
|
2011-10-14 00:12:02 +02:00
|
|
|
/*
|
|
|
|
* Tip lines have object name, SP, and refname.
|
|
|
|
* Prerequisites have object name that is optionally
|
|
|
|
* followed by SP and subject line.
|
|
|
|
*/
|
2020-06-19 19:56:00 +02:00
|
|
|
if (parse_oid_hex_algop(buf.buf, &oid, &p, header->hash_algo) ||
|
2017-05-01 04:28:59 +02:00
|
|
|
(*p && !isspace(*p)) ||
|
|
|
|
(!is_prereq && !*p)) {
|
2011-10-14 00:19:31 +02:00
|
|
|
if (report_path)
|
2012-04-23 14:30:30 +02:00
|
|
|
error(_("unrecognized header: %s%s (%d)"),
|
2011-10-14 00:19:31 +02:00
|
|
|
(is_prereq ? "-" : ""), buf.buf, (int)buf.len);
|
2011-10-14 00:12:02 +02:00
|
|
|
status = -1;
|
|
|
|
break;
|
|
|
|
} else {
|
bundle: remove "ref_list" in favor of string-list.c API
Move away from the "struct ref_list" in bundle.c in favor of the
almost identical string-list.c API.
That API fits this use-case perfectly, but did not exist in its
current form when this code was added in 2e0afafebd (Add git-bundle:
move objects and references by archive, 2007-02-22), with hindsight we
could have used the path-list API, which later got renamed to
string-list. See 8fd2cb4069 (Extract helper bits from
c-merge-recursive work, 2006-07-25)
We need to change "name" to "string" and "oid" to "util" to make this
conversion, but other than that the APIs are pretty much identical for
what bundle.c made use of.
Let's also replace the memset(..,0,...) pattern with a more idiomatic
"INIT" macro, and finally add a *_release() function so to free the
allocated memory.
Before this the add_to_ref_list() would leak memory, now e.g. "bundle
list-heads" reports no memory leaks at all under valgrind.
In the bundle_header_init() function we're using a clever trick to
memcpy() what we'd get from the corresponding
BUNDLE_HEADER_INIT. There is a concurrent series to make use of that
pattern more generally, see [1].
1. https://lore.kernel.org/git/cover-0.5-00000000000-20210701T104855Z-avarab@gmail.com/
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Acked-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2021-07-02 11:57:32 +02:00
|
|
|
struct object_id *dup = oiddup(&oid);
|
2011-10-14 00:12:02 +02:00
|
|
|
if (is_prereq)
|
bundle: remove "ref_list" in favor of string-list.c API
Move away from the "struct ref_list" in bundle.c in favor of the
almost identical string-list.c API.
That API fits this use-case perfectly, but did not exist in its
current form when this code was added in 2e0afafebd (Add git-bundle:
move objects and references by archive, 2007-02-22), with hindsight we
could have used the path-list API, which later got renamed to
string-list. See 8fd2cb4069 (Extract helper bits from
c-merge-recursive work, 2006-07-25)
We need to change "name" to "string" and "oid" to "util" to make this
conversion, but other than that the APIs are pretty much identical for
what bundle.c made use of.
Let's also replace the memset(..,0,...) pattern with a more idiomatic
"INIT" macro, and finally add a *_release() function so to free the
allocated memory.
Before this the add_to_ref_list() would leak memory, now e.g. "bundle
list-heads" reports no memory leaks at all under valgrind.
In the bundle_header_init() function we're using a clever trick to
memcpy() what we'd get from the corresponding
BUNDLE_HEADER_INIT. There is a concurrent series to make use of that
pattern more generally, see [1].
1. https://lore.kernel.org/git/cover-0.5-00000000000-20210701T104855Z-avarab@gmail.com/
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Acked-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2021-07-02 11:57:32 +02:00
|
|
|
string_list_append(&header->prerequisites, "")->util = dup;
|
2011-10-14 00:12:02 +02:00
|
|
|
else
|
bundle: remove "ref_list" in favor of string-list.c API
Move away from the "struct ref_list" in bundle.c in favor of the
almost identical string-list.c API.
That API fits this use-case perfectly, but did not exist in its
current form when this code was added in 2e0afafebd (Add git-bundle:
move objects and references by archive, 2007-02-22), with hindsight we
could have used the path-list API, which later got renamed to
string-list. See 8fd2cb4069 (Extract helper bits from
c-merge-recursive work, 2006-07-25)
We need to change "name" to "string" and "oid" to "util" to make this
conversion, but other than that the APIs are pretty much identical for
what bundle.c made use of.
Let's also replace the memset(..,0,...) pattern with a more idiomatic
"INIT" macro, and finally add a *_release() function so to free the
allocated memory.
Before this the add_to_ref_list() would leak memory, now e.g. "bundle
list-heads" reports no memory leaks at all under valgrind.
In the bundle_header_init() function we're using a clever trick to
memcpy() what we'd get from the corresponding
BUNDLE_HEADER_INIT. There is a concurrent series to make use of that
pattern more generally, see [1].
1. https://lore.kernel.org/git/cover-0.5-00000000000-20210701T104855Z-avarab@gmail.com/
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Acked-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2021-07-02 11:57:32 +02:00
|
|
|
string_list_append(&header->references, p + 1)->util = dup;
|
2007-09-11 05:03:15 +02:00
|
|
|
}
|
|
|
|
}
|
2011-10-14 00:12:02 +02:00
|
|
|
|
|
|
|
abort:
|
|
|
|
if (status) {
|
|
|
|
close(fd);
|
|
|
|
fd = -1;
|
2007-09-11 05:03:15 +02:00
|
|
|
}
|
2011-10-14 00:12:02 +02:00
|
|
|
strbuf_release(&buf);
|
2007-09-11 05:03:15 +02:00
|
|
|
return fd;
|
|
|
|
}
|
|
|
|
|
2011-10-14 00:19:31 +02:00
|
|
|
int read_bundle_header(const char *path, struct bundle_header *header)
|
|
|
|
{
|
|
|
|
int fd = open(path, O_RDONLY);
|
|
|
|
|
2007-09-11 05:03:15 +02:00
|
|
|
if (fd < 0)
|
2012-04-23 14:30:30 +02:00
|
|
|
return error(_("could not open '%s'"), path);
|
2011-10-14 00:19:31 +02:00
|
|
|
return parse_bundle_header(fd, header, path);
|
|
|
|
}
|
|
|
|
|
|
|
|
int is_bundle(const char *path, int quiet)
|
|
|
|
{
|
bundle: remove "ref_list" in favor of string-list.c API
Move away from the "struct ref_list" in bundle.c in favor of the
almost identical string-list.c API.
That API fits this use-case perfectly, but did not exist in its
current form when this code was added in 2e0afafebd (Add git-bundle:
move objects and references by archive, 2007-02-22), with hindsight we
could have used the path-list API, which later got renamed to
string-list. See 8fd2cb4069 (Extract helper bits from
c-merge-recursive work, 2006-07-25)
We need to change "name" to "string" and "oid" to "util" to make this
conversion, but other than that the APIs are pretty much identical for
what bundle.c made use of.
Let's also replace the memset(..,0,...) pattern with a more idiomatic
"INIT" macro, and finally add a *_release() function so to free the
allocated memory.
Before this the add_to_ref_list() would leak memory, now e.g. "bundle
list-heads" reports no memory leaks at all under valgrind.
In the bundle_header_init() function we're using a clever trick to
memcpy() what we'd get from the corresponding
BUNDLE_HEADER_INIT. There is a concurrent series to make use of that
pattern more generally, see [1].
1. https://lore.kernel.org/git/cover-0.5-00000000000-20210701T104855Z-avarab@gmail.com/
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Acked-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2021-07-02 11:57:32 +02:00
|
|
|
struct bundle_header header = BUNDLE_HEADER_INIT;
|
2011-10-14 00:19:31 +02:00
|
|
|
int fd = open(path, O_RDONLY);
|
|
|
|
|
|
|
|
if (fd < 0)
|
|
|
|
return 0;
|
|
|
|
fd = parse_bundle_header(fd, &header, quiet ? NULL : path);
|
|
|
|
if (fd >= 0)
|
|
|
|
close(fd);
|
bundle: remove "ref_list" in favor of string-list.c API
Move away from the "struct ref_list" in bundle.c in favor of the
almost identical string-list.c API.
That API fits this use-case perfectly, but did not exist in its
current form when this code was added in 2e0afafebd (Add git-bundle:
move objects and references by archive, 2007-02-22), with hindsight we
could have used the path-list API, which later got renamed to
string-list. See 8fd2cb4069 (Extract helper bits from
c-merge-recursive work, 2006-07-25)
We need to change "name" to "string" and "oid" to "util" to make this
conversion, but other than that the APIs are pretty much identical for
what bundle.c made use of.
Let's also replace the memset(..,0,...) pattern with a more idiomatic
"INIT" macro, and finally add a *_release() function so to free the
allocated memory.
Before this the add_to_ref_list() would leak memory, now e.g. "bundle
list-heads" reports no memory leaks at all under valgrind.
In the bundle_header_init() function we're using a clever trick to
memcpy() what we'd get from the corresponding
BUNDLE_HEADER_INIT. There is a concurrent series to make use of that
pattern more generally, see [1].
1. https://lore.kernel.org/git/cover-0.5-00000000000-20210701T104855Z-avarab@gmail.com/
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Acked-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2021-07-02 11:57:32 +02:00
|
|
|
bundle_header_release(&header);
|
2011-10-14 00:19:31 +02:00
|
|
|
return (fd >= 0);
|
2007-09-11 05:03:15 +02:00
|
|
|
}
|
|
|
|
|
bundle: remove "ref_list" in favor of string-list.c API
Move away from the "struct ref_list" in bundle.c in favor of the
almost identical string-list.c API.
That API fits this use-case perfectly, but did not exist in its
current form when this code was added in 2e0afafebd (Add git-bundle:
move objects and references by archive, 2007-02-22), with hindsight we
could have used the path-list API, which later got renamed to
string-list. See 8fd2cb4069 (Extract helper bits from
c-merge-recursive work, 2006-07-25)
We need to change "name" to "string" and "oid" to "util" to make this
conversion, but other than that the APIs are pretty much identical for
what bundle.c made use of.
Let's also replace the memset(..,0,...) pattern with a more idiomatic
"INIT" macro, and finally add a *_release() function so to free the
allocated memory.
Before this the add_to_ref_list() would leak memory, now e.g. "bundle
list-heads" reports no memory leaks at all under valgrind.
In the bundle_header_init() function we're using a clever trick to
memcpy() what we'd get from the corresponding
BUNDLE_HEADER_INIT. There is a concurrent series to make use of that
pattern more generally, see [1].
1. https://lore.kernel.org/git/cover-0.5-00000000000-20210701T104855Z-avarab@gmail.com/
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Acked-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2021-07-02 11:57:32 +02:00
|
|
|
static int list_refs(struct string_list *r, int argc, const char **argv)
|
2007-09-11 05:03:15 +02:00
|
|
|
{
|
|
|
|
int i;
|
|
|
|
|
|
|
|
for (i = 0; i < r->nr; i++) {
|
2021-07-02 11:57:31 +02:00
|
|
|
struct object_id *oid;
|
|
|
|
const char *name;
|
|
|
|
|
2007-09-11 05:03:15 +02:00
|
|
|
if (argc > 1) {
|
|
|
|
int j;
|
|
|
|
for (j = 1; j < argc; j++)
|
bundle: remove "ref_list" in favor of string-list.c API
Move away from the "struct ref_list" in bundle.c in favor of the
almost identical string-list.c API.
That API fits this use-case perfectly, but did not exist in its
current form when this code was added in 2e0afafebd (Add git-bundle:
move objects and references by archive, 2007-02-22), with hindsight we
could have used the path-list API, which later got renamed to
string-list. See 8fd2cb4069 (Extract helper bits from
c-merge-recursive work, 2006-07-25)
We need to change "name" to "string" and "oid" to "util" to make this
conversion, but other than that the APIs are pretty much identical for
what bundle.c made use of.
Let's also replace the memset(..,0,...) pattern with a more idiomatic
"INIT" macro, and finally add a *_release() function so to free the
allocated memory.
Before this the add_to_ref_list() would leak memory, now e.g. "bundle
list-heads" reports no memory leaks at all under valgrind.
In the bundle_header_init() function we're using a clever trick to
memcpy() what we'd get from the corresponding
BUNDLE_HEADER_INIT. There is a concurrent series to make use of that
pattern more generally, see [1].
1. https://lore.kernel.org/git/cover-0.5-00000000000-20210701T104855Z-avarab@gmail.com/
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Acked-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2021-07-02 11:57:32 +02:00
|
|
|
if (!strcmp(r->items[i].string, argv[j]))
|
2007-09-11 05:03:15 +02:00
|
|
|
break;
|
|
|
|
if (j == argc)
|
|
|
|
continue;
|
|
|
|
}
|
2021-07-02 11:57:31 +02:00
|
|
|
|
bundle: remove "ref_list" in favor of string-list.c API
Move away from the "struct ref_list" in bundle.c in favor of the
almost identical string-list.c API.
That API fits this use-case perfectly, but did not exist in its
current form when this code was added in 2e0afafebd (Add git-bundle:
move objects and references by archive, 2007-02-22), with hindsight we
could have used the path-list API, which later got renamed to
string-list. See 8fd2cb4069 (Extract helper bits from
c-merge-recursive work, 2006-07-25)
We need to change "name" to "string" and "oid" to "util" to make this
conversion, but other than that the APIs are pretty much identical for
what bundle.c made use of.
Let's also replace the memset(..,0,...) pattern with a more idiomatic
"INIT" macro, and finally add a *_release() function so to free the
allocated memory.
Before this the add_to_ref_list() would leak memory, now e.g. "bundle
list-heads" reports no memory leaks at all under valgrind.
In the bundle_header_init() function we're using a clever trick to
memcpy() what we'd get from the corresponding
BUNDLE_HEADER_INIT. There is a concurrent series to make use of that
pattern more generally, see [1].
1. https://lore.kernel.org/git/cover-0.5-00000000000-20210701T104855Z-avarab@gmail.com/
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Acked-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2021-07-02 11:57:32 +02:00
|
|
|
oid = r->items[i].util;
|
|
|
|
name = r->items[i].string;
|
2021-07-02 11:57:31 +02:00
|
|
|
printf("%s %s\n", oid_to_hex(oid), name);
|
2007-09-11 05:03:15 +02:00
|
|
|
}
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2014-03-25 14:23:26 +01:00
|
|
|
/* Remember to update object flag allocation in object.h */
|
2007-09-11 05:03:15 +02:00
|
|
|
#define PREREQ_MARK (1u<<16)
|
|
|
|
|
2018-11-10 06:49:01 +01:00
|
|
|
int verify_bundle(struct repository *r,
|
|
|
|
struct bundle_header *header,
|
|
|
|
int verbose)
|
2007-09-11 05:03:15 +02:00
|
|
|
{
|
|
|
|
/*
|
|
|
|
* Do fast check, then if any prereqs are missing then go line by line
|
|
|
|
* to be verbose about the errors
|
|
|
|
*/
|
bundle: remove "ref_list" in favor of string-list.c API
Move away from the "struct ref_list" in bundle.c in favor of the
almost identical string-list.c API.
That API fits this use-case perfectly, but did not exist in its
current form when this code was added in 2e0afafebd (Add git-bundle:
move objects and references by archive, 2007-02-22), with hindsight we
could have used the path-list API, which later got renamed to
string-list. See 8fd2cb4069 (Extract helper bits from
c-merge-recursive work, 2006-07-25)
We need to change "name" to "string" and "oid" to "util" to make this
conversion, but other than that the APIs are pretty much identical for
what bundle.c made use of.
Let's also replace the memset(..,0,...) pattern with a more idiomatic
"INIT" macro, and finally add a *_release() function so to free the
allocated memory.
Before this the add_to_ref_list() would leak memory, now e.g. "bundle
list-heads" reports no memory leaks at all under valgrind.
In the bundle_header_init() function we're using a clever trick to
memcpy() what we'd get from the corresponding
BUNDLE_HEADER_INIT. There is a concurrent series to make use of that
pattern more generally, see [1].
1. https://lore.kernel.org/git/cover-0.5-00000000000-20210701T104855Z-avarab@gmail.com/
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Acked-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2021-07-02 11:57:32 +02:00
|
|
|
struct string_list *p = &header->prerequisites;
|
2007-09-11 05:03:15 +02:00
|
|
|
struct rev_info revs;
|
2009-05-21 11:32:44 +02:00
|
|
|
const char *argv[] = {NULL, "--all", NULL};
|
2007-09-11 05:03:15 +02:00
|
|
|
struct commit *commit;
|
|
|
|
int i, ret = 0, req_nr;
|
2012-04-23 14:30:30 +02:00
|
|
|
const char *message = _("Repository lacks these prerequisite commits:");
|
2007-09-11 05:03:15 +02:00
|
|
|
|
2019-05-27 21:59:14 +02:00
|
|
|
if (!r || !r->objects || !r->objects->odb)
|
|
|
|
return error(_("need a repository to verify a bundle"));
|
|
|
|
|
2018-11-10 06:49:01 +01:00
|
|
|
repo_init_revisions(r, &revs, NULL);
|
2007-09-11 05:03:15 +02:00
|
|
|
for (i = 0; i < p->nr; i++) {
|
bundle: remove "ref_list" in favor of string-list.c API
Move away from the "struct ref_list" in bundle.c in favor of the
almost identical string-list.c API.
That API fits this use-case perfectly, but did not exist in its
current form when this code was added in 2e0afafebd (Add git-bundle:
move objects and references by archive, 2007-02-22), with hindsight we
could have used the path-list API, which later got renamed to
string-list. See 8fd2cb4069 (Extract helper bits from
c-merge-recursive work, 2006-07-25)
We need to change "name" to "string" and "oid" to "util" to make this
conversion, but other than that the APIs are pretty much identical for
what bundle.c made use of.
Let's also replace the memset(..,0,...) pattern with a more idiomatic
"INIT" macro, and finally add a *_release() function so to free the
allocated memory.
Before this the add_to_ref_list() would leak memory, now e.g. "bundle
list-heads" reports no memory leaks at all under valgrind.
In the bundle_header_init() function we're using a clever trick to
memcpy() what we'd get from the corresponding
BUNDLE_HEADER_INIT. There is a concurrent series to make use of that
pattern more generally, see [1].
1. https://lore.kernel.org/git/cover-0.5-00000000000-20210701T104855Z-avarab@gmail.com/
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Acked-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2021-07-02 11:57:32 +02:00
|
|
|
struct string_list_item *e = p->items + i;
|
|
|
|
const char *name = e->string;
|
|
|
|
struct object_id *oid = e->util;
|
2021-07-02 11:57:31 +02:00
|
|
|
struct object *o = parse_object(r, oid);
|
2007-09-11 05:03:15 +02:00
|
|
|
if (o) {
|
|
|
|
o->flags |= PREREQ_MARK;
|
2021-07-02 11:57:31 +02:00
|
|
|
add_pending_object(&revs, o, name);
|
2007-09-11 05:03:15 +02:00
|
|
|
continue;
|
|
|
|
}
|
|
|
|
if (++ret == 1)
|
2008-11-10 22:07:52 +01:00
|
|
|
error("%s", message);
|
2021-07-02 11:57:31 +02:00
|
|
|
error("%s %s", oid_to_hex(oid), name);
|
2007-09-11 05:03:15 +02:00
|
|
|
}
|
|
|
|
if (revs.pending.nr != p->nr)
|
|
|
|
return ret;
|
|
|
|
req_nr = revs.pending.nr;
|
|
|
|
setup_revisions(2, argv, &revs, NULL);
|
|
|
|
|
2008-02-18 08:31:56 +01:00
|
|
|
if (prepare_revision_walk(&revs))
|
2012-04-23 14:30:30 +02:00
|
|
|
die(_("revision walk setup failed"));
|
2007-09-11 05:03:15 +02:00
|
|
|
|
|
|
|
i = req_nr;
|
|
|
|
while (i && (commit = get_revision(&revs)))
|
|
|
|
if (commit->object.flags & PREREQ_MARK)
|
|
|
|
i--;
|
|
|
|
|
2017-12-25 18:46:14 +01:00
|
|
|
for (i = 0; i < p->nr; i++) {
|
bundle: remove "ref_list" in favor of string-list.c API
Move away from the "struct ref_list" in bundle.c in favor of the
almost identical string-list.c API.
That API fits this use-case perfectly, but did not exist in its
current form when this code was added in 2e0afafebd (Add git-bundle:
move objects and references by archive, 2007-02-22), with hindsight we
could have used the path-list API, which later got renamed to
string-list. See 8fd2cb4069 (Extract helper bits from
c-merge-recursive work, 2006-07-25)
We need to change "name" to "string" and "oid" to "util" to make this
conversion, but other than that the APIs are pretty much identical for
what bundle.c made use of.
Let's also replace the memset(..,0,...) pattern with a more idiomatic
"INIT" macro, and finally add a *_release() function so to free the
allocated memory.
Before this the add_to_ref_list() would leak memory, now e.g. "bundle
list-heads" reports no memory leaks at all under valgrind.
In the bundle_header_init() function we're using a clever trick to
memcpy() what we'd get from the corresponding
BUNDLE_HEADER_INIT. There is a concurrent series to make use of that
pattern more generally, see [1].
1. https://lore.kernel.org/git/cover-0.5-00000000000-20210701T104855Z-avarab@gmail.com/
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Acked-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2021-07-02 11:57:32 +02:00
|
|
|
struct string_list_item *e = p->items + i;
|
|
|
|
const char *name = e->string;
|
|
|
|
const struct object_id *oid = e->util;
|
2021-07-02 11:57:31 +02:00
|
|
|
struct object *o = parse_object(r, oid);
|
2017-12-25 18:46:14 +01:00
|
|
|
assert(o); /* otherwise we'd have returned early */
|
|
|
|
if (o->flags & SHOWN)
|
|
|
|
continue;
|
|
|
|
if (++ret == 1)
|
|
|
|
error("%s", message);
|
2021-07-02 11:57:31 +02:00
|
|
|
error("%s %s", oid_to_hex(oid), name);
|
2017-12-25 18:46:14 +01:00
|
|
|
}
|
2007-09-11 05:03:15 +02:00
|
|
|
|
2017-09-23 01:34:51 +02:00
|
|
|
/* Clean up objects used, as they will be reused. */
|
2017-12-25 18:46:14 +01:00
|
|
|
for (i = 0; i < p->nr; i++) {
|
bundle: remove "ref_list" in favor of string-list.c API
Move away from the "struct ref_list" in bundle.c in favor of the
almost identical string-list.c API.
That API fits this use-case perfectly, but did not exist in its
current form when this code was added in 2e0afafebd (Add git-bundle:
move objects and references by archive, 2007-02-22), with hindsight we
could have used the path-list API, which later got renamed to
string-list. See 8fd2cb4069 (Extract helper bits from
c-merge-recursive work, 2006-07-25)
We need to change "name" to "string" and "oid" to "util" to make this
conversion, but other than that the APIs are pretty much identical for
what bundle.c made use of.
Let's also replace the memset(..,0,...) pattern with a more idiomatic
"INIT" macro, and finally add a *_release() function so to free the
allocated memory.
Before this the add_to_ref_list() would leak memory, now e.g. "bundle
list-heads" reports no memory leaks at all under valgrind.
In the bundle_header_init() function we're using a clever trick to
memcpy() what we'd get from the corresponding
BUNDLE_HEADER_INIT. There is a concurrent series to make use of that
pattern more generally, see [1].
1. https://lore.kernel.org/git/cover-0.5-00000000000-20210701T104855Z-avarab@gmail.com/
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Acked-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2021-07-02 11:57:32 +02:00
|
|
|
struct string_list_item *e = p->items + i;
|
|
|
|
struct object_id *oid = e->util;
|
2021-07-02 11:57:31 +02:00
|
|
|
commit = lookup_commit_reference_gently(r, oid, 1);
|
2017-12-25 18:46:14 +01:00
|
|
|
if (commit)
|
|
|
|
clear_commit_marks(commit, ALL_REV_FLAGS);
|
|
|
|
}
|
2007-09-11 05:03:15 +02:00
|
|
|
|
|
|
|
if (verbose) {
|
bundle: remove "ref_list" in favor of string-list.c API
Move away from the "struct ref_list" in bundle.c in favor of the
almost identical string-list.c API.
That API fits this use-case perfectly, but did not exist in its
current form when this code was added in 2e0afafebd (Add git-bundle:
move objects and references by archive, 2007-02-22), with hindsight we
could have used the path-list API, which later got renamed to
string-list. See 8fd2cb4069 (Extract helper bits from
c-merge-recursive work, 2006-07-25)
We need to change "name" to "string" and "oid" to "util" to make this
conversion, but other than that the APIs are pretty much identical for
what bundle.c made use of.
Let's also replace the memset(..,0,...) pattern with a more idiomatic
"INIT" macro, and finally add a *_release() function so to free the
allocated memory.
Before this the add_to_ref_list() would leak memory, now e.g. "bundle
list-heads" reports no memory leaks at all under valgrind.
In the bundle_header_init() function we're using a clever trick to
memcpy() what we'd get from the corresponding
BUNDLE_HEADER_INIT. There is a concurrent series to make use of that
pattern more generally, see [1].
1. https://lore.kernel.org/git/cover-0.5-00000000000-20210701T104855Z-avarab@gmail.com/
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Acked-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2021-07-02 11:57:32 +02:00
|
|
|
struct string_list *r;
|
2007-09-11 05:03:15 +02:00
|
|
|
|
|
|
|
r = &header->references;
|
2013-03-08 19:01:26 +01:00
|
|
|
printf_ln(Q_("The bundle contains this ref:",
|
|
|
|
"The bundle contains these %d refs:",
|
2012-04-23 14:30:30 +02:00
|
|
|
r->nr),
|
|
|
|
r->nr);
|
2007-09-11 05:03:15 +02:00
|
|
|
list_refs(r, 0, NULL);
|
2013-03-07 01:56:35 +01:00
|
|
|
r = &header->prerequisites;
|
2012-06-04 20:51:13 +02:00
|
|
|
if (!r->nr) {
|
|
|
|
printf_ln(_("The bundle records a complete history."));
|
|
|
|
} else {
|
2013-03-08 19:01:26 +01:00
|
|
|
printf_ln(Q_("The bundle requires this ref:",
|
|
|
|
"The bundle requires these %d refs:",
|
2012-06-04 20:51:13 +02:00
|
|
|
r->nr),
|
|
|
|
r->nr);
|
|
|
|
list_refs(r, 0, NULL);
|
|
|
|
}
|
2007-09-11 05:03:15 +02:00
|
|
|
}
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
|
|
|
int list_bundle_refs(struct bundle_header *header, int argc, const char **argv)
|
|
|
|
{
|
|
|
|
return list_refs(&header->references, argc, argv);
|
|
|
|
}
|
|
|
|
|
2009-01-02 19:08:46 +01:00
|
|
|
static int is_tag_in_date_range(struct object *tag, struct rev_info *revs)
|
|
|
|
{
|
|
|
|
unsigned long size;
|
|
|
|
enum object_type type;
|
2014-10-04 00:40:24 +02:00
|
|
|
char *buf = NULL, *line, *lineend;
|
2017-04-26 21:29:31 +02:00
|
|
|
timestamp_t date;
|
2014-10-04 00:40:24 +02:00
|
|
|
int result = 1;
|
2009-01-02 19:08:46 +01:00
|
|
|
|
|
|
|
if (revs->max_age == -1 && revs->min_age == -1)
|
2014-10-04 00:40:24 +02:00
|
|
|
goto out;
|
2009-01-02 19:08:46 +01:00
|
|
|
|
sha1_file: convert read_sha1_file to struct object_id
Convert read_sha1_file to take a pointer to struct object_id and rename
it read_object_file. Do the same for read_sha1_file_extended.
Convert one use in grep.c to use the new function without any other code
change, since the pointer being passed is a void pointer that is already
initialized with a pointer to struct object_id. Update the declaration
and definitions of the modified functions, and apply the following
semantic patch to convert the remaining callers:
@@
expression E1, E2, E3;
@@
- read_sha1_file(E1.hash, E2, E3)
+ read_object_file(&E1, E2, E3)
@@
expression E1, E2, E3;
@@
- read_sha1_file(E1->hash, E2, E3)
+ read_object_file(E1, E2, E3)
@@
expression E1, E2, E3, E4;
@@
- read_sha1_file_extended(E1.hash, E2, E3, E4)
+ read_object_file_extended(&E1, E2, E3, E4)
@@
expression E1, E2, E3, E4;
@@
- read_sha1_file_extended(E1->hash, E2, E3, E4)
+ read_object_file_extended(E1, E2, E3, E4)
Signed-off-by: brian m. carlson <sandals@crustytoothpaste.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2018-03-12 03:27:53 +01:00
|
|
|
buf = read_object_file(&tag->oid, &type, &size);
|
2009-01-02 19:08:46 +01:00
|
|
|
if (!buf)
|
2014-10-04 00:40:24 +02:00
|
|
|
goto out;
|
2009-01-02 19:08:46 +01:00
|
|
|
line = memmem(buf, size, "\ntagger ", 8);
|
|
|
|
if (!line++)
|
2014-10-04 00:40:24 +02:00
|
|
|
goto out;
|
2014-08-02 10:39:06 +02:00
|
|
|
lineend = memchr(line, '\n', buf + size - line);
|
|
|
|
line = memchr(line, '>', lineend ? lineend - line : buf + size - line);
|
2009-01-02 19:08:46 +01:00
|
|
|
if (!line++)
|
2014-10-04 00:40:24 +02:00
|
|
|
goto out;
|
2017-04-21 12:45:44 +02:00
|
|
|
date = parse_timestamp(line, NULL, 10);
|
2014-10-04 00:40:24 +02:00
|
|
|
result = (revs->max_age == -1 || revs->max_age < date) &&
|
2009-01-02 19:08:46 +01:00
|
|
|
(revs->min_age == -1 || revs->min_age > date);
|
2014-10-04 00:40:24 +02:00
|
|
|
out:
|
|
|
|
free(buf);
|
|
|
|
return result;
|
2009-01-02 19:08:46 +01:00
|
|
|
}
|
|
|
|
|
2015-08-10 11:47:37 +02:00
|
|
|
|
bundle: dup() output descriptor closer to point-of-use
When writing a bundle to a file, the bundle code actually creates
"your.bundle.lock" using our lockfile interface. We feed that output
descriptor to a child git-pack-objects via run-command, which has the
quirk that it closes the output descriptor in the parent.
To avoid confusing the lockfile code (which still thinks the descriptor
is valid), we dup() it, and operate on the duplicate.
However, this has a confusing side effect: after the dup() but before we
call pack-objects, we have _two_ descriptors open to the lockfile. If we
call die() during that time, the lockfile code will try to clean up the
partially-written file. It knows to close() the file before unlinking,
since on some platforms (i.e., Windows) the open file would block the
deletion. But it doesn't know about the duplicate descriptor. On
Windows, triggering an error at the right part of the code will result
in the cleanup failing and the lockfile being left in the filesystem.
We can solve this by moving the dup() much closer to start_command(),
shrinking the window in which we have the second descriptor open. It's
easy to place this in such a way that no die() is possible. We could
still die due to a signal in the exact wrong moment, but we already
tolerate races there (e.g., a signal could come before we manage to put
the file on the cleanup list in the first place).
As a bonus, this shields create_bundle() itself from the duplicate-fd
trick, and we can simplify its error handling (note that the lock
rollback now happens unconditionally, but that's OK; it's a noop if we
didn't open the lock in the first place).
The included test uses an empty bundle to cause a failure at the right
spot in the code, because that's easy to trigger (the other likely
errors are write() problems like ENOSPC). Note that it would already
pass on non-Windows systems (because they are happy to unlink an
already-open file).
Based-on-a-patch-by: Gaël Lhez <gael.lhez@gmail.com>
Signed-off-by: Jeff King <peff@peff.net>
Tested-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2018-11-16 10:43:59 +01:00
|
|
|
/* Write the pack data to bundle_fd */
|
2020-07-28 22:24:53 +02:00
|
|
|
static int write_pack_data(int bundle_fd, struct rev_info *revs, struct strvec *pack_options)
|
2014-10-30 18:45:41 +01:00
|
|
|
{
|
|
|
|
struct child_process pack_objects = CHILD_PROCESS_INIT;
|
|
|
|
int i;
|
|
|
|
|
2020-07-28 22:24:53 +02:00
|
|
|
strvec_pushl(&pack_objects.args,
|
strvec: fix indentation in renamed calls
Code which split an argv_array call across multiple lines, like:
argv_array_pushl(&args, "one argument",
"another argument", "and more",
NULL);
was recently mechanically renamed to use strvec, which results in
mis-matched indentation like:
strvec_pushl(&args, "one argument",
"another argument", "and more",
NULL);
Let's fix these up to align the arguments with the opening paren. I did
this manually by sifting through the results of:
git jump grep 'strvec_.*,$'
and liberally applying my editor's auto-format. Most of the changes are
of the form shown above, though I also normalized a few that had
originally used a single-tab indentation (rather than our usual style of
aligning with the open paren). I also rewrapped a couple of obvious
cases (e.g., where previously too-long lines became short enough to fit
on one), but I wasn't aggressive about it. In cases broken to three or
more lines, the grouping of arguments is sometimes meaningful, and it
wasn't worth my time or reviewer time to ponder each case individually.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2020-07-28 22:26:31 +02:00
|
|
|
"pack-objects",
|
|
|
|
"--stdout", "--thin", "--delta-base-offset",
|
|
|
|
NULL);
|
2020-07-29 02:37:20 +02:00
|
|
|
strvec_pushv(&pack_objects.args, pack_options->v);
|
2014-10-30 18:45:41 +01:00
|
|
|
pack_objects.in = -1;
|
|
|
|
pack_objects.out = bundle_fd;
|
|
|
|
pack_objects.git_cmd = 1;
|
bundle: dup() output descriptor closer to point-of-use
When writing a bundle to a file, the bundle code actually creates
"your.bundle.lock" using our lockfile interface. We feed that output
descriptor to a child git-pack-objects via run-command, which has the
quirk that it closes the output descriptor in the parent.
To avoid confusing the lockfile code (which still thinks the descriptor
is valid), we dup() it, and operate on the duplicate.
However, this has a confusing side effect: after the dup() but before we
call pack-objects, we have _two_ descriptors open to the lockfile. If we
call die() during that time, the lockfile code will try to clean up the
partially-written file. It knows to close() the file before unlinking,
since on some platforms (i.e., Windows) the open file would block the
deletion. But it doesn't know about the duplicate descriptor. On
Windows, triggering an error at the right part of the code will result
in the cleanup failing and the lockfile being left in the filesystem.
We can solve this by moving the dup() much closer to start_command(),
shrinking the window in which we have the second descriptor open. It's
easy to place this in such a way that no die() is possible. We could
still die due to a signal in the exact wrong moment, but we already
tolerate races there (e.g., a signal could come before we manage to put
the file on the cleanup list in the first place).
As a bonus, this shields create_bundle() itself from the duplicate-fd
trick, and we can simplify its error handling (note that the lock
rollback now happens unconditionally, but that's OK; it's a noop if we
didn't open the lock in the first place).
The included test uses an empty bundle to cause a failure at the right
spot in the code, because that's easy to trigger (the other likely
errors are write() problems like ENOSPC). Note that it would already
pass on non-Windows systems (because they are happy to unlink an
already-open file).
Based-on-a-patch-by: Gaël Lhez <gael.lhez@gmail.com>
Signed-off-by: Jeff King <peff@peff.net>
Tested-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2018-11-16 10:43:59 +01:00
|
|
|
|
|
|
|
/*
|
|
|
|
* start_command() will close our descriptor if it's >1. Duplicate it
|
|
|
|
* to avoid surprising the caller.
|
|
|
|
*/
|
|
|
|
if (pack_objects.out > 1) {
|
|
|
|
pack_objects.out = dup(pack_objects.out);
|
|
|
|
if (pack_objects.out < 0) {
|
|
|
|
error_errno(_("unable to dup bundle descriptor"));
|
|
|
|
child_process_clear(&pack_objects);
|
|
|
|
return -1;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2014-10-30 18:45:41 +01:00
|
|
|
if (start_command(&pack_objects))
|
|
|
|
return error(_("Could not spawn pack-objects"));
|
|
|
|
|
|
|
|
for (i = 0; i < revs->pending.nr; i++) {
|
|
|
|
struct object *object = revs->pending.objects[i].item;
|
|
|
|
if (object->flags & UNINTERESTING)
|
|
|
|
write_or_die(pack_objects.in, "^", 1);
|
2019-08-18 22:04:11 +02:00
|
|
|
write_or_die(pack_objects.in, oid_to_hex(&object->oid), the_hash_algo->hexsz);
|
2014-10-30 18:45:41 +01:00
|
|
|
write_or_die(pack_objects.in, "\n", 1);
|
|
|
|
}
|
|
|
|
close(pack_objects.in);
|
|
|
|
if (finish_command(&pack_objects))
|
|
|
|
return error(_("pack-objects died"));
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2014-10-30 22:35:24 +01:00
|
|
|
/*
|
|
|
|
* Write out bundle refs based on the tips already
|
|
|
|
* parsed into revs.pending. As a side effect, may
|
|
|
|
* manipulate revs.pending to include additional
|
|
|
|
* necessary objects (like tags).
|
|
|
|
*
|
|
|
|
* Returns the number of refs written, or negative
|
|
|
|
* on error.
|
|
|
|
*/
|
|
|
|
static int write_bundle_refs(int bundle_fd, struct rev_info *revs)
|
2014-10-30 19:01:37 +01:00
|
|
|
{
|
2014-10-30 22:35:24 +01:00
|
|
|
int i;
|
|
|
|
int ref_count = 0;
|
2009-01-18 07:27:08 +01:00
|
|
|
|
2014-10-30 22:35:24 +01:00
|
|
|
for (i = 0; i < revs->pending.nr; i++) {
|
|
|
|
struct object_array_entry *e = revs->pending.objects + i;
|
2015-11-10 03:22:28 +01:00
|
|
|
struct object_id oid;
|
2007-09-11 05:03:15 +02:00
|
|
|
char *ref;
|
2007-11-23 01:51:18 +01:00
|
|
|
const char *display_ref;
|
|
|
|
int flag;
|
2007-09-11 05:03:15 +02:00
|
|
|
|
|
|
|
if (e->item->flags & UNINTERESTING)
|
|
|
|
continue;
|
2020-09-02 00:28:09 +02:00
|
|
|
if (dwim_ref(e->name, strlen(e->name), &oid, &ref, 0) != 1)
|
2015-03-11 00:51:48 +01:00
|
|
|
goto skip_write_ref;
|
2017-10-16 00:06:56 +02:00
|
|
|
if (read_ref_full(e->name, RESOLVE_REF_READING, &oid, &flag))
|
2007-11-23 01:51:18 +01:00
|
|
|
flag = 0;
|
|
|
|
display_ref = (flag & REF_ISSYMREF) ? e->name : ref;
|
|
|
|
|
2009-01-02 19:08:46 +01:00
|
|
|
if (e->item->type == OBJ_TAG &&
|
2014-10-30 22:35:24 +01:00
|
|
|
!is_tag_in_date_range(e->item, revs)) {
|
2009-01-02 19:08:46 +01:00
|
|
|
e->item->flags |= UNINTERESTING;
|
2015-03-11 00:51:48 +01:00
|
|
|
goto skip_write_ref;
|
2009-01-02 19:08:46 +01:00
|
|
|
}
|
|
|
|
|
2007-09-11 05:03:15 +02:00
|
|
|
/*
|
|
|
|
* Make sure the refs we wrote out is correct; --max-count and
|
|
|
|
* other limiting options could have prevented all the tips
|
|
|
|
* from getting output.
|
|
|
|
*
|
|
|
|
* Non commit objects such as tags and blobs do not have
|
|
|
|
* this issue as they are not affected by those extra
|
|
|
|
* constraints.
|
|
|
|
*/
|
|
|
|
if (!(e->item->flags & SHOWN) && e->item->type == OBJ_COMMIT) {
|
2012-04-23 14:30:30 +02:00
|
|
|
warning(_("ref '%s' is excluded by the rev-list options"),
|
2007-09-11 05:03:15 +02:00
|
|
|
e->name);
|
2015-03-11 00:51:48 +01:00
|
|
|
goto skip_write_ref;
|
2007-09-11 05:03:15 +02:00
|
|
|
}
|
|
|
|
/*
|
|
|
|
* If you run "git bundle create bndl v1.0..v2.0", the
|
|
|
|
* name of the positive ref is "v2.0" but that is the
|
|
|
|
* commit that is referenced by the tag, and not the tag
|
|
|
|
* itself.
|
|
|
|
*/
|
2018-08-28 23:22:48 +02:00
|
|
|
if (!oideq(&oid, &e->item->oid)) {
|
2007-09-11 05:03:15 +02:00
|
|
|
/*
|
|
|
|
* Is this the positive end of a range expressed
|
|
|
|
* in terms of a tag (e.g. v2.0 from the range
|
|
|
|
* "v1.0..v2.0")?
|
|
|
|
*/
|
2018-11-10 06:49:01 +01:00
|
|
|
struct commit *one = lookup_commit_reference(revs->repo, &oid);
|
2007-09-11 05:03:15 +02:00
|
|
|
struct object *obj;
|
|
|
|
|
|
|
|
if (e->item == &(one->object)) {
|
|
|
|
/*
|
|
|
|
* Need to include e->name as an
|
|
|
|
* independent ref to the pack-objects
|
|
|
|
* input, so that the tag is included
|
|
|
|
* in the output; otherwise we would
|
|
|
|
* end up triggering "empty bundle"
|
|
|
|
* error.
|
|
|
|
*/
|
object: convert parse_object* to take struct object_id
Make parse_object, parse_object_or_die, and parse_object_buffer take a
pointer to struct object_id. Remove the temporary variables inserted
earlier, since they are no longer necessary. Transform all of the
callers using the following semantic patch:
@@
expression E1;
@@
- parse_object(E1.hash)
+ parse_object(&E1)
@@
expression E1;
@@
- parse_object(E1->hash)
+ parse_object(E1)
@@
expression E1, E2;
@@
- parse_object_or_die(E1.hash, E2)
+ parse_object_or_die(&E1, E2)
@@
expression E1, E2;
@@
- parse_object_or_die(E1->hash, E2)
+ parse_object_or_die(E1, E2)
@@
expression E1, E2, E3, E4, E5;
@@
- parse_object_buffer(E1.hash, E2, E3, E4, E5)
+ parse_object_buffer(&E1, E2, E3, E4, E5)
@@
expression E1, E2, E3, E4, E5;
@@
- parse_object_buffer(E1->hash, E2, E3, E4, E5)
+ parse_object_buffer(E1, E2, E3, E4, E5)
Signed-off-by: brian m. carlson <sandals@crustytoothpaste.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2017-05-07 00:10:38 +02:00
|
|
|
obj = parse_object_or_die(&oid, e->name);
|
2007-09-11 05:03:15 +02:00
|
|
|
obj->flags |= SHOWN;
|
2014-10-30 22:35:24 +01:00
|
|
|
add_pending_object(revs, obj, e->name);
|
2007-09-11 05:03:15 +02:00
|
|
|
}
|
2015-03-11 00:51:48 +01:00
|
|
|
goto skip_write_ref;
|
2007-09-11 05:03:15 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
ref_count++;
|
2019-08-18 22:04:11 +02:00
|
|
|
write_or_die(bundle_fd, oid_to_hex(&e->item->oid), the_hash_algo->hexsz);
|
2007-09-11 05:03:15 +02:00
|
|
|
write_or_die(bundle_fd, " ", 1);
|
2007-11-23 01:51:18 +01:00
|
|
|
write_or_die(bundle_fd, display_ref, strlen(display_ref));
|
2007-09-11 05:03:15 +02:00
|
|
|
write_or_die(bundle_fd, "\n", 1);
|
2015-03-11 00:51:48 +01:00
|
|
|
skip_write_ref:
|
2007-09-11 05:03:15 +02:00
|
|
|
free(ref);
|
|
|
|
}
|
|
|
|
|
|
|
|
/* end header */
|
|
|
|
write_or_die(bundle_fd, "\n", 1);
|
2014-10-30 22:35:24 +01:00
|
|
|
return ref_count;
|
|
|
|
}
|
|
|
|
|
bundle: arguments can be read from stdin
In order to create an incremental bundle, we need to pass many arguments
to let git-bundle ignore some already packed commits. It will be more
convenient to pass args via stdin. But the current implementation does
not allow us to do this.
This is because args are parsed twice when creating bundle. The first
time for parsing args is in `compute_and_write_prerequisites()` by
running `git-rev-list` command to write prerequisites in bundle file,
and stdin is consumed in this step if "--stdin" option is provided for
`git-bundle`. Later nothing can be read from stdin when running
`setup_revisions()` in `create_bundle()`.
The solution is to parse args once by removing the entire function
`compute_and_write_prerequisites()` and then calling function
`setup_revisions()`. In order to write prerequisites for bundle, will
call `prepare_revision_walk()` and `traverse_commit_list()`. But after
calling `prepare_revision_walk()`, the object array `revs.pending` is
left empty, and the following steps could not work properly with the
empty object array (`revs.pending`). Therefore, make a copy of `revs`
to `revs_copy` for later use right after calling `setup_revisions()`.
The copy of `revs_copy` is not a deep copy, it shares the same objects
with `revs`. The object array of `revs` has been cleared, but objects
themselves are still kept. Flags of objects may change after calling
`prepare_revision_walk()`, we can use these changed flags without
calling the `git rev-list` command and parsing its output like the
former implementation.
Also add testcases for git bundle in t6020, which read args from stdin.
Signed-off-by: Jiang Xin <zhiyou.jx@alibaba-inc.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2021-01-12 03:27:03 +01:00
|
|
|
struct bundle_prerequisites_info {
|
|
|
|
struct object_array *pending;
|
|
|
|
int fd;
|
|
|
|
};
|
|
|
|
|
|
|
|
static void write_bundle_prerequisites(struct commit *commit, void *data)
|
|
|
|
{
|
|
|
|
struct bundle_prerequisites_info *bpi = data;
|
|
|
|
struct object *object;
|
|
|
|
struct pretty_print_context ctx = { 0 };
|
|
|
|
struct strbuf buf = STRBUF_INIT;
|
|
|
|
|
|
|
|
if (!(commit->object.flags & BOUNDARY))
|
|
|
|
return;
|
|
|
|
strbuf_addf(&buf, "-%s ", oid_to_hex(&commit->object.oid));
|
|
|
|
write_or_die(bpi->fd, buf.buf, buf.len);
|
|
|
|
|
|
|
|
ctx.fmt = CMIT_FMT_ONELINE;
|
|
|
|
ctx.output_encoding = get_log_output_encoding();
|
|
|
|
strbuf_reset(&buf);
|
|
|
|
pretty_print_commit(&ctx, commit, &buf);
|
|
|
|
strbuf_trim(&buf);
|
|
|
|
|
|
|
|
object = (struct object *)commit;
|
|
|
|
object->flags |= UNINTERESTING;
|
|
|
|
add_object_array_with_path(object, buf.buf, bpi->pending, S_IFINVALID,
|
|
|
|
NULL);
|
|
|
|
strbuf_addch(&buf, '\n');
|
|
|
|
write_or_die(bpi->fd, buf.buf, buf.len);
|
|
|
|
strbuf_release(&buf);
|
|
|
|
}
|
|
|
|
|
2019-01-24 14:11:51 +01:00
|
|
|
int create_bundle(struct repository *r, const char *path,
|
2020-08-12 03:04:11 +02:00
|
|
|
int argc, const char **argv, struct strvec *pack_options, int version)
|
2014-10-30 22:35:24 +01:00
|
|
|
{
|
2018-05-09 22:55:38 +02:00
|
|
|
struct lock_file lock = LOCK_INIT;
|
2014-10-30 22:35:24 +01:00
|
|
|
int bundle_fd = -1;
|
|
|
|
int bundle_to_stdout;
|
|
|
|
int ref_count = 0;
|
bundle: arguments can be read from stdin
In order to create an incremental bundle, we need to pass many arguments
to let git-bundle ignore some already packed commits. It will be more
convenient to pass args via stdin. But the current implementation does
not allow us to do this.
This is because args are parsed twice when creating bundle. The first
time for parsing args is in `compute_and_write_prerequisites()` by
running `git-rev-list` command to write prerequisites in bundle file,
and stdin is consumed in this step if "--stdin" option is provided for
`git-bundle`. Later nothing can be read from stdin when running
`setup_revisions()` in `create_bundle()`.
The solution is to parse args once by removing the entire function
`compute_and_write_prerequisites()` and then calling function
`setup_revisions()`. In order to write prerequisites for bundle, will
call `prepare_revision_walk()` and `traverse_commit_list()`. But after
calling `prepare_revision_walk()`, the object array `revs.pending` is
left empty, and the following steps could not work properly with the
empty object array (`revs.pending`). Therefore, make a copy of `revs`
to `revs_copy` for later use right after calling `setup_revisions()`.
The copy of `revs_copy` is not a deep copy, it shares the same objects
with `revs`. The object array of `revs` has been cleared, but objects
themselves are still kept. Flags of objects may change after calling
`prepare_revision_walk()`, we can use these changed flags without
calling the `git rev-list` command and parsing its output like the
former implementation.
Also add testcases for git bundle in t6020, which read args from stdin.
Signed-off-by: Jiang Xin <zhiyou.jx@alibaba-inc.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2021-01-12 03:27:03 +01:00
|
|
|
struct rev_info revs, revs_copy;
|
2020-07-30 01:14:20 +02:00
|
|
|
int min_version = the_hash_algo == &hash_algos[GIT_HASH_SHA1] ? 2 : 3;
|
bundle: arguments can be read from stdin
In order to create an incremental bundle, we need to pass many arguments
to let git-bundle ignore some already packed commits. It will be more
convenient to pass args via stdin. But the current implementation does
not allow us to do this.
This is because args are parsed twice when creating bundle. The first
time for parsing args is in `compute_and_write_prerequisites()` by
running `git-rev-list` command to write prerequisites in bundle file,
and stdin is consumed in this step if "--stdin" option is provided for
`git-bundle`. Later nothing can be read from stdin when running
`setup_revisions()` in `create_bundle()`.
The solution is to parse args once by removing the entire function
`compute_and_write_prerequisites()` and then calling function
`setup_revisions()`. In order to write prerequisites for bundle, will
call `prepare_revision_walk()` and `traverse_commit_list()`. But after
calling `prepare_revision_walk()`, the object array `revs.pending` is
left empty, and the following steps could not work properly with the
empty object array (`revs.pending`). Therefore, make a copy of `revs`
to `revs_copy` for later use right after calling `setup_revisions()`.
The copy of `revs_copy` is not a deep copy, it shares the same objects
with `revs`. The object array of `revs` has been cleared, but objects
themselves are still kept. Flags of objects may change after calling
`prepare_revision_walk()`, we can use these changed flags without
calling the `git rev-list` command and parsing its output like the
former implementation.
Also add testcases for git bundle in t6020, which read args from stdin.
Signed-off-by: Jiang Xin <zhiyou.jx@alibaba-inc.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2021-01-12 03:27:03 +01:00
|
|
|
struct bundle_prerequisites_info bpi;
|
|
|
|
int i;
|
2014-10-30 22:35:24 +01:00
|
|
|
|
|
|
|
bundle_to_stdout = !strcmp(path, "-");
|
|
|
|
if (bundle_to_stdout)
|
|
|
|
bundle_fd = 1;
|
bundle: dup() output descriptor closer to point-of-use
When writing a bundle to a file, the bundle code actually creates
"your.bundle.lock" using our lockfile interface. We feed that output
descriptor to a child git-pack-objects via run-command, which has the
quirk that it closes the output descriptor in the parent.
To avoid confusing the lockfile code (which still thinks the descriptor
is valid), we dup() it, and operate on the duplicate.
However, this has a confusing side effect: after the dup() but before we
call pack-objects, we have _two_ descriptors open to the lockfile. If we
call die() during that time, the lockfile code will try to clean up the
partially-written file. It knows to close() the file before unlinking,
since on some platforms (i.e., Windows) the open file would block the
deletion. But it doesn't know about the duplicate descriptor. On
Windows, triggering an error at the right part of the code will result
in the cleanup failing and the lockfile being left in the filesystem.
We can solve this by moving the dup() much closer to start_command(),
shrinking the window in which we have the second descriptor open. It's
easy to place this in such a way that no die() is possible. We could
still die due to a signal in the exact wrong moment, but we already
tolerate races there (e.g., a signal could come before we manage to put
the file on the cleanup list in the first place).
As a bonus, this shields create_bundle() itself from the duplicate-fd
trick, and we can simplify its error handling (note that the lock
rollback now happens unconditionally, but that's OK; it's a noop if we
didn't open the lock in the first place).
The included test uses an empty bundle to cause a failure at the right
spot in the code, because that's easy to trigger (the other likely
errors are write() problems like ENOSPC). Note that it would already
pass on non-Windows systems (because they are happy to unlink an
already-open file).
Based-on-a-patch-by: Gaël Lhez <gael.lhez@gmail.com>
Signed-off-by: Jeff King <peff@peff.net>
Tested-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2018-11-16 10:43:59 +01:00
|
|
|
else
|
2014-10-30 22:35:24 +01:00
|
|
|
bundle_fd = hold_lock_file_for_update(&lock, path,
|
|
|
|
LOCK_DIE_ON_ERROR);
|
|
|
|
|
2020-07-30 01:14:20 +02:00
|
|
|
if (version == -1)
|
|
|
|
version = min_version;
|
|
|
|
|
|
|
|
if (version < 2 || version > 3) {
|
|
|
|
die(_("unsupported bundle version %d"), version);
|
|
|
|
} else if (version < min_version) {
|
|
|
|
die(_("cannot write bundle version %d with algorithm %s"), version, the_hash_algo->name);
|
|
|
|
} else if (version == 2) {
|
|
|
|
write_or_die(bundle_fd, v2_bundle_signature, strlen(v2_bundle_signature));
|
|
|
|
} else {
|
|
|
|
const char *capability = "@object-format=";
|
|
|
|
write_or_die(bundle_fd, v3_bundle_signature, strlen(v3_bundle_signature));
|
|
|
|
write_or_die(bundle_fd, capability, strlen(capability));
|
|
|
|
write_or_die(bundle_fd, the_hash_algo->name, strlen(the_hash_algo->name));
|
|
|
|
write_or_die(bundle_fd, "\n", 1);
|
|
|
|
}
|
2014-10-30 22:35:24 +01:00
|
|
|
|
|
|
|
/* init revs to list objects for pack-objects later */
|
|
|
|
save_commit_buffer = 0;
|
2018-11-10 06:49:01 +01:00
|
|
|
repo_init_revisions(r, &revs, NULL);
|
2014-10-30 22:35:24 +01:00
|
|
|
|
|
|
|
argc = setup_revisions(argc, argv, &revs, NULL);
|
|
|
|
|
2016-04-01 02:35:45 +02:00
|
|
|
if (argc > 1) {
|
|
|
|
error(_("unrecognized argument: %s"), argv[1]);
|
|
|
|
goto err;
|
|
|
|
}
|
2014-10-30 22:35:24 +01:00
|
|
|
|
bundle: arguments can be read from stdin
In order to create an incremental bundle, we need to pass many arguments
to let git-bundle ignore some already packed commits. It will be more
convenient to pass args via stdin. But the current implementation does
not allow us to do this.
This is because args are parsed twice when creating bundle. The first
time for parsing args is in `compute_and_write_prerequisites()` by
running `git-rev-list` command to write prerequisites in bundle file,
and stdin is consumed in this step if "--stdin" option is provided for
`git-bundle`. Later nothing can be read from stdin when running
`setup_revisions()` in `create_bundle()`.
The solution is to parse args once by removing the entire function
`compute_and_write_prerequisites()` and then calling function
`setup_revisions()`. In order to write prerequisites for bundle, will
call `prepare_revision_walk()` and `traverse_commit_list()`. But after
calling `prepare_revision_walk()`, the object array `revs.pending` is
left empty, and the following steps could not work properly with the
empty object array (`revs.pending`). Therefore, make a copy of `revs`
to `revs_copy` for later use right after calling `setup_revisions()`.
The copy of `revs_copy` is not a deep copy, it shares the same objects
with `revs`. The object array of `revs` has been cleared, but objects
themselves are still kept. Flags of objects may change after calling
`prepare_revision_walk()`, we can use these changed flags without
calling the `git rev-list` command and parsing its output like the
former implementation.
Also add testcases for git bundle in t6020, which read args from stdin.
Signed-off-by: Jiang Xin <zhiyou.jx@alibaba-inc.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2021-01-12 03:27:03 +01:00
|
|
|
/* save revs.pending in revs_copy for later use */
|
|
|
|
memcpy(&revs_copy, &revs, sizeof(revs));
|
|
|
|
revs_copy.pending.nr = 0;
|
|
|
|
revs_copy.pending.alloc = 0;
|
|
|
|
revs_copy.pending.objects = NULL;
|
|
|
|
for (i = 0; i < revs.pending.nr; i++) {
|
|
|
|
struct object_array_entry *e = revs.pending.objects + i;
|
|
|
|
if (e)
|
|
|
|
add_object_array_with_path(e->item, e->name,
|
|
|
|
&revs_copy.pending,
|
|
|
|
e->mode, e->path);
|
|
|
|
}
|
2014-10-30 22:35:24 +01:00
|
|
|
|
bundle: arguments can be read from stdin
In order to create an incremental bundle, we need to pass many arguments
to let git-bundle ignore some already packed commits. It will be more
convenient to pass args via stdin. But the current implementation does
not allow us to do this.
This is because args are parsed twice when creating bundle. The first
time for parsing args is in `compute_and_write_prerequisites()` by
running `git-rev-list` command to write prerequisites in bundle file,
and stdin is consumed in this step if "--stdin" option is provided for
`git-bundle`. Later nothing can be read from stdin when running
`setup_revisions()` in `create_bundle()`.
The solution is to parse args once by removing the entire function
`compute_and_write_prerequisites()` and then calling function
`setup_revisions()`. In order to write prerequisites for bundle, will
call `prepare_revision_walk()` and `traverse_commit_list()`. But after
calling `prepare_revision_walk()`, the object array `revs.pending` is
left empty, and the following steps could not work properly with the
empty object array (`revs.pending`). Therefore, make a copy of `revs`
to `revs_copy` for later use right after calling `setup_revisions()`.
The copy of `revs_copy` is not a deep copy, it shares the same objects
with `revs`. The object array of `revs` has been cleared, but objects
themselves are still kept. Flags of objects may change after calling
`prepare_revision_walk()`, we can use these changed flags without
calling the `git rev-list` command and parsing its output like the
former implementation.
Also add testcases for git bundle in t6020, which read args from stdin.
Signed-off-by: Jiang Xin <zhiyou.jx@alibaba-inc.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2021-01-12 03:27:03 +01:00
|
|
|
/* write prerequisites */
|
|
|
|
revs.boundary = 1;
|
|
|
|
if (prepare_revision_walk(&revs))
|
|
|
|
die("revision walk setup failed");
|
|
|
|
bpi.fd = bundle_fd;
|
|
|
|
bpi.pending = &revs_copy.pending;
|
|
|
|
traverse_commit_list(&revs, write_bundle_prerequisites, NULL, &bpi);
|
|
|
|
object_array_remove_duplicates(&revs_copy.pending);
|
|
|
|
|
|
|
|
/* write bundle refs */
|
|
|
|
ref_count = write_bundle_refs(bundle_fd, &revs_copy);
|
2014-10-30 22:35:24 +01:00
|
|
|
if (!ref_count)
|
|
|
|
die(_("Refusing to create empty bundle."));
|
|
|
|
else if (ref_count < 0)
|
2016-04-01 02:35:45 +02:00
|
|
|
goto err;
|
2007-09-11 05:03:15 +02:00
|
|
|
|
|
|
|
/* write pack */
|
bundle: arguments can be read from stdin
In order to create an incremental bundle, we need to pass many arguments
to let git-bundle ignore some already packed commits. It will be more
convenient to pass args via stdin. But the current implementation does
not allow us to do this.
This is because args are parsed twice when creating bundle. The first
time for parsing args is in `compute_and_write_prerequisites()` by
running `git-rev-list` command to write prerequisites in bundle file,
and stdin is consumed in this step if "--stdin" option is provided for
`git-bundle`. Later nothing can be read from stdin when running
`setup_revisions()` in `create_bundle()`.
The solution is to parse args once by removing the entire function
`compute_and_write_prerequisites()` and then calling function
`setup_revisions()`. In order to write prerequisites for bundle, will
call `prepare_revision_walk()` and `traverse_commit_list()`. But after
calling `prepare_revision_walk()`, the object array `revs.pending` is
left empty, and the following steps could not work properly with the
empty object array (`revs.pending`). Therefore, make a copy of `revs`
to `revs_copy` for later use right after calling `setup_revisions()`.
The copy of `revs_copy` is not a deep copy, it shares the same objects
with `revs`. The object array of `revs` has been cleared, but objects
themselves are still kept. Flags of objects may change after calling
`prepare_revision_walk()`, we can use these changed flags without
calling the `git rev-list` command and parsing its output like the
former implementation.
Also add testcases for git bundle in t6020, which read args from stdin.
Signed-off-by: Jiang Xin <zhiyou.jx@alibaba-inc.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2021-01-12 03:27:03 +01:00
|
|
|
if (write_pack_data(bundle_fd, &revs_copy, pack_options))
|
2016-04-01 02:35:45 +02:00
|
|
|
goto err;
|
2008-01-16 20:12:46 +01:00
|
|
|
|
2010-08-27 22:31:47 +02:00
|
|
|
if (!bundle_to_stdout) {
|
|
|
|
if (commit_lock_file(&lock))
|
2012-04-23 14:30:30 +02:00
|
|
|
die_errno(_("cannot create '%s'"), path);
|
2010-08-27 22:31:47 +02:00
|
|
|
}
|
2008-02-21 23:42:56 +01:00
|
|
|
return 0;
|
2016-04-01 02:35:45 +02:00
|
|
|
err:
|
bundle: dup() output descriptor closer to point-of-use
When writing a bundle to a file, the bundle code actually creates
"your.bundle.lock" using our lockfile interface. We feed that output
descriptor to a child git-pack-objects via run-command, which has the
quirk that it closes the output descriptor in the parent.
To avoid confusing the lockfile code (which still thinks the descriptor
is valid), we dup() it, and operate on the duplicate.
However, this has a confusing side effect: after the dup() but before we
call pack-objects, we have _two_ descriptors open to the lockfile. If we
call die() during that time, the lockfile code will try to clean up the
partially-written file. It knows to close() the file before unlinking,
since on some platforms (i.e., Windows) the open file would block the
deletion. But it doesn't know about the duplicate descriptor. On
Windows, triggering an error at the right part of the code will result
in the cleanup failing and the lockfile being left in the filesystem.
We can solve this by moving the dup() much closer to start_command(),
shrinking the window in which we have the second descriptor open. It's
easy to place this in such a way that no die() is possible. We could
still die due to a signal in the exact wrong moment, but we already
tolerate races there (e.g., a signal could come before we manage to put
the file on the cleanup list in the first place).
As a bonus, this shields create_bundle() itself from the duplicate-fd
trick, and we can simplify its error handling (note that the lock
rollback now happens unconditionally, but that's OK; it's a noop if we
didn't open the lock in the first place).
The included test uses an empty bundle to cause a failure at the right
spot in the code, because that's easy to trigger (the other likely
errors are write() problems like ENOSPC). Note that it would already
pass on non-Windows systems (because they are happy to unlink an
already-open file).
Based-on-a-patch-by: Gaël Lhez <gael.lhez@gmail.com>
Signed-off-by: Jeff King <peff@peff.net>
Tested-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2018-11-16 10:43:59 +01:00
|
|
|
rollback_lock_file(&lock);
|
2016-04-01 02:35:45 +02:00
|
|
|
return -1;
|
2007-09-11 05:03:15 +02:00
|
|
|
}
|
|
|
|
|
2018-11-10 06:49:01 +01:00
|
|
|
int unbundle(struct repository *r, struct bundle_header *header,
|
2021-09-05 09:34:43 +02:00
|
|
|
int bundle_fd, struct strvec *extra_index_pack_args)
|
2007-09-11 05:03:15 +02:00
|
|
|
{
|
2014-08-19 21:09:35 +02:00
|
|
|
struct child_process ip = CHILD_PROCESS_INIT;
|
2021-09-05 09:34:43 +02:00
|
|
|
strvec_pushl(&ip.args, "index-pack", "--fix-thin", "--stdin", NULL);
|
2007-09-11 05:03:15 +02:00
|
|
|
|
2021-09-05 09:34:43 +02:00
|
|
|
if (extra_index_pack_args) {
|
|
|
|
strvec_pushv(&ip.args, extra_index_pack_args->v);
|
|
|
|
strvec_clear(extra_index_pack_args);
|
|
|
|
}
|
2011-09-19 01:52:32 +02:00
|
|
|
|
2018-11-10 06:49:01 +01:00
|
|
|
if (verify_bundle(r, header, 0))
|
2007-09-11 05:03:15 +02:00
|
|
|
return -1;
|
|
|
|
ip.in = bundle_fd;
|
|
|
|
ip.no_stdout = 1;
|
|
|
|
ip.git_cmd = 1;
|
|
|
|
if (run_command(&ip))
|
2012-04-23 14:30:30 +02:00
|
|
|
return error(_("index-pack died"));
|
2007-09-11 05:03:15 +02:00
|
|
|
return 0;
|
|
|
|
}
|