#30318 [Bgs->Opn]: Bzip2 warning required

ID: 30318 User updated by: aidan@php.net Reported By: aidan@php.net -Status: Bogus +Status: Open Bug Type: Livedocs problem PHP Version: Irrelevant New Comment: Failing to intialise a variable before using it IS a bug, I don't see how this is any different. This patch ensures that the bzip2 compression stream is enabled before it is used. This patch will stop error notices being spewed over manual pages. The informative message could be omited, but as there is already a bug regarding this exact problem (#29521) I cannot see why it would be a problem. I ofcourse agree with Derick, it should ALSO go in the readme. Previous Comments: ------------------------------------------------------------------------ [2004-10-04 13:58:54] aidan@php.net Here's the patch formatted correctly: http://www.powertrip.co.za/livedocs/viewpatch.php?patch=7 ------------------------------------------------------------------------ [2004-10-04 13:51:08] derick@php.net No bug here, this should go into the README instead. ------------------------------------------------------------------------ [2004-10-04 13:45:44] aidan@php.net Description: ------------ This patch adds a check to ensure the bzip2 compression stream is enabled before using it. Actual result: -------------- --- livedocs\mk_notes.1.4.php Thu Aug 19 06:56:57 2004 +++ livedocs\mk_notes.php Wed Sep 08 09:35:08 2004 @@ -21,6 +21,14 @@ // // $Id: mk_notes.php,v 1.4 2004/08/18 20:56:57 iliaa Exp $ +// Check the compress.bzip2 stream is enabled +if (!in_array('compress.bzip2', stream_get_wrappers())) { + echo "Warning: Unable to build user notes: The bzip2 stream is not enabled.\n"; + echo "You must build with --with-bz2.\n"; + echo "####################\n\n"; + return false; +} + $create = << ------------------------------------------------------------------------ -- Edit this bug report at http://bugs.php.net/?id=30318&edit=1