Skip to content

Commit

Permalink
Revert "receive-pack: use strict mode for unpacking objects"
Browse files Browse the repository at this point in the history
This reverts commit 28f72a0.
  • Loading branch information
Junio C Hamano committed Mar 4, 2008
1 parent 27b4070 commit 9eb7a50
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 29 deletions.
6 changes: 0 additions & 6 deletions Documentation/config.txt
Original file line number Diff line number Diff line change
Expand Up @@ -939,12 +939,6 @@ imap::
The configuration variables in the 'imap' section are described
in linkgit:git-imap-send[1].

receive.fsckObjects::
If it is set to true, git-receive-pack will check all received
objects. It will abort in the case of a malformed object or a
broken link. The result of an abort are only dangling objects.
The default value is true.

receive.unpackLimit::
If the number of objects received in a push is below this
limit then the objects will be unpacked into loose object
Expand Down
36 changes: 13 additions & 23 deletions receive-pack.c
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@
static const char receive_pack_usage[] = "git-receive-pack <git-dir>";

static int deny_non_fast_forwards = 0;
static int receive_fsck_objects = 1;
static int receive_unpack_limit = -1;
static int transfer_unpack_limit = -1;
static int unpack_limit = 100;
Expand All @@ -36,11 +35,6 @@ static int receive_pack_config(const char *var, const char *value)
return 0;
}

if (strcmp(var, "receive.fsckobjects") == 0) {
receive_fsck_objects = git_config_bool(var, value);
return 0;
}

return git_default_config(var, value);
}

Expand Down Expand Up @@ -374,13 +368,11 @@ static const char *unpack(void)
ntohl(hdr.hdr_version), ntohl(hdr.hdr_entries));

if (ntohl(hdr.hdr_entries) < unpack_limit) {
int code, i = 0;
const char *unpacker[4];
unpacker[i++] = "unpack-objects";
if (receive_fsck_objects)
unpacker[i++] = "--strict";
unpacker[i++] = hdr_arg;
unpacker[i++] = NULL;
int code;
const char *unpacker[3];
unpacker[0] = "unpack-objects";
unpacker[1] = hdr_arg;
unpacker[2] = NULL;
code = run_command_v_opt(unpacker, RUN_GIT_CMD);
switch (code) {
case 0:
Expand All @@ -401,23 +393,21 @@ static const char *unpack(void)
return "unpacker exited with error code";
}
} else {
const char *keeper[7];
int s, status, i = 0;
const char *keeper[6];
int s, status;
char keep_arg[256];
struct child_process ip;

s = sprintf(keep_arg, "--keep=receive-pack %i on ", getpid());
if (gethostname(keep_arg + s, sizeof(keep_arg) - s))
strcpy(keep_arg + s, "localhost");

keeper[i++] = "index-pack";
keeper[i++] = "--stdin";
if (receive_fsck_objects)
keeper[i++] = "--strict";
keeper[i++] = "--fix-thin";
keeper[i++] = hdr_arg;
keeper[i++] = keep_arg;
keeper[i++] = NULL;
keeper[0] = "index-pack";
keeper[1] = "--stdin";
keeper[2] = "--fix-thin";
keeper[3] = hdr_arg;
keeper[4] = keep_arg;
keeper[5] = NULL;
memset(&ip, 0, sizeof(ip));
ip.argv = keeper;
ip.out = -1;
Expand Down

0 comments on commit 9eb7a50

Please sign in to comment.