merge bitcoin#21255: run test-symbol-check for RISC-V

This commit is contained in:
Kittywhiskers Van Gogh 2021-02-21 10:11:13 +08:00
parent ef65f610bb
commit 1be3afd82c
2 changed files with 8 additions and 12 deletions

View File

@ -15,4 +15,3 @@ export RUN_UNIT_TESTS_SEQUENTIAL="true"
export RUN_UNIT_TESTS="false" export RUN_UNIT_TESTS="false"
export GOAL="install" export GOAL="install"
export BITCOIN_CONFIG="--enable-glibc-back-compat --enable-zmq --enable-reduce-exports LDFLAGS=-static-libstdc++" export BITCOIN_CONFIG="--enable-glibc-back-compat --enable-zmq --enable-reduce-exports LDFLAGS=-static-libstdc++"
export RUN_SECURITY_TESTS="true"

View File

@ -23,29 +23,26 @@ class TestSymbolChecks(unittest.TestCase):
executable = 'test1' executable = 'test1'
cc = 'gcc' cc = 'gcc'
# there's no way to do this test for RISC-V at the moment; bionic's libc is 2.27 # renameat2 was introduced in GLIBC 2.28, so is newer than the upper limit
# and we allow all symbols from 2.27. # of glibc for all platforms
if 'riscv' in get_machine(cc):
self.skipTest("test not available for RISC-V")
# memfd_create was introduced in GLIBC 2.27, so is newer than the upper limit of
# all but RISC-V but still available on bionic
with open(source, 'w', encoding="utf8") as f: with open(source, 'w', encoding="utf8") as f:
f.write(''' f.write('''
#define _GNU_SOURCE #define _GNU_SOURCE
#include <sys/mman.h> #include <stdio.h>
#include <linux/fs.h>
int memfd_create(const char *name, unsigned int flags); int renameat2(int olddirfd, const char *oldpath,
int newdirfd, const char *newpath, unsigned int flags);
int main() int main()
{ {
memfd_create("test", 0); renameat2(0, "test", 0, "test_", RENAME_EXCHANGE);
return 0; return 0;
} }
''') ''')
self.assertEqual(call_symbol_check(cc, source, executable, []), self.assertEqual(call_symbol_check(cc, source, executable, []),
(1, executable + ': symbol memfd_create from unsupported version GLIBC_2.27\n' + (1, executable + ': symbol renameat2 from unsupported version GLIBC_2.28\n' +
executable + ': failed IMPORTED_SYMBOLS')) executable + ': failed IMPORTED_SYMBOLS'))
# -lutil is part of the libc6 package so a safe bet that it's installed # -lutil is part of the libc6 package so a safe bet that it's installed