[Debian-ha-maintainers] Bug#883524: crmsh: autopkgtest portability fixes
Steve Langasek
steve.langasek at canonical.com
Mon Dec 4 18:25:58 UTC 2017
Package: crmsh
Version: 3.0.1-2
Severity: normal
Tags: patch
User: ubuntu-devel at lists.ubuntu.com
Usertags: origin-ubuntu bionic ubuntu-patch autopkgtest
Dear maintainers,
The newest version of the crmsh introduces several new autopkgtests, which
is great; but these tests were failing in Ubuntu for a couple of reasons.
- One of the new tests tries to purge the less package with dpkg --purge.
This works fine in a default Debian CI environment, but in Ubuntu, less
is a dependency of the ubuntu-minimal metapackage which is installed by
default - so dpkg --purge fails.
- Another test adds a test dependency on dmidecode. This works fine on
architectures where dmidecode exists; but Ubuntu runs autopkgtests on
all its architectures, and dmidecode does not exist on ppc64el or s390x.
The attached patch addresses both of these issues, and lets the crmsh
autopkgtests pass on all Ubuntu architectures. I believe the patch is also
suitable for inclusion in Debian.
Thanks for considering,
--
Steve Langasek Give me a lever long enough and a Free OS
Debian Developer to set it on, and I can move the world.
Ubuntu Developer http://www.debian.org/
slangasek at ubuntu.com vorlon at debian.org
-------------- next part --------------
diff -Nru crmsh-3.0.1/debian/tests/control crmsh-3.0.1/debian/tests/control
--- crmsh-3.0.1/debian/tests/control 2017-09-01 06:10:52.000000000 -0700
+++ crmsh-3.0.1/debian/tests/control 2017-12-03 22:44:50.000000000 -0800
@@ -23,5 +23,5 @@
Restrictions: needs-root, allow-stderr, isolation-container
Tests: pacemaker-cluster-init.sh
-Depends: @, pacemaker, corosync, csync2, ntp, ufw, openssh-server, dmidecode, booth, booth-pacemaker
+Depends: @, pacemaker, corosync, csync2, ntp, ufw, openssh-server, dmidecode [any-amd64 arm64 armhf any-i386], booth, booth-pacemaker
Restrictions: needs-root, allow-stderr, isolation-container
diff -Nru crmsh-3.0.1/debian/tests/utils.sh crmsh-3.0.1/debian/tests/utils.sh
--- crmsh-3.0.1/debian/tests/utils.sh 2017-06-08 02:32:07.000000000 -0700
+++ crmsh-3.0.1/debian/tests/utils.sh 2017-12-03 22:42:26.000000000 -0800
@@ -3,7 +3,7 @@
set -ex
export LC_ALL=C
-PKG=less
+PKG=vim
dpkg --purge $PKG
More information about the Debian-ha-maintainers
mailing list