t5305: use "git -C"
authorJeff King <redacted>
Mon, 5 Sep 2016 21:52:17 +0000 (17:52 -0400)
committerJunio C Hamano <redacted>
Wed, 7 Sep 2016 18:45:28 +0000 (11:45 -0700)
commit948a7fd242a28e61e454f62001716af2c9d614b0
tree498ad3d7792789755e4733692948722489b0587a
parent2076353f473aa3cbcbd2623f1765432fa275acec
t5305: use "git -C"

This test unpacks objects into a separate repository, and
accesses it by setting GIT_DIR in a subshell. We can do the
same thing these days by using "git init <repo>" and "git
-C". In most cases this is shorter, though when there are
multiple commands, we may end up repeating the "-C".

However, this repetition can actually be a good thing. This
patch also fixes a bug introduced by 512477b (tests: use
"env" to run commands with temporary env-var settings,
2014-03-18). That commit essentially converted:

   (GIT_DIR=...; export GIT_DIR
    cmd1 &&
    cmd2)

into:

   (GIT_DIR=... cmd1 &&
    cmd2)

which obviously loses the GIT_DIR setting for cmd2 (we never
noticed the bug because it simply runs "cmd2" in the parent
repo, which means we were simply failing to test anything
interesting). By using "git -C" rather than a subshell, it
becomes quite obvious where each command is supposed to be
running.

Signed-off-by: Jeff King <redacted>
Signed-off-by: Junio C Hamano <redacted>
t/t5305-include-tag.sh
git clone https://git.99rst.org/PROJECT