Re: Bug Report ( including test script ): Non-Fastforward merges misses directory deletion
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