Page MenuHome GnuPG

Remove non-symbol npth_sigmask/npth_sigwait

Authored by MaskRay on Jan 16 2023, 7:35 PM.



npth_sigmask and npth_sigwait are not symbols, so
-Wl,--no-undefined-version will report errors. Actually npth_sigwait is
not defined, so just remove it. The npth_sigmask macro is used by
gnupg/g13/g13.c, so keep it.

  • src/ Remove npth_sigmask and npth_sigwait.
  • w32/npth.h: Likewise.
  • src/libnpth.vers: Likewise.

symbols to fix -Wl,--no-undefined-version

Test Plan

make all

Diff Detail

rPTH nPth
Lint Skipped
Unit Tests Skipped

Event Timeline

MaskRay created this revision.

I don't have write permission to the repository.

This patch fixes linker errors with GNU ld/gold/ld.lld --no-undefined-version. See and

MaskRay edited the summary of this revision. (Show Details)
MaskRay removed a reviewer: gniibe.

keep the macro as it is used

Thank you for the patch.

I'm going to apply the first one, as it is needed in the case, and it makes sense.

On the other hand, API-wise, for POSIX, having npth_sigwait makes sense.
So, I keep the declarations, at least for now.

This revision is now accepted and ready to land.Jan 17 2023, 5:59 AM