kernel
/
git
mirror of https://git.kernel.org/pub/scm/git/git.git
1
0
Fork 0
Code Releases Activity

git-verify-pack: insist on .idx extension 

git-verify-pack can be called with a filename without .idx extension.
add_packed_git() on the other hand depends on its presence.  So
instead of trying to call it with whatever the user gave us check for
that extension and add it if it's missing.

That means that you can't name your index file "blah" and your pack
file ".pack" anymore ("git-verify-pack blah" currently works in that
case).  I think this regression is a good change. ;-)

Signed-off-by: Rene Scharfe <rene.scharfe@lsrfire.ath.cx>
Signed-off-by: Junio C Hamano <junkio@cox.net>
maint
Rene Scharfe 2006-08-10 17:02:33 +02:00 committed by Junio C Hamano
parent ae9c86f2b6
commit 68f4c78b95
1 changed files with 5 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
verify-pack.c
View File

@ -18,13 +18,12 @@ static int verify_one_pack(const char *path, int verbose)
if (has_extension(arg, len, ".pack")) { if (has_extension(arg, len, ".pack")) {
strcpy(arg + len - 5, ".idx"); strcpy(arg + len - 5, ".idx");
len--; len--;
} } else if (!has_extension(arg, len, ".idx")) {
/* Should name foo.idx now */ if (len + 4 >= PATH_MAX)
if ((g = add_packed_git(arg, len, 1))) return error("name too long: %s.idx", arg);
break;
/* No? did you name just foo? */
strcpy(arg + len, ".idx"); strcpy(arg + len, ".idx");
len += 4; len += 4;
}
if ((g = add_packed_git(arg, len, 1))) if ((g = add_packed_git(arg, len, 1)))
break; break;
return error("packfile %s not found.", arg); return error("packfile %s not found.", arg);