Skip to content

Commit

Permalink
Fix test/recipes/01-test_symbol_presence.t to allow for stripped libr…
Browse files Browse the repository at this point in the history
…aries

It's a small change to the 'nm' call, to have it look at dynamic symbols
rather than the normal ones.

Fixes openssl#16810

Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from openssl#16822)
  • Loading branch information
levitte committed Oct 13, 2021
1 parent f087ebc commit a85b4de
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/recipes/01-test_symbol_presence.t
Original file line number Diff line number Diff line change
Expand Up @@ -48,12 +48,12 @@ foreach my $libname (@libnames) {
*OSTDOUT = *STDOUT;
open STDERR, ">", devnull();
open STDOUT, ">", devnull();
my @nm_lines = map { s|\R$||; $_ } `nm -Pg $shlibpath 2> /dev/null`;
my @nm_lines = map { s|\R$||; $_ } `nm -DPg $shlibpath 2> /dev/null`;
close STDERR;
close STDOUT;
*STDERR = *OSTDERR;
*STDOUT = *OSTDOUT;
skip "Can't run 'nm -Pg $shlibpath' => $?... ignoring", 2
skip "Can't run 'nm -DPg $shlibpath' => $?... ignoring", 2
unless $? == 0;

my $bldtop = bldtop_dir();
Expand Down

0 comments on commit a85b4de

Please sign in to comment.