Bug 1256587 - Only check rpmbuild, genisoimage and dsymutil on the targets they are relevant for draft
authorMike Hommey <mh+mozilla@glandium.org>
Tue, 15 Mar 2016 18:19:24 +0900
changeset 340452 335041a03821d5886230958bbe12bdfe32301e47
parent 340450 a0159169eb568e9442622819bbe5bec7189e8a4c
child 340690 5c26cd29c3f6f1cff79ec6ebc5dc7870cdc45e86
push id12967
push userbmo:mh+mozilla@glandium.org
push dateTue, 15 Mar 2016 09:20:43 +0000
bugs1256587
milestone48.0a1
Bug 1256587 - Only check rpmbuild, genisoimage and dsymutil on the targets they are relevant for
moz.configure
--- a/moz.configure
+++ b/moz.configure
@@ -109,18 +109,26 @@ def perl_version_check(min_version):
                   'A full perl installation is required.')
 
 perl_version_check('5.006')
 
 
 # Miscellaneous programs
 # ==============================================================
 check_prog('DOXYGEN', ('doxygen',), allow_missing=True)
-check_prog('DSYMUTIL', ('dsymutil', 'llvm-dsymutil'), allow_missing=True)
-check_prog('GENISOIMAGE', ('genisoimage',), allow_missing=True)
-check_prog('RPMBUILD', ('rpmbuild',), allow_missing=True)
 check_prog('UNZIP', ('unzip',))
 check_prog('XARGS', ('xargs',))
 check_prog('ZIP', ('zip',))
 
+@depends(target)
+def mac_programs(target):
+    if target.os == 'Darwin':
+        check_prog('DSYMUTIL', ('dsymutil', 'llvm-dsymutil'), allow_missing=True)
+        check_prog('GENISOIMAGE', ('genisoimage',), allow_missing=True)
+
+@depends(target)
+def linux_programs(target):
+    if target.os == 'GNU' and target.kernel == 'Linux':
+        check_prog('RPMBUILD', ('rpmbuild',), allow_missing=True)
+
 
 # Fallthrough to autoconf-based configure
 include('build/moz.configure/old.configure')