| Package home | Report new bug | New search | Development Roadmap | Status: Open | Feedback | All | Closed Since Version 0.2.5 |
| Bug #14902 | Unit tests vs Notice: Only variable references should be returned by reference | ||
|---|---|---|---|
| Submitted: | 2008-10-27 22:50 UTC | Modified: | 2008-11-10 00:45 UTC |
| From: | doconnor | Assigned: | doconnor |
| Status: | Closed | Package: | Net_Monitor (version CVS) |
| PHP Version: | 5.2.4 | OS: | |
| Roadmaps: | (Not assigned) | ||
[2008-10-27 22:50 UTC] doconnor
(Daniel O'Connor)
Description:
------------
Unit tests are failing in CVS.
Test script:
---------------
pear run-tests
Expected result:
----------------
Tests pass
Actual result:
--------------
57) /home/clockwerx/pear/Net_Monitor/tests/multiple.phpt
--- Expected
+++ Actual
@@ -0,0 +1,5 @@
+Notice: Only variable references should be returned by reference in /home/clockwerx/pear/Net_Monitor/Monitor.php on line 361
+
+Notice: Only variable references should be returned by reference in /home/clockwerx/pear/Net_Monitor/Monitor.php on line 361
+
+ ** ERROR: process timed out **
\ No newline at end of file
Comments
[2008-11-01 08:28 UTC] cmv
(Colin Viebrock)
Their are no lo ger any active developers for this package. If you can
submit a patch to fix this bug, I'll apply it. Otherwise it'll have to go
unfixed until someone else can maintain this package.
[2008-11-01 09:39 UTC] doconnor
(Daniel O'Connor)
If you would like, Colin, I can mark this pkg as unmaintained then pear qa will look after it until new maintainers take over :)
[2008-11-10 00:45 UTC] doconnor
(Daniel O'Connor)
This bug has been fixed in CVS.
If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET).
If this was a problem with the pear.php.net website, the change should be live shortly.
Otherwise, the fix will appear in the package's next release.
Thank you for the report and for helping us make PEAR better.
This is fixed in CVS, but now there are network problems breaking the tests