apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Lucian Adrian Grijincu" <lucian.griji...@gmail.com>
Subject Re: [PATCH] apr_dir_remove_recursive
Date Tue, 15 Apr 2008 15:14:36 GMT
How about this?
It got some review in the past[1]
is it fit for inclusion?

[1] http://www.mail-archive.com/dev@apr.apache.org/msg17650.html

On Mon, Oct 29, 2007 at 1:13 AM, Lucian Adrian Grijincu
<lucian.grijincu@gmail.com> wrote:
> Attached a patch that adds apr_dir_remove_recursive for trunk/
>
>  I've only tested this on Ubuntu 7.10.
>  It ads a new "fileutil.c" file in unix/
>
>  Someone should test this, I currently cannot (OS not available):
>  1) I've also added a #SOURCE to apr.dsp and libapr.dsp for windows
>     build referencing the unix file.
>  2) Added a reference in NWGNUmakefile too.
>  3) For os2 there's a new file fileutil.c that #includes the unix/ equivalent.
>
>  There's a testcase too.
>
>  BTW, I wanted to add a test for deleting symlinks, but there is no APR
>  API to create a symlink and on another thread on this list someone
>  expressed their opposition for platform specific tests. What should be
>  done in this case?



-- 
Lucian

Mime
View raw message