binarycrusader / libproxy

Automatically exported from code.google.com/p/libproxy
GNU Lesser General Public License v2.1
0 stars 0 forks source link

Perl bindings cause build to fail on OS X #106

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
$ cmake ../
MODULES TO BUILD:
        config_envvar
        config_gnome
        config_kde4
    *   config_macosx
        config_w32reg
    *   ignore_domain
    *   ignore_hostname
    *   ignore_ip
        network_networkmanager
        pacrunner_mozjs
    *   pacrunner_webkit
    *   wpad_dns_alias

-- Could NOT find PerlLibs  (missing:  PERL_LIBRARY PERL_INCLUDE_PATH)
CMake Error: The following variables are used in this project, but they are 
set to NOTFOUND.
Please set them or make sure they are set and tested correctly in the CMake 
files:
PERL_INCLUDE_PATH (ADVANCED)
   used as include directory in directory 
/Users/nathaniel/Documents/libproxy/bindings/perl/src
PERL_LIBRARY (ADVANCED)
    linked by target "PLlibproxy" in directory 
/Users/nathaniel/Documents/libproxy/bindings/perl/src

-- Configuring incomplete, errors occurred!

Original issue reported on code.google.com by npmccallum@gmail.com on 29 Mar 2010 at 5:31

GoogleCodeExporter commented 9 years ago
Proposed patch:

Index: bindings/perl/CMakeLists.txt
===================================================================
--- bindings/perl/CMakeLists.txt    (revision 654)
+++ bindings/perl/CMakeLists.txt    (working copy)
@@ -1,7 +1,7 @@
 find_package(Perl)
 include (FindPerlLibs)

-if(PERL_FOUND)
+if(PERL_FOUND and PERLLIBS_FOUND)

   # Some distributions install perl packages in vendor when shipped with the distro.
   # Let's make their lifes easier by offering an install flag for this usecase.

Original comment by dominiqu...@gmail.com on 30 Mar 2010 at 7:52

GoogleCodeExporter commented 9 years ago
Fixed in r655

Original comment by npmccallum@gmail.com on 30 Mar 2010 at 12:46

GoogleCodeExporter commented 9 years ago

Original comment by nicolas.dufresne@gmail.com on 18 May 2010 at 3:04