Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Make test suite link to static LDNS rather than dynamic. #200

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Make test suite link to static LDNS rather than dynamic. #200

wants to merge 1 commit into from

Commits on Dec 6, 2022

  1. Make test suite link to static LDNS rather than dynamic. 

    Issue NLnetLabs#197: Cygwin fails these tests because it’s dicey to link
dynamically to a library version that isn’t installed in the $PATH.
Rather than adding platform-specific logic, this changeset alters the
test suite to link statically rather than dynamically, which alleviates
the problem.

Notably, this doesn’t resolve all problems with Cygwin in the test suite;
other tests fail for other reasons. (See the GitHub issue for details.)
    @FGasper
    FGasper committed Dec 6, 2022
    Configuration menu
    Copy the full SHA
    1f27666 View commit details
    Browse the repository at this point in the history