git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [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).