Backed out changeset 6134b92d1b08 (bug 1277483)
authorCarsten "Tomcat" Book <cbook@mozilla.com>
Tue, 07 Jun 2016 11:46:16 +0200
changeset 300864 15fb9fb5775c432966d9ff321283111dfb0cf8ea
parent 300863 c1c93849181d026f852b9217693c2b6818c07bb3
child 300865 d818934b7acaeeea56ba390129c8f74424ac7a96
push id19599
push usercbook@mozilla.com
push dateWed, 08 Jun 2016 10:16:21 +0000
treeherderfx-team@81f4cc3f6f4c [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs1277483
milestone50.0a1
backs out6134b92d1b089f3a7d3e36025d52f491befb4ae1
Backed out changeset 6134b92d1b08 (bug 1277483)
build/moz.configure/toolchain.configure
python/mozbuild/mozpack/executables.py
--- a/build/moz.configure/toolchain.configure
+++ b/build/moz.configure/toolchain.configure
@@ -617,12 +617,11 @@ def enable_strip(value):
 set_config('ENABLE_STRIP', enable_strip)
 
 
 option('--disable-install-strip',
        help='Disable stripping of libs & executables when packaging')
 
 @depends('--disable-install-strip')
 def disable_install_strip(value):
-    if not value:
-        return True
+    return not value
 
 set_config('PKG_SKIP_STRIP', disable_install_strip)
--- a/python/mozbuild/mozpack/executables.py
+++ b/python/mozbuild/mozpack/executables.py
@@ -80,17 +80,17 @@ def is_executable(path):
     return get_type(path) != UNKNOWN
 
 
 def may_strip(path):
     '''
     Return whether strip() should be called
     '''
     from buildconfig import substs
-    return not substs.get('PKG_SKIP_STRIP')
+    return not substs['PKG_SKIP_STRIP']
 
 
 def strip(path):
     '''
     Execute the STRIP command with STRIP_FLAGS on the given path.
     '''
     from buildconfig import substs
     strip = substs['STRIP']