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

Re: Bug: rebase when an author uses accents in name on MacOSx

From
LRLanny Ripple <lanny@spotinfluence.com>
Date
May 31, 2012, 14:56 UTC
Message-ID
<938B14B8-EA46-492B-A8C1-15C5E93F22AC@spotinfluence.com>
In-Reply-To
<87bol4ifvn.fsf@thomas.inf.ethz.ch>
A platform sed that is broken would be worked around by recognizing the platform.  If the bulk of platforms are sane then fixing the path for the platform is the easiest solution.  If the bulk of platforms don't have a sane sed then not using sed is probably the solution.  If you want to make the path variable then a platform config file built automatically figuring out default paths that could be changed by the host's admin comes to mind (much like, oh I don't know, building a list of shell variables named author-script to use during rebases).
You could modify the name, email, and timestamp so they didn't have to be parsed.  Separate lines in the header for name, email, and date would work.  A length encoding of the three fields at the beginning of the header line or in some other (new) location if backwards compatibility is a concern.  There's lots of rather straight-forward technical solutions.
Now if what you are saying is the social cost of applying any of those solutions is too high that's a horse of a different color.
Anyway thanks to everyone for the clue that got me working again,
  -ljr
---
Lanny Ripple
lanny@spotinfluence.com
On May 31, 2012, at 9:28 AM, Thomas Rast wrote:
Show 19 quoted lines
> Lanny Ripple <lanny@spotinfluence.com> writes:
> 
>> Bingo.
>> 
>>  lanny;~> echo "Rémi Leblond" | LANG=C LC_ALL=C /usr/bin/sed -ne 's/.*/GIT_AUTHOR_NAME='\''&'\''/p'
>>  GIT_AUTHOR_NAME='Rémi Leblond'
>> 
>> Just occurred to me that I'm using fink and that git-am doesn't use
>> /usr/bin/sed but just sed.  My suggestion is to be explicit on the
>> path in git-am.
> 
> Then how would you work around a platform sed that is broken?  You can't
> just install a new one in another directory if we use an absolute path.
> Which apparently is what people have to do on Solaris, see the
> SANE_TOOL_PATH setting for the Makefile.
> 
> -- 
> Thomas Rast
> trast@{inf,student}.ethz.ch
Previous: Thomas RastNext: Junio C Hamano
Message 8 of 21 in “Bug: rebase when an author uses accents in name on MacOSx”
  1. Lanny RippleMay 30, 2012
  2. Junio C HamanoMay 30, 2012
  3. Jürgen KreilederMay 30, 2012
  4. Jeff KingMay 31, 2012
  5. Junio C HamanoMay 31, 2012
  6. Lanny RippleMay 31, 2012
  7. Thomas RastMay 31, 2012
  8. Lanny RippleMay 31, 2012
  9. Junio C HamanoMay 31, 2012
  10. Lanny RippleMay 31, 2012
  11. Junio C HamanoMay 31, 2012
  12. Lanny RippleMay 31, 2012
  13. Jeff KingJun 1, 2012
  14. Lanny RippleJun 1, 2012
  15. Jeff KingJun 2, 2012
  16. Lanny RippleJun 2, 2012
  17. Junio C HamanoJun 1, 2012
  18. Lanny RippleJun 1, 2012
  19. Junio C HamanoJun 1, 2012
  20. Jeff KingJun 2, 2012
  21. Thomas RastMay 31, 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.