* [PATCH] tests: rename test to work around GNU gettext bug
@ 2010-07-19 20:13 Ævar Arnfjörð Bjarmason
2010-07-19 20:28 ` [PATCH v2] " Ævar Arnfjörð Bjarmason
0 siblings, 1 reply; 3+ messages in thread
From: Ævar Arnfjörð Bjarmason @ 2010-07-19 20:13 UTC (permalink / raw)
To: git; +Cc: Junio C Hamano, Thomas Rast,
Ævar Arnfjörð Bjarmason
Rename t0200-gettext.sh to t0200-gettext-basic.sh. Versions of GNU
gettext before v0.15 (released in 2006) fail to run the test because
of an old bug in gettext.sh.
To conform with the FHS gettext.sh prints a help message when it's
invoked directly. To do this it checks if $0 is "gettext.sh". This
check was broken before v0.15, it was checking for *\gettext.sh (to
support Windows), but now correctly checks for *\\gettext.sh.
t0200-gettext-basic.sh matched the former broken check, so on systems
with an old GNU gettext (like RHEL 5.4) the test simply printed:
./t0200-gettext.sh
GNU gettext shell script function library version 0.14.6
Usage: . gettext.sh
FATAL: Unexpected exit with code 1
Which is just the gettext.sh help output:
$ gettext.sh
GNU gettext shell script function library version 0.17
Usage: . gettext.sh
Change the test name to t0200-gettext-basic.sh to work around the
issue.
Reported-by: Thomas Rast <trast@student.ethz.ch>
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
---
This patch is against pu. It requires the gettext series already
there.
t/{t0200-gettext.sh => t0200-gettext-basic.sh} | 0
1 files changed, 0 insertions(+), 0 deletions(-)
rename t/{t0200-gettext.sh => t0200-gettext-basic.sh} (100%)
diff --git a/t/t0200-gettext.sh b/t/t0200-gettext-basic.sh
similarity index 100%
rename from t/t0200-gettext.sh
rename to t/t0200-gettext-basic.sh
--
1.7.2.rc3.125.g94e09.dirty
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH v2] tests: rename test to work around GNU gettext bug
2010-07-19 20:13 [PATCH] tests: rename test to work around GNU gettext bug Ævar Arnfjörð Bjarmason
@ 2010-07-19 20:28 ` Ævar Arnfjörð Bjarmason
2010-07-19 20:38 ` Thomas Rast
0 siblings, 1 reply; 3+ messages in thread
From: Ævar Arnfjörð Bjarmason @ 2010-07-19 20:28 UTC (permalink / raw)
To: git; +Cc: Junio C Hamano, Thomas Rast,
Ævar Arnfjörð Bjarmason
Rename t0200-gettext.sh to t0200-gettext-basic.sh. Versions of GNU
gettext before v0.15 (released in 2006) fail to run the test because
of an old bug in gettext.sh.
To conform with the FHS gettext.sh prints a help message when it's
invoked directly. To do this it checks if $0 is "gettext.sh". This
check was broken before v0.15, it was checking for *\gettext.sh (to
support Windows), but now correctly checks for *\\gettext.sh.
t0200-gettext.sh matched the former broken check, so on systems with
an old GNU gettext (like RHEL 5.4) the test simply printed:
./t0200-gettext.sh
GNU gettext shell script function library version 0.14.6
Usage: . gettext.sh
FATAL: Unexpected exit with code 1
Which is just the gettext.sh help output:
$ gettext.sh
GNU gettext shell script function library version 0.17
Usage: . gettext.sh
Change the test name to t0200-gettext-basic.sh to work around that.
Reported-by: Thomas Rast <trast@student.ethz.ch>
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
---
Thomas Rast noted a typo in the commit message offlist. This fixes
that.
t/{t0200-gettext.sh => t0200-gettext-basic.sh} | 0
1 files changed, 0 insertions(+), 0 deletions(-)
rename t/{t0200-gettext.sh => t0200-gettext-basic.sh} (100%)
diff --git a/t/t0200-gettext.sh b/t/t0200-gettext-basic.sh
similarity index 100%
rename from t/t0200-gettext.sh
rename to t/t0200-gettext-basic.sh
--
1.7.2.rc3.125.g94e09.dirty
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2] tests: rename test to work around GNU gettext bug
2010-07-19 20:28 ` [PATCH v2] " Ævar Arnfjörð Bjarmason
@ 2010-07-19 20:38 ` Thomas Rast
0 siblings, 0 replies; 3+ messages in thread
From: Thomas Rast @ 2010-07-19 20:38 UTC (permalink / raw)
To: Ævar Arnfjörð Bjarmason; +Cc: git, Junio C Hamano
Ævar Arnfjörð Bjarmason wrote:
> t0200-gettext.sh matched the former broken check, so on systems with
> an old GNU gettext (like RHEL 5.4) the test simply printed:
>
> ./t0200-gettext.sh
> GNU gettext shell script function library version 0.14.6
> Usage: . gettext.sh
> FATAL: Unexpected exit with code 1
ACK and tested (on the RHEL5.4 that started this). Thanks for the
help and fast fix!
--
Thomas Rast
trast@{inf,student}.ethz.ch
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2010-07-19 20:39 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-07-19 20:13 [PATCH] tests: rename test to work around GNU gettext bug Ævar Arnfjörð Bjarmason
2010-07-19 20:28 ` [PATCH v2] " Ævar Arnfjörð Bjarmason
2010-07-19 20:38 ` Thomas Rast
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).