Skip to content

Commit 47d9c22

Browse files
committed
Only run external symbols test on Linux. Closes #304.
1 parent 2fac991 commit 47d9c22

File tree

2 files changed

+13
-0
lines changed

2 files changed

+13
-0
lines changed

Changes.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,8 @@
1+
## 1.7.1
2+
3+
* The external symbols test now only runs on Linux. It assumes a Linux
4+
environment. Reported by Carlo Cabrera. GitHub #304.
5+
16
## 1.7.0 - 2022-09-28
27

38
* `FD_CLOEXEC` is now set on platforms that do not support `O_CLOEXEC`.

t/external_symbols_t.pl

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,12 +5,20 @@
55

66
use FindBin qw( $Bin );
77

8+
_skip_tests_if_not_linux();
89
_skip_tests_if_required_modules_are_not_present();
910
_skip_tests_if_nm_is_not_present();
1011
_test_libs_external_symbols();
1112

1213
done_testing();
1314

15+
sub _skip_tests_if_not_linux {
16+
return if $^O eq 'linux';
17+
18+
print "1..0 # skip all tests skipped - this test requires Linux.\n";
19+
exit 0;
20+
}
21+
1422
sub _skip_tests_if_required_modules_are_not_present {
1523
eval <<'EOF';
1624
use Test::More 0.88;

0 commit comments

Comments
 (0)