php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Request #58378 Additional SEARCH_PATH location for use with MacPorts
Submitted: 2008-10-14 13:32 UTC Modified: 2008-10-19 17:28 UTC
From: jhohle at godaddy dot com Assigned:
Status: Closed Package: geoip (PECL)
PHP Version: 5.2.5 OS: Mac OS X Leopard (10.5.5)
Private report: No CVE-ID: None
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes.
If you forgot your password, you can retrieve your password here.
Password:
Status:
Package:
Bug Type:
Summary:
From: jhohle at godaddy dot com
New email:
PHP Version: OS:

 

 [2008-10-14 13:32 UTC] jhohle at godaddy dot com
Description:
------------
The config.m4 provided with the geoip package does not incude 
the default MacPorts path, making it difficult to install with 
pear/pecl on Mac OS X.

The following patch to config.m4 allows it to build cleanly:

13c13
<   SEARCH_PATH="/usr/local /usr /sw"
---
>   SEARCH_PATH="/usr/local /usr /sw /opt/local"






Patches

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2008-10-14 14:36 UTC] ohill@php.net
Next time, please send unified diff (using -u).

I'll add this shortly, perhaps tonight.

Thank you for your feedback
 [2008-10-14 14:41 UTC] jhohle at godaddy dot com
Apologies, here's a unified diff for completeness.

--- config.m4	2008-06-12 11:07:19.000000000 -0700
+++ config.m4.mine	2008-10-14 10:12:33.000000000 -0700
@@ -10,7 +10,7 @@
 if test "$PHP_GEOIP" != "no"; then
 
   # --with-geoip -> check with-path
-  SEARCH_PATH="/usr/local /usr /sw"
+  SEARCH_PATH="/usr/local /usr /sw /opt/local"
   SEARCH_FOR="/include/GeoIP.h"
   if test -r $PHP_GEOIP/$SEARCH_FOR; then
     GEOIP_DIR=$PHP_GEOIP
 [2008-10-19 17:28 UTC] ohill@php.net
This bug has been fixed in CVS.

In case this was a documentation problem, the fix will show up at the
end of next Sunday (CET) on pecl.php.net.

In case this was a pecl.php.net website problem, the change will show
up on the website in short time.
 
Thank you for the report, and for helping us make PECL better.

This doesn't justify a release, but it will be in the next one, sitting in CVS for now.

Thank you.
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Fri Dec 27 05:01:27 2024 UTC