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

Re: [PATCH v2] Get rid of the non portable shell export VAR=VALUE costruct

From
Junio C Hamano <gitster@pobox.com>
Date
May 23, 2014, 21:56 UTC
Message-ID
<xmqqtx8ggnw5.fsf@gitster.dls.corp.google.com>
In-Reply-To
<20140523184405.GE12314@google.com>
Jonathan Nieder <jrnieder@gmail.com> writes:
> -	/^\s*export\s+[^=]*=/ and err '"export FOO=bar" is not por...
> +	/\bexport\s+[A-Za-z0-9_]*=/ and err '"export FOO=bar" is n...

I see you also tightened what the "variable" to be assigned should look like, which is a good change to avoid false positives.

Because the inter-word dash is taken as a word "\b"oundary, that change incidentally prevents a false positive from triggering on something like:

	git fast-export --option=value
;-)
Previous: Junio C HamanoNext: Eric Sunshine
Message 9 of 10 in “Get rid of the non portable shell export VAR=VALUE costruct”
  1. Get rid of the non portable shell export VAR=VALUE costructElia Pinto, May 23, 2014
  2. Jonathan NiederMay 23, 2014
  3. Junio C HamanoMay 23, 2014
  4. Junio C HamanoMay 23, 2014
  5. Jonathan NiederMay 23, 2014
  6. Jonathan NiederMay 23, 2014
  7. Torsten BögershausenMay 23, 2014
  8. Junio C HamanoMay 23, 2014
  9. Junio C HamanoMay 23, 2014
  10. Eric SunshineMay 23, 2014

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.