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

Re: shell compatibility issues with SunOS 5.10

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
May 6, 2009, 09:38 UTC
Message-ID
<alpine.DEB.1.00.0905061137300.14230@intel-tinevez-2-302>
In-Reply-To
<loom.20090506T091628-717@post.gmane.org>
Hi,
On Wed, 6 May 2009, Ralf Wildenhues wrote:
Show 39 quoted lines
> Nguyen Thai Ngoc Duy writes:
> > On Wed, May 6, 2009 at 4:45 PM, Johannes Sixt wrote:
> > > Nguyen Thai Ngoc Duy schrieb:
> > >>       # normalize path:
> > >>       # multiple //; leading ./; /./; /../; trailing /
> > >>       path=$(printf '%s/\n' "$path" |
> > >>               sed -e '
> > >>                       s|//*|/|g
> > >>                       s|^\(\./\)*||
> > >>                       s|/\./|/|g
> > >>                       :start
> > >>                       s|\([^/]*\)/\.\./||
> > >>                       tstart
> > >>                       s|/*$||
> > >>               ')
> 
> > It says nothing. The result of "printf '%s\n' ./foo/bar | sed -e blah"
> > is just wrong, (i.e. "./" remains). I stripped down to "sed -e
> > 's|^\(\./\)*||'", does not work. Probably due to \( \) pair. Skimmed
> > through sed manpage, seems no mention of bracket grouping.
> 
> Quoting 'info Autoconf "Limitation of Usual Tools"':
> 
>      Some `sed' implementations, e.g., Solaris, restrict the special
>      role of the asterisk to one-character regular expressions.  This
>      may lead to unexpected behavior:
> 
>           $ echo '1*23*4' | /usr/bin/sed 's/\(.\)*/x/g'
>           x2x4
>           $ echo '1*23*4' | /usr/xpg4/bin/sed 's/\(.\)*/x/g'
>           x
> 
> You can work around it in this case with
>   :again
>   s|^\./||
>   t again
> 
> BTW, you should put a space between t and the label (but not between
> : and label), POSIX requires that and some sed versions expect it.

Maybe the time is better spent on turning submodule into a builtin, before it gets even larger, and before we have to jump through even more hoops because of shell compatibility issues?

Ciao, Dscho

Previous: Ralf WildenhuesNext: Nguyen Thai Ngoc Duy
Message 8 of 31 in “shell compatibility issues with SunOS 5.10”
  1. Nguyen Thai Ngoc DuyMay 6, 2009
  2. Junio C HamanoMay 6, 2009
  3. Nguyen Thai Ngoc DuyMay 6, 2009
  4. Nguyen Thai Ngoc DuyMay 7, 2009
  5. Johannes SixtMay 6, 2009
  6. Nguyen Thai Ngoc DuyMay 6, 2009
  7. Ralf WildenhuesMay 6, 2009
  8. Johannes SchindelinMay 6, 2009
  9. Nguyen Thai Ngoc DuyMay 6, 2009
  10. Jeff KingMay 6, 2009
  11. Brandon CaseyMay 6, 2009
  12. 0/4 workaround some Solaris sed issuesBrandon Casey, May 6, 2009
  13. 1/4 t4118: add missing '&&'Brandon Casey, May 6, 2009
  14. 2/4 t4118: avoid sed invocation on file without terminating newlineBrandon Casey, May 6, 2009
  15. 3/4 t/annotate-tests.sh: avoid passing a non-newline terminated file to sedBrandon Casey, May 6, 2009
  16. 4/4 t4200: avoid passing a non-newline terminated file to sedBrandon Casey, May 6, 2009
  17. Junio C HamanoMay 6, 2009
  18. Brandon CaseyMay 6, 2009
  19. Junio C HamanoMay 6, 2009
  20. 1/2 t4200: remove two unnecessary linesBrandon Casey, May 6, 2009
  21. 2/2 t4200: convert sed expression which operates on non-text file to perlBrandon Casey, May 6, 2009
  22. Nguyen Thai Ngoc DuyMay 6, 2009
  23. Nguyen Thai Ngoc DuyMay 7, 2009
  24. Brandon CaseyMay 7, 2009
  25. Junio C HamanoMay 7, 2009
  26. Johannes SixtMay 7, 2009
  27. Brandon CaseyMay 7, 2009
  28. Nguyen Thai Ngoc DuyMay 6, 2009
  29. Brandon CaseyMay 7, 2009
  30. Junio C HamanoMay 7, 2009
  31. Brandon CaseyMay 7, 2009

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.