#22556 [Bgs->Opn]: configure file reports bogus mkdir -p failure

ID: 22556 User updated by: john345 at earthlink dot net Reported By: john345 at earthlink dot net -Status: Bogus +Status: Open Bug Type: Compile Failure Operating System: SCO OpenServer 5.0.6 PHP Version: 4.3.1 New Comment: I really don't want to piss anyone off. I really like PHP. But why do stupid things in configure? At least add a comment that says we do this because we want mkdir -p to fail. Better yet, just don't use mkdir -p. There are several valid solutions. Use mkdir without -p. Trap the error output of mkdir -p. Test for the directories before trying to make them. Use rm -rf to remove the directories before creating them. Previous Comments: ------------------------------------------------------------------------ [2003-03-08 10:33:26] sniper@php.net Yeah, right. But it's still not a bug. ------------------------------------------------------------------------ [2003-03-07 16:54:51] john345 at earthlink dot net If it is done twice on purpose, then why do it at all? Doing it twice will always make the test fail. If the issue is the fact that mkdir -p complains when the directory already exists, test for the directory before trying to make it or send the error to dev/null. ------------------------------------------------------------------------ [2003-03-07 00:10:52] sniper@php.net It's actually intentionally done twice. Comment from acinclude.m4: "`mkdir -p' must be quiet about creating existing directories" ------------------------------------------------------------------------ [2003-03-06 15:05:08] john345 at earthlink dot net Tried php4-STABLE-latest.tar.gz and the duplicate line is still in the configure file, just at a different line number, 83186. ------------------------------------------------------------------------ [2003-03-05 13:36:24] sniper@php.net Please try using this CVS snapshot: http://snaps.php.net/php4-STABLE-latest.tar.gz For Windows: http://snaps.php.net/win32/php4-win32-STABLE-latest.zip I think something like this was already fixed.. ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at http://bugs.php.net/22556 -- Edit this bug report at http://bugs.php.net/?id=22556&edit=1