highperfocused/bitcoin
1
0
Fork 0
mirror of https://github.com/bitcoin/bitcoin.git synced 2025-05-15 22:31:38 +02:00
Code Issues Packages Projects Releases Wiki Activity

scripts: check macOS SDK version is set

Browse Source 
Clangs Darwin driver should infer the SDK version used during compilation, and
forward that through to the linker. Add a check that this has been done, and the
expected SDK version is set.

Should help prevent issues like #21771 in future.
This commit is contained in:
fanquake 2021-05-09 16:18:53 +08:00
parent c972345bac
commit aa80b5759d
No known key found for this signature in database
GPG Key ID: 2EEB9F5CC09526C1
2 changed files with 10 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
contrib/devtools/symbol-check.py
View File

@ -218,6 +218,12 @@ def check_MACHO_min_os(filename) -> bool:
return True return True
return False return False
def check_MACHO_sdk(filename) -> bool:
binary = lief.parse(filename)
if binary.build_version.sdk == [10, 15, 6]:
return True
return False
def check_PE_libraries(filename) -> bool: def check_PE_libraries(filename) -> bool:
ok: bool = True ok: bool = True
binary = lief.parse(filename) binary = lief.parse(filename)
@ -244,6 +250,7 @@ CHECKS = {
'MACHO': [ 'MACHO': [
('DYNAMIC_LIBRARIES', check_MACHO_libraries), ('DYNAMIC_LIBRARIES', check_MACHO_libraries),
('MIN_OS', check_MACHO_min_os), ('MIN_OS', check_MACHO_min_os),
('SDK', check_MACHO_sdk),
], ],
'PE' : [ 'PE' : [
('DYNAMIC_LIBRARIES', check_PE_libraries), ('DYNAMIC_LIBRARIES', check_PE_libraries),

6
contrib/devtools/test-symbol-check.py
View File

@ -98,7 +98,7 @@ class TestSymbolChecks(unittest.TestCase):
self.assertEqual(call_symbol_check(cc, source, executable, ['-lexpat']), self.assertEqual(call_symbol_check(cc, source, executable, ['-lexpat']),
(1, 'libexpat.1.dylib is not in ALLOWED_LIBRARIES!\n' + (1, 'libexpat.1.dylib is not in ALLOWED_LIBRARIES!\n' +
executable + ': failed DYNAMIC_LIBRARIES MIN_OS')) f'{executable}: failed DYNAMIC_LIBRARIES MIN_OS SDK'))
source = 'test2.c' source = 'test2.c'
executable = 'test2' executable = 'test2'
@ -114,7 +114,7 @@ class TestSymbolChecks(unittest.TestCase):
''') ''')
self.assertEqual(call_symbol_check(cc, source, executable, ['-framework', 'CoreGraphics']), self.assertEqual(call_symbol_check(cc, source, executable, ['-framework', 'CoreGraphics']),
(1, executable + ': failed MIN_OS')) (1, f'{executable}: failed MIN_OS SDK'))
source = 'test3.c' source = 'test3.c'
executable = 'test3' executable = 'test3'
@ -127,7 +127,7 @@ class TestSymbolChecks(unittest.TestCase):
''') ''')
self.assertEqual(call_symbol_check(cc, source, executable, ['-mmacosx-version-min=10.14']), self.assertEqual(call_symbol_check(cc, source, executable, ['-mmacosx-version-min=10.14']),
(0, '')) (1, f'{executable}: failed SDK'))
def test_PE(self): def test_PE(self):
source = 'test1.c' source = 'test1.c'