From: Jeff King <peff@peff.net>
To: git@vger.kernel.org
Subject: Re: valgrind test script integration
Date: Sat, 12 Jan 2008 06:36:09 -0500 [thread overview]
Message-ID: <20080112113608.GB24589@coredump.intra.peff.net> (raw)
In-Reply-To: <20080112111044.GA24257@coredump.intra.peff.net>
On Sat, Jan 12, 2008 at 06:10:44AM -0500, Jeff King wrote:
> - We only catch calls to 'git', not 'git-foo' (and in fact for that
> reason this doesn't catch the t7300 bug by itself, since that uses
> git-clean). A follow-on patch will deal with this.
And here it is.
This replaces all usage of "git-foo" with "git foo" in the
test scripts. The replacement was done semi-manually; a
fully automatic replacement won't work because the pattern
"git-" appears in several other contexts (e.g.,
"--git-dir=", ref names, filenames, etc).
Obviously another route would be intercepting git-* calls,
as well, but my impression is that we are ultimately heading
towards a "git foo is the right way" situation, in which
case this cleanup is eventually necessary anyway.
[the original got eaten by the list since the patch is almost 150K;
the diffstat is below, and I am making the patch available at
git://repo.or.cz/git/peff.git master
]
t/t1400-update-ref.sh | 10 +-
t/t2005-checkout-index-symlinks.sh | 4 +-
t/t2050-git-dir-relative.sh | 4 +-
t/t2102-update-index-symlinks.sh | 2 +-
t/t2200-add-update.sh | 12 +-
t/t3020-ls-files-error-unmatch.sh | 2 +-
t/t3030-merge-recursive.sh | 14 +-
t/t3200-branch.sh | 16 +-
t/t3210-pack-refs.sh | 4 +-
t/t3400-rebase.sh | 6 +-
t/t3401-rebase-partial.sh | 20 +-
t/t3500-cherry.sh | 12 +-
t/t3600-rm.sh | 4 +-
t/t3800-mktag.sh | 8 +-
t/t3900-i18n-commit.sh | 6 +-
t/t3901-i18n-patch.sh | 16 +-
t/t4012-diff-binary.sh | 2 +-
t/t4103-apply-binary.sh | 26 ++--
t/t5300-pack-object.sh | 14 +-
t/t5301-sliding-window.sh | 4 +-
t/t5302-pack-index.sh | 10 +-
t/t5400-send-pack.sh | 30 ++--
t/t5401-update-hooks.sh | 4 +-
t/t5402-post-merge-hook.sh | 4 +-
t/t5403-post-checkout-hook.sh | 4 +-
t/t5500-fetch-pack.sh | 4 +-
t/t5510-fetch.sh | 2 +-
t/t5530-upload-pack-error.sh | 4 +-
t/t5600-clone-fail-cleanup.sh | 6 +-
t/t6006-rev-list-format.sh | 6 +-
t/t6025-merge-symlinks.sh | 32 ++--
t/t6026-merge-attr.sh | 12 +-
t/t6030-bisect-porcelain.sh | 2 +-
t/t6120-describe.sh | 30 ++--
t/t6300-for-each-ref.sh | 28 ++--
t/t7001-mv.sh | 12 +-
t/t7003-filter-branch.sh | 18 +-
t/t7004-tag.sh | 316 ++++++++++++++++----------------
t/t7101-reset.sh | 6 +-
t/t7300-clean.sh | 38 ++--
t/t7400-submodule-basic.sh | 46 +++---
t/t7501-commit.sh | 44 +++---
t/t9100-git-svn-basic.sh | 44 +++---
t/t9101-git-svn-props.sh | 62 +++---
t/t9102-git-svn-deep-rmdir.sh | 6 +-
t/t9104-git-svn-follow-parent.sh | 36 ++--
t/t9105-git-svn-commit-diff.sh | 8 +-
t/t9106-git-svn-commit-diff-clobber.sh | 12 +-
t/t9107-git-svn-migrate.sh | 16 +-
t/t9108-git-svn-glob.sh | 4 +-
t/t9110-git-svn-use-svm-props.sh | 8 +-
t/t9111-git-svn-use-svnsync-props.sh | 8 +-
t/t9112-git-svn-md5less-file.sh | 4 +-
t/t9116-git-svn-log.sh | 4 +-
t/t9119-git-svn-info.sh | 120 ++++++------
t/t9200-git-cvsexportcommit.sh | 10 +-
t/t9300-fast-import.sh | 64 ++++----
t/t9400-git-cvsserver-server.sh | 30 ++--
58 files changed, 640 insertions(+), 640 deletions(-)
next prev parent reply other threads:[~2008-01-12 11:36 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-01-12 7:11 [ANNOUNCE] GIT 1.5.4-rc3 Junio C Hamano
2008-01-12 7:26 ` Ismail Dönmez
2008-01-12 7:34 ` Junio C Hamano
2008-01-12 7:47 ` Ismail Dönmez
2008-01-12 8:04 ` Junio C Hamano
2008-01-12 9:04 ` Jeff King
2008-01-12 9:05 ` Jeff King
2008-01-12 11:10 ` valgrind test script integration Jeff King
2008-01-12 11:36 ` Jeff King [this message]
2008-01-12 17:10 ` Johannes Schindelin
2008-01-12 18:12 ` Jeff King
2008-01-12 13:01 ` [ANNOUNCE] GIT 1.5.4-rc3 René Scharfe
2008-01-12 13:10 ` Ismail Dönmez
2008-01-12 16:57 ` Johannes Schindelin
2008-01-12 18:09 ` Jeff King
2008-01-12 19:09 ` Junio C Hamano
2008-01-12 14:24 ` Roger C. Soares
2008-01-12 19:13 ` Junio C Hamano
[not found] ` <7vsl13wmas.fsf-jO8aZxhGsIagbBziECNbOZn29agUkmeCHZ5vskTnxNA@public.gmane.org>
2008-01-13 14:41 ` Steffen Prohaska
2008-01-21 2:37 ` [ANNOUNCE] GIT 1.5.4-rc4 Junio C Hamano
[not found] ` <7vsl0r3nvc.fsf-jO8aZxhGsIagbBziECNbOZn29agUkmeCHZ5vskTnxNA@public.gmane.org>
2008-01-21 22:22 ` Steffen Prohaska
[not found] ` <BB330DC7-9F38-4D41-85A2-D475701B6083-wjoc1KHpMeg@public.gmane.org>
2008-01-22 1:21 ` Johannes Schindelin
[not found] ` <alpine.LSU.1.00.0801220121300.5731-OGWIkrnhIhzN0uC3ymp8PA@public.gmane.org>
2008-01-22 5:50 ` Steffen Prohaska
2008-01-28 10:56 ` [ANNOUNCE] GIT 1.5.4-rc5 Junio C Hamano
2008-01-28 18:38 ` Jeff King
2008-01-29 0:36 ` Gustaf Hendeby
2008-01-29 1:25 ` Junio C Hamano
2008-01-29 1:29 ` Jeff King
2008-01-29 6:06 ` Steffen Prohaska
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
List information: http://vger.kernel.org/majordomo-info.html
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20080112113608.GB24589@coredump.intra.peff.net \
--to=peff@peff.net \
--cc=git@vger.kernel.org \
/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.
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
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 read-only IMAP folder(s) and NNTP newsgroup(s).