Subject: bug#14477: closed (Re: bug#14477: [PATCH] tests: numfmt: use the printf program, not the shell builtin)

From: [email-masked] (GNU bug Tracking System)
Date: Sun, 26 May 2013 23:21:02 +0000

Your bug report

#14477: [PATCH] tests: numfmt: use the printf program, not the shell builtin

which was filed against the coreutils package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
14477: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=14477
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: bug#14477: [PATCH] tests: numfmt: use the printf program, not the shell builtin
From: Pádraig Brady <[email-masked]>
To: Stefano Lattarini <[email-masked]>
Cc: [email-masked]
Date: Mon, 27 May 2013 00:18:46 +0100
List-Id: <debbugs-submit.debbugs.gnu.org>

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.

Subject: [PATCH] tests: numfmt: use the printf program, not the shell builtin
From: Stefano Lattarini <[email-masked]>
To: [email-masked]
Cc: Stefano Lattarini <[email-masked]>
Date: Mon, 27 May 2013 00:06:00 +0200
List-Id: <debbugs-submit.debbugs.gnu.org>

* 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