#38413 [Opn->Csd]: ext/standard/tests/file/bug22414.phpt unnecessary code

ID: 38413 Updated by: tony2001@php.net Reported By: chtekk at gentoo dot org -Status: Open +Status: Closed -Bug Type: *General Issues +Bug Type: Unknown/Other Function Operating System: Gentoo Linux PHP Version: 4.4.3 New Comment: This bug has been fixed in CVS. 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. Previous Comments: ------------------------------------------------------------------------ [2006-08-10 13:11:18] chtekk at gentoo dot org Description: ------------ While running the test suite, Gentoo's sandbox (a mechanism to keep the building and testing of packages separate from the actual system, fex. denieing random access to files outside the build dir) failed telling me about a "unlink /passthru_test" in ext/standard/tests/file/bug22414.phpt, checking that test shows that on both PHP4_4, 5_1 as well as 5_2 branches it has the following code in it: @unlink($pwd . '/passthru_test'); That code seems unnecessary and useless to me, as well as incorrect. $pwd never gets defined or used in any way in the rest of that test, and that passthru_test file is not used anywhere. Please fix the test by removing that line, you'll find working diffs for PHP4 and PHP5 at the following urls: http://overlays.gentoo.org/proj/php/browser/patches/php-patches/4.4.3/4.4.3/php4.4.3-test_bug22414.patch?format=txt http://overlays.gentoo.org/proj/php/browser/patches/php-patches/5.1.4/5.1.4/php5.1.4-test_bug22414.patch?format=txt Expected result: ---------------- A working patch. ;) ------------------------------------------------------------------------ -- Edit this bug report at http://bugs.php.net/?id=38413&edit=1