From: Matthew Ogilvie <mmogilvi_git@miniinfo.net>
To: git@vger.kernel.org, gitster@pobox.com
Cc: Matthew Ogilvie <mmogilvi_git@miniinfo.net>
Subject: [PATCH v2 0/6] Run test suite without dashed commands in PATH
Date: Sun, 29 Nov 2009 23:19:25 -0700 [thread overview]
Message-ID: <1259561971-25730-1-git-send-email-mmogilvi_git@miniinfo.net> (raw)
This patch series runs the test suite without the dashed commands
in the PATH.
Changes since version 1:
- Added patch 3 that documents the pre-existing GIT_TEST_INSTALLED
feature.
- Rename what used to be "test-bin" directory as "bin-wrappers", to
more clearly describe what it is.
- Split off patch 6 (INSTALL documention) from patch
4 (was patch 3), describing how the bin-wrappers directory
is a very convenient way to manually test an uninstalled build.
I also reworded it a bit to mention the downsides. Junio
doesn't seem to agree this is useful; splitting it off makes
it easy to leave it out.
Not changed:
There was some discussion about not building the bin-wrappers directory
unless you are actually running tests. But I don't really think it
is worth the additional complexity to manage this. The "make all"
target is already building several full-up binaries to support the
test suite (not just the small sed-ed scripts that this adds). And any
such solution would need to deal with parallel execution locking
issues, as well as additional uglyness (either duplicating a list
of bindir executables within test-lib.sh itself, or calling into a
parent directory makefile that might already be running in
a grandparent process).
Matthew Ogilvie (6):
t2300: use documented technique to invoke git-sh-setup
t3409 t4107 t7406: use dashless commands
t/README: Document GIT_TEST_INSTALLED and GIT_TEST_EXEC_PATH
build dashless "bin-wrappers" directory similar to installed bindir
run test suite without dashed git-commands in PATH
INSTALL: document a simpler way to run uninstalled builds
.gitignore | 1 +
INSTALL | 11 +++++++-
Makefile | 49 ++++++++++++++++++++++++++---------
t/README | 21 +++++++++++++++
t/t2300-cd-to-toplevel.sh | 2 +-
t/t3409-rebase-preserve-merges.sh | 6 ++--
t/t4107-apply-ignore-whitespace.sh | 20 +++++++-------
t/t7406-submodule-update.sh | 4 +-
t/test-lib.sh | 33 +++++++++++++++---------
wrap-for-bin.sh | 15 +++++++++++
10 files changed, 120 insertions(+), 42 deletions(-)
create mode 100644 wrap-for-bin.sh
next reply other threads:[~2009-11-30 6:20 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-11-30 6:19 Matthew Ogilvie [this message]
2009-11-30 6:19 ` [PATCH v2 1/6] t2300: use documented technique to invoke git-sh-setup Matthew Ogilvie
2009-11-30 6:19 ` [PATCH v2 2/6] t3409 t4107 t7406: use dashless commands Matthew Ogilvie
2009-11-30 6:19 ` [PATCH v2 3/6] t/README: Document GIT_TEST_INSTALLED and GIT_TEST_EXEC_PATH Matthew Ogilvie
2009-11-30 6:19 ` [PATCH v2 4/6] build dashless "bin-wrappers" directory similar to installed bindir Matthew Ogilvie
2009-11-30 6:19 ` [PATCH v2 5/6] run test suite without dashed git-commands in PATH Matthew Ogilvie
2009-11-30 6:19 ` [PATCH v2 6/6] INSTALL: document a simpler way to run uninstalled builds Matthew Ogilvie
2009-11-30 6:48 ` Junio C Hamano
2009-12-01 17:24 ` [PATCH v2 5/6] run test suite without dashed git-commands in PATH Jakub Narebski
2009-12-02 5:49 ` Matthew Ogilvie
2009-12-02 6:25 ` Junio C Hamano
2009-11-30 6:28 ` [PATCH v2 4/6] build dashless "bin-wrappers" directory similar to installed bindir Junio C Hamano
2009-12-01 7:33 ` Matthew Ogilvie
2009-11-30 7:16 ` [PATCH v2 0/6] Run test suite without dashed commands in PATH Junio C Hamano
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1259561971-25730-1-git-send-email-mmogilvi_git@miniinfo.net \
--to=mmogilvi_git@miniinfo.net \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).