Merge ~tsimonq2/kubuntu-packaging/+git/kubuntu-ark:kubuntu_zesty_archive into ~kubuntu-packagers/kubuntu-packaging/+git/ark:kubuntu_zesty_archive

Proposed by Simon Quigley on 2017-02-06
Status: Merged
Merged at revision: 3cd72e528ec185435041e745a896a641ca45b87e
Proposed branch: ~tsimonq2/kubuntu-packaging/+git/kubuntu-ark:kubuntu_zesty_archive
Merge into: ~kubuntu-packagers/kubuntu-packaging/+git/ark:kubuntu_zesty_archive
Diff against target: 48 lines (+18/-2)
4 files modified
debian/changelog (+6/-0)
debian/tests/control (+3/-1)
debian/tests/testsuite (+2/-1)
debian/tests/testsuite.xsession (+7/-0)
Reviewer Review Type Date Requested Status
Jose Manuel Santamaria Lema 2017-02-06 Needs Fixing on 2017-02-06
Review via email: mp+316413@code.launchpad.net

Description of the Change

Fix the autopkgtests by using okular's packaging as a model.

To post a comment you must log in.
review: Needs Fixing

These changes aren't correct at all:
1. Inside an evironment without $HOME, one of the tests would hang; this problem wasn't addressed at all while at the same time they are other unneeded, bad changes, see below.
2. xauth, dbus-x11, texlive-latex-base, xscreensaver were added to the test depends. They are not needed.
3. No need to call dbus-launch from the scripts.
4. No need to call kdeinit5 from the scripts

> Fix the autopkgtests by using okular's packaging as a model.

It seems to me rather than using the okular as a source of inspiration you just copied some files without actually checking very well what you did.

I have pushed the proper fix to kubuntu_zesty_archive.

Rik Mills (rikmills) wrote :

1. The tests all passed and the package migrated.

2. quite likely

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/debian/changelog b/debian/changelog
2index 7cae88e..90666aa 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,9 @@
6+ark (4:16.12.1-0ubuntu2) UNRELEASED; urgency=medium
7+
8+ * Fix the autopkgtests by using okular as a model (LP: #1662060).
9+
10+ -- Simon Quigley <tsimonq2@ubuntu.com> Sun, 05 Feb 2017 22:17:10 -0600
11+
12 ark (4:16.12.1-0ubuntu1) zesty; urgency=medium
13
14 [ José Manuel Santamaría Lema ]
15diff --git a/debian/tests/control b/debian/tests/control
16index 2b720f3..1f6913d 100644
17--- a/debian/tests/control
18+++ b/debian/tests/control
19@@ -1,3 +1,5 @@
20 Tests: testsuite
21-Depends: @, @builddeps@, build-essential, xvfb
22+Depends: @, @builddeps@, build-essential,
23+ xvfb, xauth, dbus-x11,
24+ texlive-latex-base, xscreensaver
25 Restrictions: build-needed
26diff --git a/debian/tests/testsuite b/debian/tests/testsuite
27index f6083b5..fa0feb1 100755
28--- a/debian/tests/testsuite
29+++ b/debian/tests/testsuite
30@@ -1,3 +1,4 @@
31 #!/bin/sh
32
33-xvfb-run -a dh_auto_test
34+xvfb-run -a --server-args="-screen 0 1024x768x24" \
35+ dbus-launch --exit-with-session debian/tests/testsuite.xsession
36diff --git a/debian/tests/testsuite.xsession b/debian/tests/testsuite.xsession
37new file mode 100755
38index 0000000..b1c5c10
39--- /dev/null
40+++ b/debian/tests/testsuite.xsession
41@@ -0,0 +1,7 @@
42+#!/bin/sh
43+export LANG=C.UTF-8
44+export LC_ALL=C.UTF-8
45+kdeinit5
46+trap "kdeinit5_shutdown" EXIT
47+
48+dh_auto_test

Subscribers

People subscribed via source and target branches