build: fix default dtrace flag on Linux
authorShigeki Ohtsu <ohtsu@iij.ad.jp>
Thu, 1 Nov 2012 05:14:46 +0000 (14:14 +0900)
committerBen Noordhuis <info@bnoordhuis.nl>
Thu, 1 Nov 2012 21:20:21 +0000 (22:20 +0100)
DTrace on Linux should not be enabled by default because not all systems will
have the proper headers installed. Only enable when --with-dtrace is passed to
the configure script.

configure

index 48cc610..bd6e0cd 100755 (executable)
--- a/configure
+++ b/configure
@@ -134,7 +134,7 @@ parser.add_option("--shared-zlib-libname",
 parser.add_option("--with-dtrace",
     action="store_true",
     dest="with_dtrace",
-    help="Build with DTrace (default is true on supported systems)")
+    help="Build with DTrace (default is true on sunos)")
 
 parser.add_option("--without-dtrace",
     action="store_true",
@@ -386,7 +386,7 @@ def configure_node(o):
     o['variables']['node_use_dtrace'] = b(not options.without_dtrace)
   elif sys.platform.startswith('linux'):
     o['variables']['node_use_dtrace'] = 'false'
-    o['variables']['node_use_systemtap'] = b(not options.without_dtrace)
+    o['variables']['node_use_systemtap'] = b(options.with_dtrace)
     if options.systemtap_includes:
       o['include_dirs'] += [options.systemtap_includes]
   elif b(options.with_dtrace) == 'true':
@@ -394,6 +394,7 @@ def configure_node(o):
        'DTrace is currently only supported on SunOS or Linux systems.')
   else:
     o['variables']['node_use_dtrace'] = 'false'
+    o['variables']['node_use_systemtap'] = 'false'
 
   if options.no_ifaddrs:
     o['defines'] += ['SUNOS_NO_IFADDRS']