From: Johannes Schindelin Date: Thu, 13 Aug 2015 07:32:14 GMT Subject: Re: [PATCH jk/prune-mtime] prune: close directory earlier during loose-object directory traversal Message-ID: <557753b4c14eb3f90270007782bc3a73@www.dscho.org> In-Reply-To: <55CB85A5.5040802@kdbg.org> Hi, On 2015-08-12 19:43, Johannes Sixt wrote: > 27e1e22d (prune: factor out loose-object directory traversal, 2014-10-16) > introduced a new function for_each_loose_file_in_objdir() with a helper > for_each_file_in_obj_subdir(). The latter calls callbacks for each file > found during a directory traversal and finally also a callback for the > directory itself. > > git-prune uses the function to clean up the object directory. In > particular, in the directory callback it calls rmdir(). On Windows XP, > this rmdir call fails, because the directory is still open while the > callback is called. Close the directory before calling the callback. > > Signed-off-by: Johannes Sixt > --- > My Windows 8.1 machine does not require this fix for some unkonwn > reason. But we still cater for Windows XP users, where this change > is a real improvement. I believe that we have a concrete bug report for that: https://github.com/git-for-windows/git/issues/231 Ciao, Johannes -- -- *** Please reply-to-all at all times *** *** (do not pretend to know who is subscribed and who is not) *** *** Please avoid top-posting. *** The msysGit Wiki is here: https://github.com/msysgit/msysgit/wiki - Github accounts are free. You received this message because you are subscribed to the Google Groups "msysGit" group. To post to this group, send email to msysgit@googlegroups.com To unsubscribe from this group, send email to msysgit+unsubscribe@googlegroups.com For more options, and view previous threads, visit this group at http://groups.google.com/group/msysgit?hl=en_US?hl=en --- You received this message because you are subscribed to the Google Groups "Git for Windows" group. To unsubscribe from this group and stop receiving emails from it, send an email to msysgit+unsubscribe@googlegroups.com. For more options, visit https://groups.google.com/d/optout.