[REVIEW 3-6] fdo#53006: Remove directories too, not just plain files

Petr Mladek pmladek at suse.cz
Mon Aug 13 02:38:05 PDT 2012


Stephan Bergmann píše v Pá 10. 08. 2012 v 19:41 +0200:
> On 08/10/2012 05:29 PM, Petr Mladek wrote:
> > Just to be sure. removeTree(url) removes only files and not directories.
> > Is it by intention? I think that both variants would be fine.
> 
> Oh, good catch.  No, it was an oversight to not remove directories.  So 
> <http://cgit.freedesktop.org/libreoffice/core/commit/?id=0ab5ce54c441b12ea6c12a2cedb9069f36cfd0f4> 
> "fdo#53006: Remove directories too, not just plain files" fixes that.  I 
> think it is best to backport it to libreoffice-3-6, too.  (Seen working 
> as intended on both Linux and Windows master.)

Yup, looks and works fine => pushed into 3-6 branch.

Thanks for the fix.


Best Regards,
Petr




More information about the LibreOffice mailing list