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

Re: [PATCH 1/3] cg-mv doesn't work with bash 3.1.7 due to excessive quotes

From
PRPavel Roskin <proski@gnu.org>
Date
Mar 10, 2006, 20:48 UTC
Message-ID
<1142023722.23830.8.camel@dv>
In-Reply-To
<1141395067.30343.14.camel@dv>
Hello, Petr!
On Fri, 2006-03-03 at 09:11 -0500, Pavel Roskin wrote:
Show 5 quoted lines
> On Thu, 2006-03-02 at 21:27 -0800, Junio C Hamano wrote:
> > Pavel Roskin <proski@gnu.org> writes:
> > 
> > > -	ARGS2["${#ARGS2[@]}"]="$_git_relpath${arg%/}"
> > > +	ARGS2[${#ARGS2[@]}]="$_git_relpath${arg%/}"

Any issues with this patch? FC5 is due in a week. Expect and outcry from the new bash 3.1.7 users if the fixed cogito is not available shortly.

Bash is not as wrong as it may seem. Left hand side in assignments is already a special case in earlier versions of bash, just not in the index:

bash 3.00.16 (FC4) $ f"oo"=bar bash: foo=bar: command not found $ foo["0"]=bar $

bash 3.1.7 (FC5) $ f"oo"=bar bash: foo=bar: command not found $ foo["0"]=bar bash: "0": syntax error: operand expected (error token is ""0"") $

-- 
Regards,
Pavel Roskin
Previous: Pavel RoskinNext: Josef Weidendorfer
Message 7 of 8 in “cg-mv doesn't work with bash 3.1.7 due to excessive quotes”
  1. 1/3 cg-mv doesn't work with bash 3.1.7 due to excessive quotesPavel Roskin, Mar 3, 2006
  2. 2/3 Make tutorial-script work with current cogitoPavel Roskin, Mar 3, 2006
  3. Petr BaudisMar 19, 2006
  4. 3/3 Allow the tutorial script to be run by "make test"Pavel Roskin, Mar 3, 2006
  5. Junio C HamanoMar 3, 2006
  6. Pavel RoskinMar 3, 2006
  7. Pavel RoskinMar 10, 2006
  8. Josef WeidendorferMar 3, 2006

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.