threads / patch / 7890

patchEnsure test-genrandom availability for t5301/t5302

Subject: [PATCH] Ensure test-genrandom availability for t5301/t5302

## tl;dr

4 messages between Apr 28, 2007 and Apr 28, 2007. Diffs are folded; open one to read it.

replies: 3people: 3as markdown or json

Dana How· Apr 28, 2007, 19:46 UTC · lore
Check for this as well as test-chmtime in test-lib.sh
Signed-off-by: Dana L. How <danahow@gmail.com>
---
 t/test-lib.sh |   13 ++++++++-----
 1 files changed, 8 insertions(+), 5 deletions(-)
Show changes to t/test-lib.sh +8 −5
diff --git a/t/test-lib.sh b/t/test-lib.sh
index f2c6bd3..d7838fc 100644
--- a/t/test-lib.sh
+++ b/t/test-lib.sh
@@ -268,11 +268,14 @@ test -d ../templates/blt || {
 	error "You haven't built things yet, have you?"
 }
 
-if ! test -x ../test-chmtime; then
-	echo >&2 'You need to build test-chmtime:'
-	echo >&2 'Run "make test-chmtime" in the source (toplevel) directory'
-	exit 1
-fi
+for prog in test-chmtime test-genrandom
+do
+	if ! test -x ../$prog; then
+		echo >&2 "You need to build $prog:"
+		echo >&2 "Run \"make $prog\" in the source (toplevel) directory"
+		exit 1
+	fi
+done
 
 # Test repository
 test=trash
-- 
1.5.2.rc0.71.g4342-dirty
Junio C Hamano· Apr 28, 2007, 22:32 UTC · re: Dana How · lore

Re: [PATCH] Ensure test-genrandom availability for t5301/t5302

Dana How <how@cswitch.com> writes:
Show 33 quoted lines
> Check for this as well as test-chmtime in test-lib.sh
>
> Signed-off-by: Dana L. How <danahow@gmail.com>
> ---
>  t/test-lib.sh |   13 ++++++++-----
>  1 files changed, 8 insertions(+), 5 deletions(-)
>
> diff --git a/t/test-lib.sh b/t/test-lib.sh
> index f2c6bd3..d7838fc 100644
> --- a/t/test-lib.sh
> +++ b/t/test-lib.sh
> @@ -268,11 +268,14 @@ test -d ../templates/blt || {
>  	error "You haven't built things yet, have you?"
>  }
>  
> -if ! test -x ../test-chmtime; then
> -	echo >&2 'You need to build test-chmtime:'
> -	echo >&2 'Run "make test-chmtime" in the source (toplevel) directory'
> -	exit 1
> -fi
> +for prog in test-chmtime test-genrandom
> +do
> +	if ! test -x ../$prog; then
> +		echo >&2 "You need to build $prog:"
> +		echo >&2 "Run \"make $prog\" in the source (toplevel) directory"
> +		exit 1
> +	fi
> +done
>  
>  # Test repository
>  test=trash
> -- 
> 1.5.2.rc0.71.g4342-dirty

As these two are very small programs, why not always build them upon "make all", like this?

---
 Makefile |    6 +++++-
 1 files changed, 5 insertions(+), 1 deletions(-)
Show changes to Makefile +5 −1
diff --git a/Makefile b/Makefile
index 60c41fd..817f8ef 100644
--- a/Makefile
+++ b/Makefile
@@ -933,13 +933,17 @@ endif
 
 ### Testing rules
 
+TEST_PROGRAMS = test-chmtime$X test-genrandom$X
+
+all:: $(TEST_PROGRAMS)
+
 # GNU make supports exporting all variables by "export" without parameters.
 # However, the environment gets quite big, and some programs have problems
 # with that.
 
 export NO_SVN_TESTS
 
-test: all test-chmtime$X test-genrandom$X
+test: all
 	$(MAKE) -C t/ all
 
 test-date$X: test-date.c date.o ctype.o
Dana How· Apr 28, 2007, 22:36 UTC · re: Junio C Hamano · lore

Re: [PATCH] Ensure test-genrandom availability for t5301/t5302

Either way is fine with me. I just don't want any one to have the nasty surprise I had -- fortunately I figured it out quickly.

So you will apply your patch?
Thanks,
Dana
On 4/28/07, Junio C Hamano <junkio@cox.net> wrote:
Show 69 quoted lines
> Dana How <how@cswitch.com> writes:
>
> > Check for this as well as test-chmtime in test-lib.sh
> >
> > Signed-off-by: Dana L. How <danahow@gmail.com>
> > ---
> >  t/test-lib.sh |   13 ++++++++-----
> >  1 files changed, 8 insertions(+), 5 deletions(-)
> >
> > diff --git a/t/test-lib.sh b/t/test-lib.sh
> > index f2c6bd3..d7838fc 100644
> > --- a/t/test-lib.sh
> > +++ b/t/test-lib.sh
> > @@ -268,11 +268,14 @@ test -d ../templates/blt || {
> >       error "You haven't built things yet, have you?"
> >  }
> >
> > -if ! test -x ../test-chmtime; then
> > -     echo >&2 'You need to build test-chmtime:'
> > -     echo >&2 'Run "make test-chmtime" in the source (toplevel) directory'
> > -     exit 1
> > -fi
> > +for prog in test-chmtime test-genrandom
> > +do
> > +     if ! test -x ../$prog; then
> > +             echo >&2 "You need to build $prog:"
> > +             echo >&2 "Run \"make $prog\" in the source (toplevel) directory"
> > +             exit 1
> > +     fi
> > +done
> >
> >  # Test repository
> >  test=trash
> > --
> > 1.5.2.rc0.71.g4342-dirty
>
> As these two are very small programs, why not always build them
> upon "make all", like this?
>
> ---
>  Makefile |    6 +++++-
>  1 files changed, 5 insertions(+), 1 deletions(-)
>
> diff --git a/Makefile b/Makefile
> index 60c41fd..817f8ef 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -933,13 +933,17 @@ endif
>
>  ### Testing rules
>
> +TEST_PROGRAMS = test-chmtime$X test-genrandom$X
> +
> +all:: $(TEST_PROGRAMS)
> +
>  # GNU make supports exporting all variables by "export" without parameters.
>  # However, the environment gets quite big, and some programs have problems
>  # with that.
>
>  export NO_SVN_TESTS
>
> -test: all test-chmtime$X test-genrandom$X
> +test: all
>         $(MAKE) -C t/ all
>
>  test-date$X: test-date.c date.o ctype.o
>
>
>
-- 
Dana L. How  danahow@gmail.com  +1 650 804 5991 cell
Junio C Hamano· Apr 28, 2007, 22:48 UTC · re: Dana How · lore

Re: [PATCH] Ensure test-genrandom availability for t5301/t5302

"Dana How" <danahow@gmail.com> writes:
Show 5 quoted lines
> Either way is fine with me.
> I just don't want any one to have the nasty surprise I had --
> fortunately I figured it out quickly.
>
> So you will apply your patch?

Perhaps. I've known about this for some time, but it is usually not a problem if you do "make test" from the toplevel, hence it was a low priority for me.

← back to recent threads