GNU bug report logs - #24116
new snapshot available: diffutils-3.3.50-0353

Previous Next

Package: diffutils;

Reported by: Jim Meyering <jim <at> meyering.net>

Date: Sun, 31 Jul 2016 07:10:01 UTC

Severity: normal

Tags: notabug

Done: Jim Meyering <jim <at> meyering.net>

Bug is archived. No further changes may be made.

Full log


Message #41 received at 24116 <at> debbugs.gnu.org (full text, mbox):

From: Jim Meyering <jim <at> meyering.net>
To: Dave Gordon <david.s.gordon <at> intel.com>
Cc: 24116 <at> debbugs.gnu.org
Subject: Re: [bug-diffutils] bug#24116: bug#24116: [platform-testers] new
 snapshot available: diffutils-3.3.50-0353
Date: Fri, 5 Aug 2016 13:35:30 -0700
On Fri, Aug 5, 2016 at 8:54 AM, Dave Gordon <david.s.gordon <at> intel.com> wrote:
...
>
> after Harald's explanation, can I suggest you change the script to
> separate the "local" and the assignment? That appears to work on all
> shells, including dash(1)
>
> $ dash -c 'f() { local t; t=$(printf '\''\t\t'\''); printf "$t"; }; f' | hd
> 00000000  09 09                                             |..|
> 00000002
>
> It's a really minimal (3-character) change, and it's less ugly than
> adding the extra quotes that the standard seems to say should never
> be needed :)

Thanks, but I've already fixed it by hoisting the definition to global scope:
http://git.sv.gnu.org/cgit/diffutils.git/commit/?id=55596fcd4846a802d




This bug report was last modified 8 years and 18 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.