Federico Gimenez Nieto wrote:
> Hi Thomas,
>
> El 05/09/2010, a las 01:10, Thomas Goirand escribió:
>
>> Hi,
>>
>> I don't agree. Some scripts (like mine) run with error_reporting(E_ALL);
>> In my case, I run it from a cron job, and this would send me a mail
>> every 10 minutes (because it outputs to the standard error).
>
> Did you try the test case that i attached on the previous message? With
> error_reporting(E_ALL) it serializes a xml document to a file and then
> reads and unserializes it back without noticing any error or warning
> interfering the overall operation. The problem may be related to your code.
The following code is enough to produce a warning:
<?php
error_reporting(E_ALL);
include("/usr/share/php/XML/Unserializer.php");
?>
The thing is, if one day we backport PHP6 in Squeeze, we'll have a big
crash. So it's better to fix things when we can. Also, some applications
may run with E_ALL, so I think it was best to update it.
> As i can't reproduce the bug i can't prepare a fix for it. Don't
> hesitate to make the NMU if you feel that it is needed and you are sure
> that it won't introduce any flaws.
I did many NMU of the same kind, none had issues. I just did an NMU for
your package with the corrections. Then I ran your test.php and it
worked, so I don't think there's an issue here.
By the way, I've seen your package, and saw quite some mistakes. Like
the doc is installed in /usr/share/php/doc, and same for many things. I
believe the package should be reworked quite a bit (but as Squeeze is
released, this might not be the time to do it, unless you want to upload
in Experimental).
> Anyway, in my opinion it would be a
> good idea to report this upstream, it can be done at [1]
Yup, I've sent the patch to them. Thanks for reminding it to me! :)
Thomas Goirand (zigo)
--
To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
