Closed Bug 556451 Opened 14 years ago Closed 14 years ago

rmdir NSPR test fails on Android

Categories

(NSPR :: NSPR, defect)

ARM
Android
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: alexp, Assigned: wtc)

Details

Attachments

(1 obsolete file)

The test prints "remove directory succeeds", but returns a non-zero code.
Attached patch Fix (obsolete) — Splinter Review
Return a proper exit code.
Attachment #436402 - Flags: review?(bugmail)
Comment on attachment 436402 [details] [diff] [review]
Fix

I believe the PR_RmDir succeeding is the failure state for this test since its trying to remove a non-empty directory.

Did you try running your patched test on an existing platform?
Attachment #436402 - Flags: review?(bugmail) → review-
Comment on attachment 436402 [details] [diff] [review]
Fix

That's right - I got it wrong. The rmdir must fail for the test to succeed.

Apparently there was a bug in the older versions on Android libc - exactly related to this: rmdir successfully deleted non-empty dir.
Seems like it was fixed recently, and now this test succeeds on the Nexus One with the latest system update.
Attachment #436402 - Attachment is obsolete: true
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: