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

Re: Bug Report ( including test script ): Non-Fastforward merges misses directory deletion

From
Alex Riesen <raa.lkml@gmail.com>
Date
Feb 19, 2010, 07:40 UTC
Message-ID
<81b0412b1002182340w71aa6364tfe2e17ad6fd1b1e8@mail.gmail.com>
In-Reply-To
<20100219055721.GC22645@coredump.intra.peff.net>
On Fri, Feb 19, 2010 at 06:57, Jeff King <peff@peff.net> wrote:
Show 6 quoted lines
> On Thu, Feb 18, 2010 at 11:43:23AM +0000, Sebastian Thiel wrote:
> If we remove a path in a/deep/subdirectory, we should try to
> remove as many trailing components as possible (i.e.,
> subdirectory, then deep, then a). However, the test for the
> return value of rmdir was reversed, so we only ever deleted
> at most one level.
What was I thinking!
Sincerely-sorry: Alex Riesen <raa.lkml@gmail.com>
> This was introduced by Alex's 4a92d1b (Add remove_path: a function to
> remove as much as possible of a path, 2008-09-27), which ironically
> complained about bugs in the code it was replacing. :)
Well, it did fix the bugs it claimed to fix
Previous: Junio C Hamano
Message 5 of 5 in “Bug Report ( including test script ): Non-Fastforward merges misses directory deletion”
  1. Sebastian ThielFeb 18, 2010
  2. Sebastian ThielFeb 18, 2010
  3. Jeff KingFeb 19, 2010
  4. Junio C HamanoFeb 19, 2010
  5. Alex RiesenFeb 19, 2010

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.