Fix test/recipes/01-test_symbol_presence.t to disregard version info#16840
Closed
levitte wants to merge 1 commit intoopenssl:masterfrom
Closed
Fix test/recipes/01-test_symbol_presence.t to disregard version info#16840levitte wants to merge 1 commit intoopenssl:masterfrom
levitte wants to merge 1 commit intoopenssl:masterfrom
Conversation
The output of 'nm -DPg' contains version info attached to the symbols, which makes the test fail. Simply dropping the version info makes the test work again. Fixes openssl#16810 (followup)
levitte
added
branch: master
mattcaswell
approved these changes
Oct 14, 2021
mattcaswell
added
the
severity: urgent
Member
|
Marked as urgent since this is breaking builds. |
Member
Author
|
It's a bit interesting that the Github Actions builds didn't break... |
levitte
added
the
approval: done
openssl-machine
pushed a commit
that referenced
this pull request
Oct 14, 2021
The output of 'nm -DPg' contains version info attached to the symbols, which makes the test fail. Simply dropping the version info makes the test work again. Fixes #16810 (followup) Reviewed-by: Matt Caswell <[email protected]> (Merged from #16840)
openssl-machine
pushed a commit
that referenced
this pull request
Oct 14, 2021
The output of 'nm -DPg' contains version info attached to the symbols, which makes the test fail. Simply dropping the version info makes the test work again. Fixes #16810 (followup) Reviewed-by: Matt Caswell <[email protected]> (Merged from #16840) (cherry picked from commit 73970cb)
openssl-machine
pushed a commit
that referenced
this pull request
Oct 14, 2021
The output of 'nm -DPg' contains version info attached to the symbols, which makes the test fail. Simply dropping the version info makes the test work again. Fixes #16810 (followup) Reviewed-by: Matt Caswell <[email protected]> (Merged from #16840) (cherry picked from commit 73970cb)
Member
Author
Member
Author
|
From observation, it seems that this fixes #16837, which I only noticed after I merged this PR. |
dstebila
pushed a commit
to open-quantum-safe/openssl
that referenced
this pull request
Mar 15, 2022
The output of 'nm -DPg' contains version info attached to the symbols,
which makes the test fail. Simply dropping the version info makes the
test work again.
Fixes #16810 (followup)