Description: ------------ Hi, was wondering if you considered updating to Net_CheckIP2. I'm close to a stable release, this is RC3: http://php5.lagged.de/pear/Net_CheckIP2-1.0.0RC3.tgz To switch, just replace the following calls: Net_CheckIP::check_ip($foo); with: Net_CheckIP2::isValid($foo); I also added a bunch of other methods to the package, let me know if you have some time to check it out, I'd appreciate the feedback. Till

Sorry, link is off: http://php5.lagged.de/pear/Net_CheckIP2/trunk/Net_CheckIP2-1.0.0RC3.tgz

I'll check that as soon as it is stable.

Sounds good, also, if you have feedback -- I'm trying to finalize this package and once it's stable, the API will be as-is. So if you have additions beforehand, please let me know.

Why not change dependencies from Net_CheckIP to filter_var with FILTER_VALIDATE_IP?

I just looked at this again since I'm still subscribed. It's probably hard to do either - Net_CheckIP2 or filter_var(). Why? Because Net_DNSBL is a PHP4 package and switching to either solution is a BC break. Net_DNSBL2 is probably due.

If you truly believe that, why not close this as "Wont Fix"?

This change seems to suggest otherwise: http://svn.php.net/viewvc/pear/packages/Net_DNSBL/trunk/Net/DNSBL.php? r1=304645&r2=304646 Applying the patch in bug #17617 will also fix this.

In those changes you made there, you removed the following: require_once 'Net/CheckIP.php'; This results in the following error: PHP Fatal error: Class 'Net_CheckIP' not found in Net/DNSBL.php on line 287 This needs to be fixed asap!

This bug has been fixed in SVN. 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. CheckIP is no longer needed.