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

use pre-built packages for ubuntu #268

Merged
merged 1 commit into from
May 24, 2022

Conversation

hannaeko
Copy link
Member

Purpose

With the last release of Ubuntu, .deb packages can be used to install Zonemaster CLI.

Context

-

Changes

  • Prefer pre built packages for ubuntu
  • Keep the CPAN installation in a separate subsection

How to test this PR

Installation on Ubuntu should work. Use nightly packages to test.

host $ docker run --rm -it ubuntu:22.04 
container # apt update && apt install -y curl
container # curl -LOs https://package.zonemaster.net/setup.sh
container # nightly=yes sh setup.sh
container # zonemaster-cli --show-testcase --show-module --level info --no-ipv6 zonemaster.net
...
container # apt info zonemaster-cli
Package: zonemaster-cli
Version: 20220523200630-1+deb11ubuntu1
Priority: optional
Section: perl
...

@hannaeko hannaeko added this to the v2022.1 milestone May 24, 2022
@hannaeko hannaeko requested review from mattias-p, matsduf, a user, tgreenx and marc-vanderwal May 24, 2022 07:56
@matsduf
Copy link
Contributor

matsduf commented May 24, 2022

Preferably testing ("how to test this PR") should be done on the target system, not on Docker.

@matsduf matsduf added P-High Priority: Issue to be solved before other A-Documentation Area: Documentation only. labels May 24, 2022
@hannaeko
Copy link
Member Author

Preferably testing ("how to test this PR") should be done on the target system, not on Docker.

I am doing my tests on a VM, I provided instructions to test quickly but the instructions inside the container can be used inside a VM as well.

@hannaeko hannaeko merged commit e3d0ff9 into zonemaster:develop May 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Documentation Area: Documentation only. P-High Priority: Issue to be solved before other
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants