apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jeff Trawick <traw...@gmail.com>
Subject Re: svn commit: r291341 - in /apr/apr/branches/0.9.x: CHANGES file_io/unix/dir.c test/testdir.c
Date Tue, 27 Sep 2005 16:59:16 GMT
On 9/27/05, Joe Orton <jorton@redhat.com> wrote:
> On Tue, Sep 27, 2005 at 01:49:29PM +0100, Joe Orton wrote:
> > On Tue, Sep 27, 2005 at 07:55:14AM -0400, Jeff Trawick wrote:
> > > This chdir && rmdir operation fails with EBUSY on AIX and HP-UX.  (I
> > > see it work on Linux; haven't tested other boxes.)  Is there another
> > > way to accomplish the test?
>
> Can you test whether the trunk code handles this OK?  If so I'll
> backport it everywhere.

I backported* it to 0.9.7 on AIX and the test completes fine now.

*
224a225
>     apr_status_t rv;
236c237,243
<     apr_assert_success(tc, "remove temp dir", apr_dir_remove(path, p));
---
>     rv = apr_dir_remove(path, p);
>     /* Some platforms cannot remove a directory which is in use. */
>     if (rv == APR_SUCCESS) {
>         CuAssert(tc, "fail to create dir",
>                  apr_dir_make_recursive("foobar", APR_OS_DEFAULT,
>                                         p) != APR_SUCCESS);
>     }
238,241d244
<     CuAssert(tc, "fail to create dir",
<              apr_dir_make_recursive("foobar", APR_OS_DEFAULT,
<                                     p) != APR_SUCCESS);
<
242a246,249
>
>     if (rv) {
>         apr_dir_remove(path, p);
>     }

Mime
View raw message