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.

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

fix ubuntu installation procedure #1084

Merged
merged 1 commit into from
Jun 2, 2022

Conversation

hannaeko
Copy link
Member

@hannaeko hannaeko commented Jun 1, 2022

Purpose

Fix LDNS installation using CPAN on Ubuntu 22.04.

Context

Ubuntu 22.04 comes with OpenSSL 3, LDNS uses OpenSSL 1.1.1, the changes in the API break the build step of Zonemaster LDNS.

The LDNS package in Ubuntu 22.04 and Debian 11 is the right version to use algorithm 15 and 16. This PR change the installation instruction to use the system LDNS and not the internal one.

Changes

  • Use system LDNS.
  • Add libldns3 (1.7.1) and libldns-dev to the dependency list.

How to test this PR

  • Install Zonemaster Engine / LDNS on Ubuntu 22.04 and Debian 11. The installation should work.

@hannaeko hannaeko added the T-Bug Type: Bug in software or error in test case description label Jun 1, 2022
@hannaeko hannaeko added this to the v2022.1 milestone Jun 1, 2022
@hannaeko hannaeko requested review from mattias-p, matsduf, a user and tgreenx June 1, 2022 12:48
Copy link
Member

@mattias-p mattias-p left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems reasonable to me.

Copy link
Contributor

@tgreenx tgreenx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested successfully on Debian 11.3 (see below). Post-sanity check is also fine.

sudo cpanm --configure-args="--no-internal-ldns" Zonemaster-LDNS-2.2.1.tar.gz

--> Working on Zonemaster-LDNS-2.2.1.tar.gz
Fetching file:///root/tmp/zonemaster/zonemaster-ldns/Zonemaster-LDNS-2.2.1.tar.gz ... OK
Configuring Zonemaster-LDNS-2.2.1 ... OK
Building and testing Zonemaster-LDNS-2.2.1 ... OK
Successfully installed Zonemaster-LDNS-2.2.1

@hannaeko hannaeko merged commit 36cf854 into zonemaster:develop Jun 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-Bug Type: Bug in software or error in test case description
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants