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) |
||||
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Jose Manuel Santamaria Lema | 2017-02-06 | Needs Fixing on 2017-02-06 | |
|
Review via email:
|
|||
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
| Rik Mills (rikmills) wrote : | # |
1. The tests all passed and the package migrated.
2. quite likely

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.