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

Re: [PATCH] t4127-apply-same-fn: Avoid sed -i

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Jul 4, 2008, 18:39 UTC
Message-ID
<alpine.DEB.1.00.0807041938360.9925@racer>
In-Reply-To
<486DC687.4070307@viscovery.net>
Hi,
On Fri, 4 Jul 2008, Johannes Sixt wrote:
> > When I created the tests for my git-apply patch, I accidently used the 
> > '-i' flag for sed.

Maybe we should modify our hooks so that this is warned about? While at it, we could warn about quoted "wc -l", too...

Ciao, Dscho

Previous: Johannes Sixt
Message 3 of 3 in “git-apply tests need to be portable”
  1. git-apply tests need to be portableDon Zickus, Jul 3, 2008
  2. t4127-apply-same-fn: Avoid sed -iJohannes Sixt, Jul 4, 2008
  3. Johannes SchindelinJul 4, 2008

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.