From: Junio C Hamano <gitster@pobox•com>
To: "Nguyễn Thái Ngọc Duy" <pclouds@gmail•com>
Cc: git@vger•kernel.org
Subject: Re: [PATCH] Move test-* to t/helper/ subdirectory
Date: Fri, 15 Apr 2016 08:09:12 -0700 [thread overview]
Message-ID: <xmqqtwj2di3r.fsf@gitster.mtv.corp.google.com> (raw)
In-Reply-To: <1460553762-12419-1-git-send-email-pclouds@gmail.com> ("Nguyễn Thái Ngọc Duy"'s message of "Wed, 13 Apr 2016 20:22:42 +0700")
Nguyễn Thái Ngọc Duy <pclouds@gmail•com> writes:
> This keeps top dir a bit less crowded. And because these programs are
> for testing purposes, it makes sense that they stay somewhere in t/
But leaves many *.o files after "make clean". Even "distclean" does
not clean them.
>
> Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail•com>
> ---
> This patch will break any patches that add new test programs.
> Luckily, none in 'next' or 'pu' does that. I know lmdb backend adds
> test-lmdb-backend, so a manual move and some .gitignore fixup is
> required there.
>
> .gitignore | 33 ----------------------
> Makefile | 24 ++++++++--------
> t/helper/.gitignore (new) | 33 ++++++++++++++++++++++
> test-chmtime.c => t/helper/test-chmtime.c | 0
> test-config.c => t/helper/test-config.c | 0
> test-ctype.c => t/helper/test-ctype.c | 0
> test-date.c => t/helper/test-date.c | 0
> test-delta.c => t/helper/test-delta.c | 0
> .../helper/test-dump-cache-tree.c | 0
> .../helper/test-dump-split-index.c | 0
> .../helper/test-dump-untracked-cache.c | 0
> test-fake-ssh.c => t/helper/test-fake-ssh.c | 0
> test-genrandom.c => t/helper/test-genrandom.c | 0
> test-hashmap.c => t/helper/test-hashmap.c | 0
> .../helper/test-index-version.c | 0
> test-line-buffer.c => t/helper/test-line-buffer.c | 0
> test-match-trees.c => t/helper/test-match-trees.c | 0
> test-mergesort.c => t/helper/test-mergesort.c | 0
> test-mktemp.c => t/helper/test-mktemp.c | 0
> .../helper/test-parse-options.c | 0
> test-path-utils.c => t/helper/test-path-utils.c | 0
> test-prio-queue.c => t/helper/test-prio-queue.c | 0
> test-read-cache.c => t/helper/test-read-cache.c | 0
> test-regex.c => t/helper/test-regex.c | 0
> .../helper/test-revision-walking.c | 0
> test-run-command.c => t/helper/test-run-command.c | 0
> .../helper/test-scrap-cache-tree.c | 0
> test-sha1-array.c => t/helper/test-sha1-array.c | 0
> test-sha1.c => t/helper/test-sha1.c | 0
> test-sha1.sh => t/helper/test-sha1.sh | 4 +--
> test-sigchain.c => t/helper/test-sigchain.c | 0
> test-string-list.c => t/helper/test-string-list.c | 0
> .../helper/test-submodule-config.c | 0
> test-subprocess.c => t/helper/test-subprocess.c | 0
> test-svn-fe.c => t/helper/test-svn-fe.c | 0
> .../helper/test-urlmatch-normalization.c | 0
> test-wildmatch.c => t/helper/test-wildmatch.c | 0
> t/t5601-clone.sh | 2 +-
> t/test-lib.sh | 4 +--
> 39 files changed, 50 insertions(+), 50 deletions(-)
> create mode 100644 t/helper/.gitignore
> rename test-chmtime.c => t/helper/test-chmtime.c (100%)
> rename test-config.c => t/helper/test-config.c (100%)
> rename test-ctype.c => t/helper/test-ctype.c (100%)
> rename test-date.c => t/helper/test-date.c (100%)
> rename test-delta.c => t/helper/test-delta.c (100%)
> rename test-dump-cache-tree.c => t/helper/test-dump-cache-tree.c (100%)
> rename test-dump-split-index.c => t/helper/test-dump-split-index.c (100%)
> rename test-dump-untracked-cache.c => t/helper/test-dump-untracked-cache.c (100%)
> rename test-fake-ssh.c => t/helper/test-fake-ssh.c (100%)
> rename test-genrandom.c => t/helper/test-genrandom.c (100%)
> rename test-hashmap.c => t/helper/test-hashmap.c (100%)
> rename test-index-version.c => t/helper/test-index-version.c (100%)
> rename test-line-buffer.c => t/helper/test-line-buffer.c (100%)
> rename test-match-trees.c => t/helper/test-match-trees.c (100%)
> rename test-mergesort.c => t/helper/test-mergesort.c (100%)
> rename test-mktemp.c => t/helper/test-mktemp.c (100%)
> rename test-parse-options.c => t/helper/test-parse-options.c (100%)
> rename test-path-utils.c => t/helper/test-path-utils.c (100%)
> rename test-prio-queue.c => t/helper/test-prio-queue.c (100%)
> rename test-read-cache.c => t/helper/test-read-cache.c (100%)
> rename test-regex.c => t/helper/test-regex.c (100%)
> rename test-revision-walking.c => t/helper/test-revision-walking.c (100%)
> rename test-run-command.c => t/helper/test-run-command.c (100%)
> rename test-scrap-cache-tree.c => t/helper/test-scrap-cache-tree.c (100%)
> rename test-sha1-array.c => t/helper/test-sha1-array.c (100%)
> rename test-sha1.c => t/helper/test-sha1.c (100%)
> rename test-sha1.sh => t/helper/test-sha1.sh (96%)
> rename test-sigchain.c => t/helper/test-sigchain.c (100%)
> rename test-string-list.c => t/helper/test-string-list.c (100%)
> rename test-submodule-config.c => t/helper/test-submodule-config.c (100%)
> rename test-subprocess.c => t/helper/test-subprocess.c (100%)
> rename test-svn-fe.c => t/helper/test-svn-fe.c (100%)
> rename test-urlmatch-normalization.c => t/helper/test-urlmatch-normalization.c (100%)
> rename test-wildmatch.c => t/helper/test-wildmatch.c (100%)
>
> diff --git a/.gitignore b/.gitignore
> index 5087ce1..05cb58a 100644
> --- a/.gitignore
> +++ b/.gitignore
> @@ -179,39 +179,6 @@
> /gitweb/gitweb.cgi
> /gitweb/static/gitweb.js
> /gitweb/static/gitweb.min.*
> -/test-chmtime
> -/test-ctype
> -/test-config
> -/test-date
> -/test-delta
> -/test-dump-cache-tree
> -/test-dump-split-index
> -/test-dump-untracked-cache
> -/test-fake-ssh
> -/test-scrap-cache-tree
> -/test-genrandom
> -/test-hashmap
> -/test-index-version
> -/test-line-buffer
> -/test-match-trees
> -/test-mergesort
> -/test-mktemp
> -/test-parse-options
> -/test-path-utils
> -/test-prio-queue
> -/test-read-cache
> -/test-regex
> -/test-revision-walking
> -/test-run-command
> -/test-sha1
> -/test-sha1-array
> -/test-sigchain
> -/test-string-list
> -/test-submodule-config
> -/test-subprocess
> -/test-svn-fe
> -/test-urlmatch-normalization
> -/test-wildmatch
> /common-cmds.h
> *.tar.gz
> *.dsc
> diff --git a/Makefile b/Makefile
> index 2742a69..0a5fb9d 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -624,7 +624,7 @@ TEST_PROGRAMS_NEED_X += test-svn-fe
> TEST_PROGRAMS_NEED_X += test-urlmatch-normalization
> TEST_PROGRAMS_NEED_X += test-wildmatch
>
> -TEST_PROGRAMS = $(patsubst %,%$X,$(TEST_PROGRAMS_NEED_X))
> +TEST_PROGRAMS = $(patsubst %,t/helper/%$X,$(TEST_PROGRAMS_NEED_X))
>
> # List built-in command $C whose implementation cmd_$C() is not in
> # builtin/$C.o but is linked in as part of some other command.
> @@ -1904,7 +1904,7 @@ VCSSVN_OBJS += vcs-svn/fast_export.o
> VCSSVN_OBJS += vcs-svn/svndiff.o
> VCSSVN_OBJS += vcs-svn/svndump.o
>
> -TEST_OBJS := $(patsubst test-%$X,test-%.o,$(TEST_PROGRAMS))
> +TEST_OBJS := $(patsubst %$X,%.o,$(TEST_PROGRAMS))
> OBJECTS := $(LIB_OBJS) $(BUILTIN_OBJS) $(PROGRAM_OBJS) $(TEST_OBJS) \
> $(XDIFF_OBJS) \
> $(VCSSVN_OBJS) \
> @@ -2211,7 +2211,7 @@ bin-wrappers/%: wrap-for-bin.sh
> @mkdir -p bin-wrappers
> $(QUIET_GEN)sed -e '1s|#!.*/sh|#!$(SHELL_PATH_SQ)|' \
> -e 's|@@BUILD_DIR@@|$(shell pwd)|' \
> - -e 's|@@PROG@@|$(@F)|' < $< > $@ && \
> + -e 's|@@PROG@@|$(patsubst test-%,t/helper/test-%,$(@F))|' < $< > $@ && \
> chmod +x $@
>
> # GNU make supports exporting all variables by "export" without parameters.
> @@ -2231,25 +2231,25 @@ perf: all
>
> .PHONY: test perf
>
> -test-ctype$X: ctype.o
> +t/helper/test-ctype$X: ctype.o
>
> -test-date$X: date.o ctype.o
> +t/helper/test-date$X: date.o ctype.o
>
> -test-delta$X: diff-delta.o patch-delta.o
> +t/helper/test-delta$X: diff-delta.o patch-delta.o
>
> -test-line-buffer$X: vcs-svn/lib.a
> +t/helper/test-line-buffer$X: vcs-svn/lib.a
>
> -test-parse-options$X: parse-options.o parse-options-cb.o
> +t/helper/test-parse-options$X: parse-options.o parse-options-cb.o
>
> -test-svn-fe$X: vcs-svn/lib.a
> +t/helper/test-svn-fe$X: vcs-svn/lib.a
>
> .PRECIOUS: $(TEST_OBJS)
>
> -test-%$X: test-%.o GIT-LDFLAGS $(GITLIBS)
> +t/helper/test-%$X: t/helper/test-%.o GIT-LDFLAGS $(GITLIBS)
> $(QUIET_LINK)$(CC) $(ALL_CFLAGS) -o $@ $(ALL_LDFLAGS) $(filter %.o,$^) $(filter %.a,$^) $(LIBS)
>
> -check-sha1:: test-sha1$X
> - ./test-sha1.sh
> +check-sha1:: t/helper/test-sha1$X
> + t/helper/test-sha1.sh
>
> SP_OBJ = $(patsubst %.o,%.sp,$(C_OBJ))
>
> diff --git a/t/helper/.gitignore b/t/helper/.gitignore
> new file mode 100644
> index 0000000..d6e8b36
> --- /dev/null
> +++ b/t/helper/.gitignore
> @@ -0,0 +1,33 @@
> +/test-chmtime
> +/test-ctype
> +/test-config
> +/test-date
> +/test-delta
> +/test-dump-cache-tree
> +/test-dump-split-index
> +/test-dump-untracked-cache
> +/test-fake-ssh
> +/test-scrap-cache-tree
> +/test-genrandom
> +/test-hashmap
> +/test-index-version
> +/test-line-buffer
> +/test-match-trees
> +/test-mergesort
> +/test-mktemp
> +/test-parse-options
> +/test-path-utils
> +/test-prio-queue
> +/test-read-cache
> +/test-regex
> +/test-revision-walking
> +/test-run-command
> +/test-sha1
> +/test-sha1-array
> +/test-sigchain
> +/test-string-list
> +/test-submodule-config
> +/test-subprocess
> +/test-svn-fe
> +/test-urlmatch-normalization
> +/test-wildmatch
> diff --git a/test-chmtime.c b/t/helper/test-chmtime.c
> similarity index 100%
> rename from test-chmtime.c
> rename to t/helper/test-chmtime.c
> diff --git a/test-config.c b/t/helper/test-config.c
> similarity index 100%
> rename from test-config.c
> rename to t/helper/test-config.c
> diff --git a/test-ctype.c b/t/helper/test-ctype.c
> similarity index 100%
> rename from test-ctype.c
> rename to t/helper/test-ctype.c
> diff --git a/test-date.c b/t/helper/test-date.c
> similarity index 100%
> rename from test-date.c
> rename to t/helper/test-date.c
> diff --git a/test-delta.c b/t/helper/test-delta.c
> similarity index 100%
> rename from test-delta.c
> rename to t/helper/test-delta.c
> diff --git a/test-dump-cache-tree.c b/t/helper/test-dump-cache-tree.c
> similarity index 100%
> rename from test-dump-cache-tree.c
> rename to t/helper/test-dump-cache-tree.c
> diff --git a/test-dump-split-index.c b/t/helper/test-dump-split-index.c
> similarity index 100%
> rename from test-dump-split-index.c
> rename to t/helper/test-dump-split-index.c
> diff --git a/test-dump-untracked-cache.c b/t/helper/test-dump-untracked-cache.c
> similarity index 100%
> rename from test-dump-untracked-cache.c
> rename to t/helper/test-dump-untracked-cache.c
> diff --git a/test-fake-ssh.c b/t/helper/test-fake-ssh.c
> similarity index 100%
> rename from test-fake-ssh.c
> rename to t/helper/test-fake-ssh.c
> diff --git a/test-genrandom.c b/t/helper/test-genrandom.c
> similarity index 100%
> rename from test-genrandom.c
> rename to t/helper/test-genrandom.c
> diff --git a/test-hashmap.c b/t/helper/test-hashmap.c
> similarity index 100%
> rename from test-hashmap.c
> rename to t/helper/test-hashmap.c
> diff --git a/test-index-version.c b/t/helper/test-index-version.c
> similarity index 100%
> rename from test-index-version.c
> rename to t/helper/test-index-version.c
> diff --git a/test-line-buffer.c b/t/helper/test-line-buffer.c
> similarity index 100%
> rename from test-line-buffer.c
> rename to t/helper/test-line-buffer.c
> diff --git a/test-match-trees.c b/t/helper/test-match-trees.c
> similarity index 100%
> rename from test-match-trees.c
> rename to t/helper/test-match-trees.c
> diff --git a/test-mergesort.c b/t/helper/test-mergesort.c
> similarity index 100%
> rename from test-mergesort.c
> rename to t/helper/test-mergesort.c
> diff --git a/test-mktemp.c b/t/helper/test-mktemp.c
> similarity index 100%
> rename from test-mktemp.c
> rename to t/helper/test-mktemp.c
> diff --git a/test-parse-options.c b/t/helper/test-parse-options.c
> similarity index 100%
> rename from test-parse-options.c
> rename to t/helper/test-parse-options.c
> diff --git a/test-path-utils.c b/t/helper/test-path-utils.c
> similarity index 100%
> rename from test-path-utils.c
> rename to t/helper/test-path-utils.c
> diff --git a/test-prio-queue.c b/t/helper/test-prio-queue.c
> similarity index 100%
> rename from test-prio-queue.c
> rename to t/helper/test-prio-queue.c
> diff --git a/test-read-cache.c b/t/helper/test-read-cache.c
> similarity index 100%
> rename from test-read-cache.c
> rename to t/helper/test-read-cache.c
> diff --git a/test-regex.c b/t/helper/test-regex.c
> similarity index 100%
> rename from test-regex.c
> rename to t/helper/test-regex.c
> diff --git a/test-revision-walking.c b/t/helper/test-revision-walking.c
> similarity index 100%
> rename from test-revision-walking.c
> rename to t/helper/test-revision-walking.c
> diff --git a/test-run-command.c b/t/helper/test-run-command.c
> similarity index 100%
> rename from test-run-command.c
> rename to t/helper/test-run-command.c
> diff --git a/test-scrap-cache-tree.c b/t/helper/test-scrap-cache-tree.c
> similarity index 100%
> rename from test-scrap-cache-tree.c
> rename to t/helper/test-scrap-cache-tree.c
> diff --git a/test-sha1-array.c b/t/helper/test-sha1-array.c
> similarity index 100%
> rename from test-sha1-array.c
> rename to t/helper/test-sha1-array.c
> diff --git a/test-sha1.c b/t/helper/test-sha1.c
> similarity index 100%
> rename from test-sha1.c
> rename to t/helper/test-sha1.c
> diff --git a/test-sha1.sh b/t/helper/test-sha1.sh
> similarity index 96%
> rename from test-sha1.sh
> rename to t/helper/test-sha1.sh
> index cef4bcc..750b95a 100755
> --- a/test-sha1.sh
> +++ b/t/helper/test-sha1.sh
> @@ -1,7 +1,7 @@
> #!/bin/sh
>
> dd if=/dev/zero bs=1048576 count=100 2>/dev/null |
> -/usr/bin/time ./test-sha1 >/dev/null
> +/usr/bin/time t/helper/test-sha1 >/dev/null
>
> while read expect cnt pfx
> do
> @@ -11,7 +11,7 @@ do
> test -z "$pfx" || echo "$pfx"
> dd if=/dev/zero bs=1048576 count=$cnt 2>/dev/null |
> perl -pe 'y/\000/g/'
> - } | ./test-sha1 $cnt
> + } | ./t/helper/test-sha1 $cnt
> )
> if test "$expect" = "$actual"
> then
> diff --git a/test-sigchain.c b/t/helper/test-sigchain.c
> similarity index 100%
> rename from test-sigchain.c
> rename to t/helper/test-sigchain.c
> diff --git a/test-string-list.c b/t/helper/test-string-list.c
> similarity index 100%
> rename from test-string-list.c
> rename to t/helper/test-string-list.c
> diff --git a/test-submodule-config.c b/t/helper/test-submodule-config.c
> similarity index 100%
> rename from test-submodule-config.c
> rename to t/helper/test-submodule-config.c
> diff --git a/test-subprocess.c b/t/helper/test-subprocess.c
> similarity index 100%
> rename from test-subprocess.c
> rename to t/helper/test-subprocess.c
> diff --git a/test-svn-fe.c b/t/helper/test-svn-fe.c
> similarity index 100%
> rename from test-svn-fe.c
> rename to t/helper/test-svn-fe.c
> diff --git a/test-urlmatch-normalization.c b/t/helper/test-urlmatch-normalization.c
> similarity index 100%
> rename from test-urlmatch-normalization.c
> rename to t/helper/test-urlmatch-normalization.c
> diff --git a/test-wildmatch.c b/t/helper/test-wildmatch.c
> similarity index 100%
> rename from test-wildmatch.c
> rename to t/helper/test-wildmatch.c
> diff --git a/t/t5601-clone.sh b/t/t5601-clone.sh
> index c1efb8e..150aeaf 100755
> --- a/t/t5601-clone.sh
> +++ b/t/t5601-clone.sh
> @@ -308,7 +308,7 @@ test_expect_success 'clone checking out a tag' '
>
> setup_ssh_wrapper () {
> test_expect_success 'setup ssh wrapper' '
> - cp "$GIT_BUILD_DIR/test-fake-ssh$X" \
> + cp "$GIT_BUILD_DIR/t/helper/test-fake-ssh$X" \
> "$TRASH_DIRECTORY/ssh-wrapper$X" &&
> GIT_SSH="$TRASH_DIRECTORY/ssh-wrapper$X" &&
> export GIT_SSH &&
> diff --git a/t/test-lib.sh b/t/test-lib.sh
> index 0b47eb6..cd0ecd4 100644
> --- a/t/test-lib.sh
> +++ b/t/test-lib.sh
> @@ -854,10 +854,10 @@ test -d "$GIT_BUILD_DIR"/templates/blt || {
> error "You haven't built things yet, have you?"
> }
>
> -if ! test -x "$GIT_BUILD_DIR"/test-chmtime
> +if ! test -x "$GIT_BUILD_DIR"/t/helper/test-chmtime
> then
> echo >&2 'You need to build test-chmtime:'
> - echo >&2 'Run "make test-chmtime" in the source (toplevel) directory'
> + echo >&2 'Run "make t/helper/test-chmtime" in the source (toplevel) directory'
> exit 1
> fi
next prev parent reply other threads:[~2016-04-15 15:09 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-04-13 13:22 [PATCH] Move test-* to t/helper/ subdirectory Nguyễn Thái Ngọc Duy
2016-04-15 15:09 ` Junio C Hamano [this message]
2016-04-15 17:06 ` Junio C Hamano
2016-04-16 0:08 ` Duy Nguyen
2016-04-26 22:07 ` Junio C Hamano
2016-04-27 0:52 ` Duy Nguyen
2016-04-27 10:18 ` Duy Nguyen
2016-04-27 16:15 ` Junio C Hamano
2016-05-01 0:28 ` Duy Nguyen
2016-05-01 6:00 ` Duy Nguyen
2016-05-02 17:34 ` Junio C Hamano
2016-05-03 0:15 ` Duy Nguyen
2016-05-08 9:51 ` Duy Nguyen
2016-05-09 16:03 ` Junio C Hamano
2016-05-10 11:58 ` Duy Nguyen
2016-05-10 20:23 ` Junio C Hamano
-- strict thread matches above, loose matches on Subject: below --
2011-02-09 22:15 [PATCH/RFC] Move test-*.c to test/ subdirectory Junio C Hamano
2011-02-10 2:14 ` [PATCH] Move test-* to t/helper/ subdirectory Nguyễn Thái Ngọc Duy
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=xmqqtwj2di3r.fsf@gitster.mtv.corp.google.com \
--to=gitster@pobox$(echo .)com \
--cc=git@vger$(echo .)kernel.org \
--cc=pclouds@gmail$(echo .)com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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