git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: t9350-fast-export.sh broken on peff/pu under Mac OS X

From
Felipe Contreras <felipe.contreras@gmail.com>
Date
Nov 11, 2012, 00:58 UTC
Message-ID
<CAMP44s3SSjDqA9cvG9T=-oVH0ADndbxi1MkWF6wR9gxcoCi_xg@mail.gmail.com>
In-Reply-To
<509EEF09.6060109@gmail.com>

On Sun, Nov 11, 2012 at 1:19 AM, Stefano Lattarini <stefano.lattarini@gmail.com> wrote:

Show 39 quoted lines
> On 11/11/2012 12:11 AM, Felipe Contreras wrote:
>> On Sat, Nov 10, 2012 at 11:39 PM, Junio C Hamano <gitster@pobox.com> wrote:
>>> Felipe Contreras <felipe.contreras@gmail.com> writes:
>>>
>>>> On Sat, Nov 10, 2012 at 3:37 PM, Torsten Bögershausen <tboegi@web.de> wrote:
>>>>> The short version:
>>>>> echo -n doesn't seem to be portable.
>>>>> The following works for me:
>>>>
>>>> Right, I was supposed to change that to:
>>>>
>>>>   true > marks-cur &&
>>>
>>> Please make it like so:
>>>
>>>         >marks-cur &&
>>>
>>> No command is necessary when creating an empty file or truncating an
>>> existing file to empty, and no SP between redirection and its target.
>>
>> That hangs on zsh (presumably waiting for stdin).
>>
> Unless you set:
>
>     NULLCMD=:
>
> early in your test script.
>
> Or, to be extra-safe, you could steal this initialization code from
> autoconf:
>
>     # Be more Bourne compatible.
>     if test -n "${ZSH_VERSION+set}" && (emulate sh) >/dev/null 2>&1; then
>       emulate sh
>       NULLCMD=:
>       setopt NO_GLOB_SUBST
>       # Pre-4.2 versions of Zsh do word splitting on ${1+"$@"}, which
>       # is contrary to our usage.  Disable this feature.
>       alias -g '${1+"$@"}'='"$@"'
It doesn't seem like zsh listens to that variable in sh mode:
% zsh -c 'emulate sh; NULLCMD=foobar; > content'
And it doesn't seem like NO_GLOB_SUBST is needed any more.
I think it doesn't hurt to do 'true > marks-cur', but whatever.
Cheers.
-- 
Felipe Contreras
Previous: Stefano LattariniNext: Stefano Lattarini
Message 6 of 7 in “t9350-fast-export.sh broken on peff/pu under Mac OS X”
  1. Torsten BögershausenNov 10, 2012
  2. Felipe ContrerasNov 10, 2012
  3. Junio C HamanoNov 10, 2012
  4. Felipe ContrerasNov 10, 2012
  5. Stefano LattariniNov 11, 2012
  6. Felipe ContrerasNov 11, 2012
  7. Stefano LattariniNov 11, 2012

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.