git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH 0/4] Run test suite without dashed commands in PATH

From
MOMatthew Ogilvie <mmogilvi_git@miniinfo.net>
Date
Nov 28, 2009, 18:38 UTC
Message-ID
<1259433537-3963-1-git-send-email-mmogilvi_git@miniinfo.net>

This patch series runs the test suite without the dashed commands in the PATH. This improves the usefulness of the test suite as examples of how to do things, and more importantly, minimizes any possibility of regressions in bindir-installed scripts that might prevent them from working in a standard install where most dashed commands are not in the PATH (git-cvsserver was broken in this way until Sep 2009: d2feb01aa5).

The scripts in the "test-bin" directory that patch 3 creates can also be useful for manually testing a build of git without installing it; you typically don't need to set any environment variables (except maybe PATH) for the test-bin scripts to use the build properly.

Trivial note: This is a cleaned up resend of part of a hodgepodge cvsserver patch series that I sent last January.

---------

By the way, has anyone considered the possibility of splitting up the large directory at the top of the git source tree? I suspect that no one is interested, or it would have been done already, but I thought I would mention it anyway. Perhaps split off separate directories for libgit.a, builtins, other C-based excutables, test support executables, scripts, output execdir, and output intermediate (object) files for each of the other directories. Or some subset of these.

-- Matthew Ogilvie [mmogilvi_git@miniinfo.net]

Matthew Ogilvie (4):
  t2300: use documented technique to invoke git-sh-setup
  t3409 t4107 t7406: use dashless commands
  build dashless "test-bin" directory similar to installed bindir
  run test suite without dashed git-commands in PATH
 .gitignore                         |    1 +
 INSTALL                            |    8 +++++-
 Makefile                           |   46 +++++++++++++++++++++++++----------
 t/README                           |    8 ++++++
 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 ++++++++++++++++---------
 test-bin-wrapper.sh                |   13 ++++++++++
 10 files changed, 99 insertions(+), 42 deletions(-)
 create mode 100644 test-bin-wrapper.sh
Next: Matthew Ogilvie
Message 1 of 14 in “Run test suite without dashed commands in PATH”
  1. 0/4 Run test suite without dashed commands in PATHMatthew Ogilvie, Nov 28, 2009
  2. 1/4 t2300: use documented technique to invoke git-sh-setupMatthew Ogilvie, Nov 28, 2009
  3. 2/4 t3409 t4107 t7406: use dashless commandsMatthew Ogilvie, Nov 28, 2009
  4. 3/4 build dashless "test-bin" directory similar to installed bindirMatthew Ogilvie, Nov 28, 2009
  5. 4/4 run test suite without dashed git-commands in PATHMatthew Ogilvie, Nov 28, 2009
  6. Junio C HamanoNov 28, 2009
  7. Jeff KingNov 28, 2009
  8. Junio C HamanoNov 29, 2009
  9. Jeff KingNov 29, 2009
  10. Junio C HamanoNov 29, 2009
  11. Jeff KingNov 29, 2009
  12. Matthew OgilvieNov 29, 2009
  13. Matthew OgilvieNov 29, 2009
  14. Nanako ShiraishiNov 30, 2009

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.