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

Re: [PATCH] filter-branch: use printf instead of echo -e

From
Michele Locati <michele@locati.it>
Date
Mar 20, 2018, 10:21 UTC
Message-ID
<d2bf28a9-ae79-9046-22f4-ceac502882f6@locati.it>
In-Reply-To
<1521539621.2474.60.camel@hellion.org.uk>
Il 20/03/2018 10:53, Ian Campbell ha scritto:
Show 12 quoted lines
> On Tue, 2018-03-20 at 00:22 -0400, Jeff King wrote:
>> Author cc'd in case there's something more interesting going on.
> 
> That code was written years ago, if I had a good reason at the time
> I've forgotten what it was and I can't think of a fresh one now.
> Switching to printf seems like a reasonable thing to do.
> 
> Perhaps switch the remaining `/bin/echo` (there are two without `-e`)
> uses to just `echo` for consistency with the rest of the file?
> 
> Ian.
>

Thanks for reply, Ian. Charles already suggested to replace /bin/echo with echo, and I already updated the patch accordingly: see https://marc.info/?l=git&m=152147479517707&w=2 Junio already queued that PATCH-v2.

-- Michele

Previous: Ian Campbell
Message 10 of 10 in “filter-branch: use printf instead of echo -e”
  1. filter-branch: use printf instead of echo -eMichele Locati, Mar 19, 2018
  2. filter-branch: use printf instead of echo -eMichele Locati, Mar 19, 2018
  3. Junio C HamanoMar 19, 2018
  4. Johannes SchindelinMar 21, 2018
  5. Michele LocatiMar 22, 2018
  6. Johannes SchindelinMar 23, 2018
  7. CB BaileyMar 19, 2018
  8. Jeff KingMar 20, 2018
  9. Ian CampbellMar 20, 2018
  10. Michele LocatiMar 20, 2018

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.