Merge lp:~robru/autopilot-qt/packaging into lp:autopilot-qt

Proposed by Robert Bruce Park on 2013-08-13
Status: Merged
Approved by: Robert Bruce Park on 2013-08-13
Approved revision: 70
Merged at revision: 69
Proposed branch: lp:~robru/autopilot-qt/packaging
Merge into: lp:autopilot-qt
Diff against target: 78 lines (+23/-19)
2 files modified
debian/control (+22/-18)
debian/libautopilot-qt-autopilot.install (+1/-1)
To merge this branch: bzr merge lp:~robru/autopilot-qt/packaging
Reviewer Review Type Date Requested Status
Thomi Richards (community) Needs Information on 2013-08-14
PS Jenkins bot continuous-integration Approve on 2013-08-13
Robert Bruce Park (community) Approve on 2013-08-13
Review via email: mp+180015@code.launchpad.net

Commit message

Minor packaging fixes for daily release.

To post a comment you must log in.
Robert Bruce Park (robru) :
review: Approve
review: Approve (continuous-integration)
Thomi Richards (thomir) wrote :

Hi,

What was the actual change here? The only thing I can see is the change of Architecture to be 'any'. Can you please explain why this was needed? I'd like to know what's happening in the package I'm currently working in, so I don't accidentally break it again in the future.

Cheers,

review: Needs Information
Robert Bruce Park (robru) wrote :

Sorry, this change was really minor so I didn't think anybody would care. The only notable change here is that I added '${shlibs:Depends}' to the Depends: line for the binary package. This change was requested by mterry as part of a MIR[0].

After that, I ran `wrap-and-sort -a -t` which just sorted all the various lists of deps for easier readability in the future, but that's essentially just a whitespace fixup... lots of noise in the diff, no technical change happening.

Also as you noticed, I changed the arch to 'any', which is something we've come to prefer since the last time the packaging for this project was looked at.

[0] https://bugs.launchpad.net/ubuntu/+source/unity-webapps-qml/+bug/1206268

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/control'
--- debian/control 2013-05-03 14:17:19 +0000
+++ debian/control 2013-08-13 22:40:48 +0000
@@ -3,17 +3,17 @@
3Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>3Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
4XSBC-Original-Maintainer: Michael Zanetti <michael.zanetti@canonical.com>4XSBC-Original-Maintainer: Michael Zanetti <michael.zanetti@canonical.com>
5Build-Depends: debhelper (>= 9),5Build-Depends: debhelper (>= 9),
6 libgl1-mesa-dev,
7 libgles2-mesa-dev,
8 libqt4-dev,
9 libxpathselect-dev (>= 1.3),
10 mesa-common-dev,
6 pkg-config,11 pkg-config,
7 qtchooser,
8 qt4-qmake,12 qt4-qmake,
9 qt5-qmake,13 qt5-qmake,
10 libqt4-dev,
11 qtbase5-dev,14 qtbase5-dev,
15 qtchooser,
12 qtdeclarative5-dev,16 qtdeclarative5-dev,
13 libxpathselect-dev (>= 1.3),
14 mesa-common-dev,
15 libgl1-mesa-dev,
16 libgles2-mesa-dev,
17 xvfb,17 xvfb,
18Standards-Version: 3.9.418Standards-Version: 3.9.4
19Section: libs19Section: libs
@@ -26,25 +26,29 @@
2626
27Package: libautopilot-qt27Package: libautopilot-qt
28Section: libs28Section: libs
29Architecture: amd64 armhf i38629Architecture: any
30Depends: ${misc:Depends},30Depends: libxpathselect1.3,
31 libxpathselect1.3,31 ${misc:Depends},
32Replaces: autopilot-qt, autopilot-qt532 ${shlibs:Depends},
33Conflicts: autopilot-qt, autopilot-qt533Replaces: autopilot-qt,
34Provides: autopilot-qt, autopilot-qt534 autopilot-qt5,
35Recommends: python-autopilot35Conflicts: autopilot-qt,
36 autopilot-qt5,
37Provides: autopilot-qt,
38 autopilot-qt5,
39Recommends: python-autopilot,
36Description: This project makes Qt applications introspectable by autopilot.40Description: This project makes Qt applications introspectable by autopilot.
37 This allows autopilot to test any existing Qt4 or Qt5 application,41 This allows autopilot to test any existing Qt4 or Qt5 application,
38 without having to rebuild the application under test.42 without having to rebuild the application under test.
3943
40Package: libautopilot-qt-autopilot44Package: libautopilot-qt-autopilot
41Section: libs45Section: libs
42Architecture: amd64 armhf i38646Architecture: any
43Depends: ${shlibs:Depends},47Depends: libautopilot-qt (>= ${source:Version}),
44 ${misc:Depends},48 python-autopilot,
45 python-support (>= 0.90),49 python-support (>= 0.90),
46 libautopilot-qt (= ${source:Version}),50 ${misc:Depends},
47 python-autopilot,51 ${shlibs:Depends},
48Description: Autopilot tests for libautopilot-qt52Description: Autopilot tests for libautopilot-qt
49 This contains sample Qt4 and Qt5 apps and autopilot tests53 This contains sample Qt4 and Qt5 apps and autopilot tests
50 to test libautopilot-qt.54 to test libautopilot-qt.
5155
=== modified file 'debian/libautopilot-qt-autopilot.install'
--- debian/libautopilot-qt-autopilot.install 2013-03-17 21:24:10 +0000
+++ debian/libautopilot-qt-autopilot.install 2013-08-13 22:40:48 +0000
@@ -1,3 +1,3 @@
1qt*/tests/autopilot/testapp/qt*testapp usr/share/libautopilot-qt/
1tests/autopilot/libautopilot_qt/* usr/lib/python2.7/dist-packages/libautopilot_qt/2tests/autopilot/libautopilot_qt/* usr/lib/python2.7/dist-packages/libautopilot_qt/
2tests/autopilot/testapp/qt*.qml usr/share/libautopilot-qt/3tests/autopilot/testapp/qt*.qml usr/share/libautopilot-qt/
3qt*/tests/autopilot/testapp/qt*testapp usr/share/libautopilot-qt/

Subscribers

People subscribed via source and target branches