Re: Repository specific git commands

["Ciprian Dorin Craciun" <ciprian.craciun@gmail.com>]

    Hello all!

    A few months ago, I've proposed a feature (and accompaning patch)
that would enable users to create a "bin" folder inside the .git
folder, where they were allowed to put special git commands, that
would be accessible just like any other git command, but only for that
specific repository.

    Now the problem is that in the new Git v1.5.5.1 something has
changed, and I can not figure out what I shall do...

    In previous versions it was easy: just add a new path in the array
inside the execv_git_cmd function.

    But now the paths were moved to the setup_path function, and I'm
not able to make it work (see my patch bellow). The problem is that
all the commands start with the current folder being the top worktree
folder...

    Any suggestions, help, documentation?

    Thanks all,
    Ciprian.


diff --git a/exec_cmd.c b/exec_cmd.c
index e189cac..2166ae3 100644
--- a/exec_cmd.c
+++ b/exec_cmd.c
@@ -41,6 +41,28 @@ static void add_path(struct strbuf *out, const char *path)
 	}
 }

+static const char *git_repo_exec_path(void)
+{
+	static char path_buffer[PATH_MAX + 1];
+	static char *path = NULL;
+	char *git_dir;
+	int nongit_ok = 1; // ????
+	
+	if (!path) {
+		path = path_buffer;
+		path[0] = '\0';
+		setup_git_directory_gently(&nongit_ok);
+		git_dir = get_git_dir();
+		if (!nongit_ok && git_dir) {
+			strncat(path, git_dir, PATH_MAX);
+			strncat(path, "/", PATH_MAX);
+			strncat(path, "bin", PATH_MAX);
+		}
+	}
+	
+	return path;
+}
+
 void setup_path(const char *cmd_path)
 {
 	const char *old_path = getenv("PATH");
@@ -48,6 +70,7 @@ void setup_path(const char *cmd_path)

 	strbuf_init(&new_path, 0);

+	add_path(&new_path, git_repo_exec_path());
 	add_path(&new_path, argv_exec_path);
 	add_path(&new_path, getenv(EXEC_PATH_ENVIRONMENT));
 	add_path(&new_path, builtin_exec_path);


On Mon, Jan 7, 2008 at 3:15 PM, Ciprian Dorin Craciun
<ciprian.craciun@gmail.com> wrote:
>    Hello all!
>
>    I have a question / proposal: I see on the mailing list a lot of
> situations when some commands (or group of commands) are very often
> used and the users demand (or would like to have) a special "git xxx"
> command for them. But adding such a command -- in almost all cases --
> is not worth the effort or increases the number of available commands
> (thus confusing even more the users).
>
>    Thus the users are left with only two possibilities:
>    -- either define an alias -- but usually it is very restrictive in
> terms of what the alias can do;
>    -- create a custom git command "git-xxx" and place it in the
> executable path -- but this requires root access.
>    (-- or write a custom script but this can not be invoked as "git xxx")
>    => Thus neither option is very useful.
>
>    So my proposal is to let the users create a special folder inside
> the .git directory, for example ".git/bin" where they can place custom
> built git files like "git-xxx", and when they issue "git xxx" this
> folder is searched first, and if the command is found it will be
> executed as any other "git-xxx".
>
>    For this I attach a very simple patch that implements it. (It
> modifies the execv_git_cmd function by adding a new path in the paths
> array.)
>
>    Comments? Opinions? Other solutions that I am not aware of?
>
>    (Please note that I am a git user for only a couple of months, and
> this is the first time I look over git source code...)
>
>    Thanks,
>    Ciprian Craciun.
>