php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Bug #53867 Test failure.
Submitted: 2011-01-28 15:20 UTC Modified: 2011-01-28 21:42 UTC
From: rquadling@php.net Assigned: felipe (profile)
Status: Closed Package: Zip Related
PHP Version: 5.3SVN-2011-01-28 (SVN) OS: Windows XP SP3
Private report: No CVE-ID: None
View Developer Edit
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes.
If you forgot your password, you can retrieve your password here.
Password:
Status:
Package:
Bug Type:
Summary:
From: rquadling@php.net
New email:
PHP Version: OS:

 

 [2011-01-28 15:20 UTC] rquadling@php.net 
Description:
------------
The ext/zip/tests/stream_meta_data.phpt test fails on windows.

The issue is in the URI test.

zip:///%stest_with_comment.zip#foo

I read this as ...

stream zip://
root /
path %s
file#query test_with_comments.zip#foo

On Windows the root is incorrect. It will be C: or D: (for example).

As the path is picked up using %s, the attached patch should work on non-windows 
as well as windows.

Patches

stream_meta_data_test_fix.patch (last revision 2011-01-28 14:20 UTC by rquadling)

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2011-01-28 21:41 UTC] felipe@php.net 
Automatic comment from SVN on behalf of felipe
Revision: http://svn.php.net/viewvc/?view=revision&revision=307829
Log: - Fixed bug #53867 (Test failure)
  patch by: rquadling at php.net
 [2011-01-28 21:42 UTC] felipe@php.net
-Status: Open +Status: Closed -Assigned To: +Assigned To: felipe
 [2011-01-28 21:42 UTC] felipe@php.net 
This bug has been fixed in SVN.

Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.
 
Thank you for the report, and for helping us make PHP better.
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved. 		Last updated: Fri Dec 27 10:01:28 2024 UTC