Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 2.4.5

Bug #2006 Invalid ping arguments for RH9
Submitted: 2004-07-28 15:42 UTC Modified: 2007-06-05 16:27 UTC
From: mscifo at php dot net Assigned: cconstantine
Status: Closed Package: Net_Ping
PHP Version: 4.3.8 OS: RH9
Roadmaps: 2.4.2    
Subscription  


 [2004-07-28 15:42 UTC] mscifo at php dot net
Description: ------------ Net_Ping uses the following arguments for the linux ping command... $this->_argRelation["linux"] = array ( "timeout" => "-t", "iface" => NULL, "ttl" => "-m", "count" => "-c", "quiet" => "-q", "size" => "-s", "deadline" => "-w" ); The ping binary on RH9 comes from the iputils-020927 package and accepts different arguments than those listed for linux.

Comments

 [2007-03-12 23:35 UTC] cellog (Greg Beaver)
what are the arguments on RH9?
 [2007-04-20 11:25 UTC] cconstantine (Craig Constantine)
It's the timeout and TTL args that cause the trouble. TIMEOUT... RH8 has no timeout arg (it just uses twice the ttl.) RH9 has timeout as -W. The package presumes -t for generic Linux. TTL... RH8 and RH9 use -t for TTL. The package presumes -m for generic Linux. or visually the args on RH8 and RH9 are: RH8 RH9 -W timeout -I -I iface -t -t ttl -c -c count -q -q quiet -s -s size -w -w deadline Unfortunately a viable patch is NOT simply a tweak to the arg mapping case code. As there are several issues with parsing the ping results as well (ie, RH8 at least has ping output that varies from what he generic Linux result parser expects. It mangles the sequence number, statistics, target IP, etc) I'm tinkering with a patch to rewrite the result parsing to scale better.
 [2007-05-10 21:46 UTC] cconstantine (Craig Constantine)
Added patch which fixes the command args for Redhat 9 (the subject of this bug) *and* Redhat 8. The ping result output parser is still seriously broken. This patch makes no attempt to fix that -- that will be dealt with with separate patches. CAUTION: you need to apply the path which fixes bug #2006 before applying this patch.
 [2007-05-10 22:10 UTC] cconstantine (Craig Constantine)
edit: I meant to say... apply the patch for bug #10788 first.
 [2007-06-05 16:27 UTC] dufuz (Helgi ├×ormar)
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.