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

Re: [PATCH] t1507: change quoting in test_did_you_mean to a more general one

From
Kacper Kornet <draenog@pld-linux.org>
Date
May 6, 2011, 08:39 UTC
Message-ID
<20110506083937.GA3719@camk.edu.pl>
In-Reply-To
<7vsjss6hmf.fsf@alter.siamese.dyndns.org>
On Thu, May 05, 2011 at 01:02:16PM -0700, Junio C Hamano wrote:
> Kacper Kornet <draenog@pld-linux.org> writes:
> > In bash and some other shells the script:
> >   x=2; unset a; echo "${a:-'$x'}"
> > prints '2'. However ksh shell prints $x. The quoting is added to
> > reproduce bash behaviour.
> What I happen to have in /usr/bin/ksh
>     $ /usr/bin/ksh --version
>       version         sh (AT&T Research) 93t+ 2009-05-01
> does not seem to have this issue.
> Whose ksh is this
It is pdksh 5.2.14 from http://www.cs.mun.ca/~michael/pdksh/
> It is broken.

You are right. It is corrected in mksh, which claims to be a successor to pdksh. Unfortunately pdksh is used by default as /bin/sh in PLD Linux Distribution which are use. Probably the time to think about the change of this default.

-- 
  Kacper Kornet
Previous: Junio C Hamano
Message 12 of 12 in “t1507: change quoting in test_did_you_mean to a more general one”
  1. t1507: change quoting in test_did_you_mean to a more general oneKacper Kornet, May 5, 2011
  2. Junio C HamanoMay 5, 2011
  3. Michael J GruberMay 6, 2011
  4. Kacper KornetMay 6, 2011
  5. Michael J GruberMay 6, 2011
  6. Johannes SixtMay 6, 2011
  7. Michael J GruberMay 6, 2011
  8. Junio C HamanoMay 6, 2011
  9. Junio C HamanoMay 9, 2011
  10. Johannes SixtMay 9, 2011
  11. Junio C HamanoMay 9, 2011
  12. Kacper KornetMay 6, 2011

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.