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

Re: [PATCH] Avoid bug in Solaris xpg4/sed as used in submodule

From
Junio C Hamano <gitster@pobox.com>
Date
Apr 9, 2012, 21:48 UTC
Message-ID
<7vy5q4tw3i.fsf@alter.siamese.dyndns.org>
In-Reply-To
<m2vcl8bon4.fsf@igel.home>
Andreas Schwab <schwab@linux-m68k.org> writes:
Show 5 quoted lines
> Ben Walton <bwalton@artsci.utoronto.ca> writes:
>
>> +	# Note: The extra [^/] is to work around a bug in Solairs'
>
> s/Solairs/Solaris/

True, but we do not have to single out the bug. The resulting code is easier to understand as it always expect at least one letter between slashes, which holds true because we are handling output from pwd.

I'd rewrite it to:
	# Turn each leading "*/" component into "../"
or something like that.
Previous: Andreas SchwabNext: Ben Walton
Message 9 of 17 in “Avoid bug in Solaris xpg4/sed as used in submodule”
  1. Avoid bug in Solaris xpg4/sed as used in submoduleBen Walton, Apr 9, 2012
  2. Andreas SchwabApr 9, 2012
  3. Ben WaltonApr 9, 2012
  4. Andreas SchwabApr 9, 2012
  5. Junio C HamanoApr 9, 2012
  6. Ben WaltonApr 9, 2012
  7. Avoid bug in Solaris xpg4/sed as used in submoduleBen Walton, Apr 9, 2012
  8. Andreas SchwabApr 9, 2012
  9. Junio C HamanoApr 9, 2012
  10. Avoid bug in Solaris xpg4/sed as used in submoduleBen Walton, Apr 10, 2012
  11. Junio C HamanoApr 10, 2012
  12. Ben WaltonApr 10, 2012
  13. Junio C HamanoApr 10, 2012
  14. Andreas SchwabApr 10, 2012
  15. Junio C HamanoApr 10, 2012
  16. Ben WaltonApr 13, 2012
  17. Avoid bug in Solaris xpg4/sed as used in submoduleBen Walton, Apr 13, 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.