git/exec_cmd.c, branch v1.5.2

Mechanical conversion to use prefixcmp()

2007-02-21T06:03:15Z

This mechanically converts strncmp() to use prefixcmp(), but only when the parameters match specific patterns, so that they can be verified easily. Leftover from this will be fixed in a separate step, including idiotic conversions like if (!strncmp("foo", arg, 3)) => if (!(-prefixcmp(arg, "foo"))) This was done by using this script in px.perl #!/usr/bin/perl -i.bak -p if (/strncmp$([^,]+), "([^\\"]*)", (\d+)$/ && (length($2) == $3)) { s|strncmp$([^,]+), "([^\\"]*)", (\d+)$|prefixcmp($1, "$2")|; } if (/strncmp$"([^\\"]*)", ([^,]+), (\d+)$/ && (length($1) == $3)) { s|strncmp$"([^\\"]*)", ([^,]+), (\d+)$|(-prefixcmp($2, "$1"))|; } and running: $ git grep -l strncmp -- '*.c' | xargs perl px.perl Signed-off-by: Junio C Hamano

Use preprocessor constants for environment variable names.

2006-12-19T09:51:51Z

We broke the discipline Linus set up to allow compiler help us avoid typos in environment names in the early days of git over time. This defines a handful preprocessor constants for environment variable names used in relatively core parts of the system. I've left out variable names specific to subsystems such as HTTP and SSL as I do not think they are big problems. Signed-off-by: Junio C Hamano

Trace into open fd and refactor tracing code.

2006-08-31T21:16:38Z

Now if GIT_TRACE is set to an integer value greater than 1 and lower than 10, we interpret this as an open fd value and we trace into it. Note that this behavior is not compatible with the previous one. We also trace whole messages using one write(2) call to make sure messages from processes do net get mixed up in the middle. It's now possible to run the tests like this: GIT_TRACE=9 make test 9>/var/tmp/trace.log Signed-off-by: Christian Couder Signed-off-by: Junio C Hamano

remove unnecessary initializations

2006-08-16T04:22:20Z

[jc: I needed to hand merge the changes to the updated codebase, so the result needs to be checked.] Signed-off-by: David Rientjes Signed-off-by: Junio C Hamano

GIT_TRACE: fix a mixed declarations and code warning

2006-07-09T07:57:23Z

Signed-off-by: Timo Hirvonen Signed-off-by: Junio C Hamano

GIT_TRACE: show which built-in/external commands are executed

2006-07-09T07:57:23Z

With the environment variable GIT_TRACE set git will show - alias expansion - built-in command execution - external command execution on stderr. Signed-off-by: Matthias Lederhofer Signed-off-by: Junio C Hamano

git_exec_path, execv_git_cmd: ignore empty environment variables

2006-05-31T04:49:01Z

Ignoring empty environment variables is good common practice. Ignoring --exec-path with empty argument won't harm, too: if user means current directory, there is a "--exec-path=." Signed-off-by: Dmitry V. Levin Signed-off-by: Junio C Hamano

execv_git_cmd: Fix stack buffer overflow.

2006-05-31T04:47:29Z

Signed-off-by: Dmitry V. Levin Signed-off-by: Junio C Hamano

cleanups: remove unused variable from exec_cmd.c

2006-04-17T22:06:40Z

Not sure whether it should be removed, or whether execv_git_cmd() should return it rather than -1 at bottom. Signed-off-by: Serge E. Hallyn Signed-off-by: Junio C Hamano

Use #define ARRAY_SIZE(x) (sizeof(x)/sizeof(x[0]))

2006-03-09T19:58:05Z

Signed-off-by: Junio C Hamano