Subject: Re: [PATCH] Make it possible to run git archive --remote without a git directory.

From: Simon Hausmann <[email-masked]>
Date: Thu, 5 Apr 2007 21:47:39 +0200

On Thursday 05 April 2007 10:12:37 Simon Hausmann wrote:
[...]

Sorry, for replying to my own mail, but attached is a revised patch that also 
allows --list to work without a git dir, as pointed out by Jeff Garzik.

Would be nice if the patch could be applied to git :)

Simon
From f893a0080d0af6d1f02c05de79d21598f614fa4b Mon Sep 17 00:00:00 2001
From: Simon Hausmann <[email-masked]>
Date: Thu, 5 Apr 2007 10:03:19 +0200
Subject: [PATCH] Make it possible to run git-archive --remote and --list without a git directory.

Try a gentle git directory setup for the archive command that remains fatal for
the local case but permits operation without a local git repository when using
--remote. If in the remote case a git directory is present it will then still be
used for proxy settings, but that is optional now.

Signed-off-by: Simon Hausmann <[email-masked]>
---
 builtin-archive.c |    6 ++++++
 git.c             |    2 +-
 2 files changed, 7 insertions(+), 1 deletions(-)

diff --git a/builtin-archive.c b/builtin-archive.c
index 2fae885..8c2fe49 100644
--- a/builtin-archive.c
+++ b/builtin-archive.c
@@ -243,6 +243,9 @@ int cmd_archive(int argc, const char **argv, const char *prefix)
 	struct archiver ar;
 	int tree_idx;
 	const char *remote = NULL;
+	int nongit = 0;
+
+	prefix = setup_git_directory_gently(&nongit);
 
 	remote = extract_remote_arg(&argc, argv);
 	if (remote)
@@ -253,6 +256,9 @@ int cmd_archive(int argc, const char **argv, const char *prefix)
 	memset(&ar, 0, sizeof(ar));
 	tree_idx = parse_archive_args(argc, argv, &ar);
 
+	if (nongit)
+		die("Not a git repository");
+
 	argv += tree_idx;
 	parse_treeish_arg(argv, &ar.args, prefix);
 	parse_pathspec_arg(argv + 1, &ar.args);
diff --git a/git.c b/git.c
index 5b1bc2a..33dd4d3 100644
--- a/git.c
+++ b/git.c
@@ -226,7 +226,7 @@ static void handle_internal_command(int argc, const char **argv, char **envp)
 		{ "add", cmd_add, RUN_SETUP | NOT_BARE },
 		{ "annotate", cmd_annotate, USE_PAGER },
 		{ "apply", cmd_apply },
-		{ "archive", cmd_archive, RUN_SETUP },
+		{ "archive", cmd_archive },
 		{ "blame", cmd_blame, RUN_SETUP },
 		{ "branch", cmd_branch, RUN_SETUP },
 		{ "bundle", cmd_bundle },
-- 
1.5.1