<feed xmlns='http://www.w3.org/2005/Atom'>
<title>user/sven/git.git/exec_cmd.c, branch v1.4.1.1</title>
<subtitle>Git
</subtitle>
<id>https://git.stealer.net/cgit.cgi/user/sven/git.git/atom?h=v1.4.1.1</id>
<link rel='self' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/atom?h=v1.4.1.1'/>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/'/>
<updated>2006-05-31T04:49:01Z</updated>
<entry>
<title>git_exec_path, execv_git_cmd: ignore empty environment variables</title>
<updated>2006-05-31T04:49:01Z</updated>
<author>
<name>Dmitry V. Levin</name>
<email>ldv@altlinux.org</email>
</author>
<published>2006-05-29T00:34:34Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=2b6016263c984b71b29373fc9354d497cd51947d'/>
<id>urn:sha1:2b6016263c984b71b29373fc9354d497cd51947d</id>
<content type='text'>
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 &lt;ldv@altlinux.org&gt;
Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
</content>
</entry>
<entry>
<title>execv_git_cmd: Fix stack buffer overflow.</title>
<updated>2006-05-31T04:47:29Z</updated>
<author>
<name>Dmitry V. Levin</name>
<email>ldv@altlinux.org</email>
</author>
<published>2006-05-30T14:58:52Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=d6859901012535b3781e5332923ede51d5977395'/>
<id>urn:sha1:d6859901012535b3781e5332923ede51d5977395</id>
<content type='text'>
Signed-off-by: Dmitry V. Levin &lt;ldv@altlinux.org&gt;
Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
</content>
</entry>
<entry>
<title>cleanups: remove unused variable from exec_cmd.c</title>
<updated>2006-04-17T22:06:40Z</updated>
<author>
<name>Serge E. Hallyn</name>
<email>serue@us.ibm.com</email>
</author>
<published>2006-04-17T15:14:48Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=bb996614de4fddfa73286479f3c6a901107a2629'/>
<id>urn:sha1:bb996614de4fddfa73286479f3c6a901107a2629</id>
<content type='text'>
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 &lt;serue@us.ibm.com&gt;
Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
</content>
</entry>
<entry>
<title>Use #define ARRAY_SIZE(x) (sizeof(x)/sizeof(x[0]))</title>
<updated>2006-03-09T19:58:05Z</updated>
<author>
<name>Junio C Hamano</name>
<email>junkio@cox.net</email>
</author>
<published>2006-03-09T19:58:05Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=b4f2a6ac9263d6e9bf3bf9c8abe607cd6def0ee8'/>
<id>urn:sha1:b4f2a6ac9263d6e9bf3bf9c8abe607cd6def0ee8</id>
<content type='text'>
Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
</content>
</entry>
<entry>
<title>Const tightening.</title>
<updated>2006-03-05T10:47:29Z</updated>
<author>
<name>Junio C Hamano</name>
<email>junkio@cox.net</email>
</author>
<published>2006-03-05T10:47:29Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=9201c707426b3dc0c894775416f576c25c008d46'/>
<id>urn:sha1:9201c707426b3dc0c894775416f576c25c008d46</id>
<content type='text'>
Mark Wooding noticed there was a type mismatch warning in git.c; this
patch does things slightly differently (mostly tightening const) and
was what I was holding onto, waiting for the setup-revisions change
to be merged into the master branch.

Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
</content>
</entry>
<entry>
<title>Use setenv(), fix warnings</title>
<updated>2006-02-26T23:06:45Z</updated>
<author>
<name>Timo Hirvonen</name>
<email>tihirvon@gmail.com</email>
</author>
<published>2006-02-26T15:13:46Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=962554c616e30991553c8497ed1e7c2a415fa84d'/>
<id>urn:sha1:962554c616e30991553c8497ed1e7c2a415fa84d</id>
<content type='text'>
  - Fix -Wundef -Wold-style-definition warnings
  - Make pll_free() static

[jc: original patch by Timo had another unrelated bits:

  - Use setenv() instead of putenv()

 I'm postponing that part for now.]

Signed-off-by: Timo Hirvonen &lt;tihirvon@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
</content>
</entry>
<entry>
<title>Exec git programs without using PATH.</title>
<updated>2006-01-14T00:49:01Z</updated>
<author>
<name>Michal Ostrowski</name>
<email>mostrows@watson.ibm.com</email>
</author>
<published>2006-01-11T02:12:17Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=77cb17e940c806a4cc811bfd66187277b729a1e7'/>
<id>urn:sha1:77cb17e940c806a4cc811bfd66187277b729a1e7</id>
<content type='text'>
The git suite may not be in PATH (and thus programs such as
git-send-pack could not exec git-rev-list).  Thus there is a need for
logic that will locate these programs.  Modifying PATH is not
desirable as it result in behavior differing from the user's
intentions, as we may end up prepending "/usr/bin" to PATH.

- git C programs will use exec*_git_cmd() APIs to exec sub-commands.
- exec*_git_cmd() will execute a git program by searching for it in
  the following directories:
	1. --exec-path (as used by "git")
	2. The GIT_EXEC_PATH environment variable.
	3. $(gitexecdir) as set in Makefile (default value $(bindir)).
- git wrapper will modify PATH as before to enable shell scripts to
  invoke "git-foo" commands.

Ideally, shell scripts should use the git wrapper to become independent
of PATH, and then modifying PATH will not be necessary.

[jc: with minor updates after a brief review.]

Signed-off-by: Michal Ostrowski &lt;mostrows@watson.ibm.com&gt;
Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
</content>
</entry>
</feed>
