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

Re: [PATCH] t5528: do not fail with FreeBSD shell

From
Jeff King <peff@peff.net>
Date
Mar 9, 2015, 06:04 UTC
Message-ID
<20150309060403.GA27128@peff.net>
In-Reply-To
<211E8F7E-5588-45B1-ACF6-BB7DFB798ABB@gmail.com>
On Sun, Mar 08, 2015 at 10:19:20PM -0700, Kyle J. McKay wrote:
Show 7 quoted lines
> >I am not convinced this isn't a violation of POSIX (which specifies that
> >field splitting is done on the results of parameter expansions outside
> >of double-quotes). But whether it is or not, we have to live with it.
> 
> That's not the only problem the shell has, t5560 had an issue, rebase had
> issues.  They've have been worked around.  It probably also affects related
> BSDs' shells as well (at least older versions that didn't change the shell).

Yeah, I hope that didn't come across as "bleh, this shell is not worth supporting". It was "whether I think it is a bug or not, it is a real problem and we must work around it".

> >For my own curiosity, what does:
> [...]

Thanks. Weird behavior, certainly, but I think the solution in your patch is the right thing to do.

-Peff
Previous: Kyle J. McKay
Message 4 of 4 in “t5528: do not fail with FreeBSD shell”
  1. t5528: do not fail with FreeBSD shellKyle J. McKay, Mar 8, 2015
  2. Jeff KingMar 8, 2015
  3. Kyle J. McKayMar 9, 2015
  4. Jeff KingMar 9, 2015

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.