--- Begin Message ---
Subject: |
[PATCH] tests: numfmt: use the printf program, not the shell builtin |
Date: |
Mon, 27 May 2013 00:06:00 +0200 |
* tests/misc/numfmt.pl: Here. This avoids a spurious failure when
/bin/sh is dash (as can happen on Debian systems).
---
tests/misc/numfmt.pl | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/misc/numfmt.pl b/tests/misc/numfmt.pl
index 61917fb..553f68b 100644
--- a/tests/misc/numfmt.pl
+++ b/tests/misc/numfmt.pl
@@ -887,7 +887,7 @@ if ($locale ne 'C')
{
# Reset locale to 'C' if LOCALE_FR_UTF8 doesn't output as expected
# as determined by the separate printf program.
- open(LOC_NUM, "LC_ALL=$locale printf \"%'d\" 1234|")
+ open(LOC_NUM, "env LC_ALL=$locale printf \"%'d\" 1234|")
or die "Can't fork command: $!";
my $loc_num = <LOC_NUM>;
close(LOC_NUM) || die "Failed to read grouped number from printf";
--
1.8.3.rc3.8.g5e49f30
--- End Message ---
--- Begin Message ---
Subject: |
Re: bug#14477: [PATCH] tests: numfmt: use the printf program, not the shell builtin |
Date: |
Mon, 27 May 2013 00:18:46 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130110 Thunderbird/17.0.2 |
On 05/26/2013 11:06 PM, Stefano Lattarini wrote:
> * tests/misc/numfmt.pl: Here. This avoids a spurious failure when
> /bin/sh is dash (as can happen on Debian systems).
> ---
> tests/misc/numfmt.pl | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/misc/numfmt.pl b/tests/misc/numfmt.pl
> index 61917fb..553f68b 100644
> --- a/tests/misc/numfmt.pl
> +++ b/tests/misc/numfmt.pl
> @@ -887,7 +887,7 @@ if ($locale ne 'C')
> {
> # Reset locale to 'C' if LOCALE_FR_UTF8 doesn't output as expected
> # as determined by the separate printf program.
> - open(LOC_NUM, "LC_ALL=$locale printf \"%'d\" 1234|")
> + open(LOC_NUM, "env LC_ALL=$locale printf \"%'d\" 1234|")
> or die "Can't fork command: $!";
> my $loc_num = <LOC_NUM>;
> close(LOC_NUM) || die "Failed to read grouped number from printf";
>
Applied.
thanks,
Pádraig.
--- End Message ---