Bug 1436255: Allow check_prog('LINKER') to fall back to $PATH. r=glandium
authorDavid Major <dmajor@mozilla.com>
Thu, 08 Feb 2018 12:01:51 +1300
changeset 402830 5d9cab23db6756aeb64142fa577b593b0695c10d
parent 402829 3509bf5962d123b1ae2064979523f8b567e4847a
child 402831 7144fcd531df304bea9bc2031fab6bc56c405095
push id99662
push userdmajor@mozilla.com
push dateWed, 07 Feb 2018 23:00:59 +0000
treeherdermozilla-inbound@5d9cab23db67 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersglandium
bugs1436255
milestone60.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1436255: Allow check_prog('LINKER') to fall back to $PATH. r=glandium
build/moz.configure/windows.configure
--- a/build/moz.configure/windows.configure
+++ b/build/moz.configure/windows.configure
@@ -424,17 +424,17 @@ def valid_mt(path):
     except subprocess.CalledProcessError:
         pass
     raise FatalCheckError('%s is not Microsoft Manifest Tool')
 
 
 set_config('MSMANIFEST_TOOL', depends(valid_mt)(lambda x: bool(x)))
 
 
-link = check_prog('LINKER', ('link.exe',), paths=vc_compiler_path)
+link = check_prog('LINKER', ('link.exe',), paths=toolchain_search_path)
 
 add_old_configure_assignment('LINKER', link)
 
 
 # Normally, we'd just have CC, etc. set to absolute paths, but the build system
 # doesn't currently handle properly the case where the paths contain spaces.
 # Additionally, there's the issue described in toolchain.configure, in
 # valid_compiler().