[108283] trunk/dports/www/nginx/Portfile

cal at macports.org cal at macports.org
Wed Jul 17 15:19:54 PDT 2013


Revision: 108283
          https://trac.macports.org/changeset/108283
Author:   cal at macports.org
Date:     2013-07-17 15:19:53 -0700 (Wed, 17 Jul 2013)
Log Message:
-----------
nginx: abort if google-perftools is broken

Modified Paths:
--------------
    trunk/dports/www/nginx/Portfile

Modified: trunk/dports/www/nginx/Portfile
===================================================================
--- trunk/dports/www/nginx/Portfile	2013-07-17 22:10:13 UTC (rev 108282)
+++ trunk/dports/www/nginx/Portfile	2013-07-17 22:19:53 UTC (rev 108283)
@@ -122,6 +122,18 @@
 variant google_perftools description {Enable Google Performance Tools profiling for workers} {
     configure.args-append   --with-google_perftools_module
     depends_lib-append      port:google-perftools
+
+    # If 'google_perftools' port doesn't find headers needed for the profiler lib, it just quietly skips it.
+    # Check if "-lprofiler" will fail -- if so, pre-empt configure-stage error with a more useful error msg.
+    pre-configure {
+        if {![file exists "${prefix}/lib/libprofiler.dylib"] && ![file exists "${prefix}/lib/libprofiler.a"]} {
+            ui_error "\
+                The 'google-perftools' port did not install a libprofiler library (libprofiler.dylib or\
+                libprofiler.a) on your version of OS X, but the +google_perftools variant will not work\
+                without this library. Disable the +google_perftools variant to continue installing ${name}"
+            error "Required libprofiler library missing from google-perftools port."
+        }
+    }
 }
 
 variant gzip_static description {Avoids compressing the same file each time it is requested} {
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.macosforge.org/pipermail/macports-changes/attachments/20130717/d5c25e82/attachment.html>


More information about the macports-changes mailing list