On Fri, Feb 19, 2010 at 06:57, Jeff King <peff@xxxxxxxx> wrote: > 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@xxxxxxxxx> > 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 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html