Merge lp:~cyphermox/autopilot-qt/inline into lp:autopilot-qt

Proposed by Mathieu Trudel-Lapierre
Status: Merged
Approved by: Michael Zanetti
Approved revision: 41
Merged at revision: 40
Proposed branch: lp:~cyphermox/autopilot-qt/inline
Merge into: lp:autopilot-qt
Diff against target: 146 lines (+35/-39)
7 files modified
.bzr-builddeb/default.conf (+2/-0)
debian/changelog (+18/-0)
debian/compat (+1/-1)
debian/control (+9/-4)
debian/copyright (+1/-21)
debian/libautopilot-qt.install (+2/-1)
debian/rules (+2/-12)
To merge this branch: bzr merge lp:~cyphermox/autopilot-qt/inline
Reviewer Review Type Date Requested Status
Michael Zanetti (community) Approve
PS Jenkins bot continuous-integration Approve
Didier Roche-Tolomelli Approve
Review via email: mp+136549@code.launchpad.net

Commit message

Update packaging for autolanding.

Description of the change

Update packaging for autolanding

To post a comment you must log in.
Revision history for this message
Mathieu Trudel-Lapierre (cyphermox) wrote :

It seems liek this is likely to explode; qt5 doesn't seem packaged yet?

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
lp:~cyphermox/autopilot-qt/inline updated
32. By Michael Zanetti

remove some debug prints.

Approved by Juhapekka Piiroinen, PS Jenkins bot.

33. By Michael Zanetti

improve debug output a little.

Approved by PS Jenkins bot.

34. By Michael Zanetti

autopilot-qt is now a single package with drivers that load correctly for each supported Qt version.

Approved by Juhapekka Piiroinen, Omer Akram.

35. By Michael Zanetti

update for Qt5 final.

Approved by Olivier Tilloy, PS Jenkins bot.

36. By Michael Zanetti

fix dependency to libxpathselect.

Approved by Chris Gagnon, PS Jenkins bot.

37. By Michael Zanetti

revert precise dependency fix. Use autopilot-qt or autopilot-qt5 for precise.

Approved by PS Jenkins bot.

38. By Thomi Richards

remove white spaces in root node name. xpathselect does not support that any more. Fixes: https://bugs.launchpad.net/bugs/1144448.

Approved by PS Jenkins bot, Michael Zanetti.

39. By Michael Zanetti

update runtime dep to xpathselect to 1.2 .

Approved by PS Jenkins bot, Olivier Tilloy.

40. By Mathieu Trudel-Lapierre

* debian/control:
  - Wrap, sort and add trailing commas to (Build-)Depends lists.
  - Bump Build-Depends on debhelper to 9.
  - Add Vcs-Bzr, Vcs-Browser and a notice to uploaders.
  - Drop Build-Depends on libindicate-qt.
* debian/copyright: license everything under GPL3.
* debian/compat: bump to compat level 9.
* debian/rules:
  - Don't make DH be verbose.
  - Clean up extra comments.
  - Simplify the dh_install process.
* debian/libautopilot-qt.install: move file copies into the install file
  where they should be (from debian/rules).

Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

83 +License: GPL-3
84 +
85 +Files: debian/*
86 +Copyright: 2012 Canonical Ltd.
87 +License: GPL-3

-> do you mind just making that one stenza please? No need to separate the debian/ folder

The other changes looks good to me. Feel free to approve the global change once your sbuild is happy ;)

review: Approve
lp:~cyphermox/autopilot-qt/inline updated
41. By Mathieu Trudel-Lapierre

Unsplit the copyright stanzas. No reason to have debian/ separate if it's covered by the same license.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Michael Zanetti (mzanetti) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== added directory '.bzr-builddeb'
2=== added file '.bzr-builddeb/default.conf'
3--- .bzr-builddeb/default.conf 1970-01-01 00:00:00 +0000
4+++ .bzr-builddeb/default.conf 2013-03-14 17:09:22 +0000
5@@ -0,0 +1,2 @@
6+[BUILDDEB]
7+split = True
8
9=== modified file 'debian/changelog'
10--- debian/changelog 2013-03-01 09:07:48 +0000
11+++ debian/changelog 2013-03-14 17:09:22 +0000
12@@ -1,3 +1,21 @@
13+libautopilot-qt (0.6-0ubuntu1) UNRELEASED; urgency=low
14+
15+ * debian/control:
16+ - Wrap, sort and add trailing commas to (Build-)Depends lists.
17+ - Bump Build-Depends on debhelper to 9.
18+ - Add Vcs-Bzr, Vcs-Browser and a notice to uploaders.
19+ - Drop Build-Depends on libindicate-qt.
20+ * debian/copyright: license everything under GPL3.
21+ * debian/compat: bump to compat level 9.
22+ * debian/rules:
23+ - Don't make DH be verbose.
24+ - Clean up extra comments.
25+ - Simplify the dh_install process.
26+ * debian/libautopilot-qt.install: move file copies into the install file
27+ where they should be (from debian/rules).
28+
29+ -- Mathieu Trudel-Lapierre <mathieu-tl@ubuntu.com> Tue, 27 Nov 2012 18:04:58 -0500
30+
31 libautopilot-qt (0.6) quantal; urgency=low
32
33 * revert to previous version. For Precise, use autopilot-qt4 or autopilot-qt5
34
35=== modified file 'debian/compat'
36--- debian/compat 2012-10-09 16:21:50 +0000
37+++ debian/compat 2013-03-14 17:09:22 +0000
38@@ -1,1 +1,1 @@
39-8
40+9
41
42=== modified file 'debian/control'
43--- debian/control 2013-03-06 12:56:19 +0000
44+++ debian/control 2013-03-14 17:09:22 +0000
45@@ -1,8 +1,8 @@
46 Source: libautopilot-qt
47 Priority: extra
48-Maintainer: Michael Zanetti <michael.zanetti@canonical.com>
49-Build-Depends: debhelper (>= 8.0.0),
50- libindicate-qt-dev,
51+Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
52+XSBC-Original-Maintainer: Michael Zanetti <michael.zanetti@canonical.com>
53+Build-Depends: debhelper (>= 9),
54 pkg-config,
55 qtchooser,
56 qt4-qmake,
57@@ -13,10 +13,15 @@
58 libxpathselect-dev,
59 mesa-common-dev,
60 libgl1-mesa-dev,
61- libgles2-mesa-dev
62+ libgles2-mesa-dev,
63 Standards-Version: 3.9.3
64 Section: libs
65 Homepage: https://launchpad.net/autopilot-qt
66+# If you aren't a member of ~autopilot but need to upload
67+# packaging changes, just go ahead. ~autopilot will notice
68+# and sync up the code again.
69+Vcs-Bzr: https://code.launchpad.net/~autopilot/autopilot-qt/trunk
70+Vcs-Browser: https://bazaar.launchpad.net/~autopilot/autopilot-qt/trunk/files
71
72 Package: libautopilot-qt
73 Section: libs
74
75=== modified file 'debian/copyright'
76--- debian/copyright 2013-01-14 10:52:56 +0000
77+++ debian/copyright 2013-03-14 17:09:22 +0000
78@@ -4,7 +4,7 @@
79
80 Files: *
81 Copyright: 2012 Canonical Ltd.
82-License:
83+License: GPL-3
84 This program is free software: you can redistribute it and/or modify it
85 under the terms of the the GNU General Public License version 3, as
86 published by the Free Software Foundation.
87@@ -20,23 +20,3 @@
88 .
89 On Debian systems, the complete text of the GNU General
90 Public License v3 can be found in `/usr/share/common-licenses/GPL-3'.
91-
92-Files: debian/*
93-Copyright: 2012 Canonical Ltd.
94-License: GPL-2+
95- This package is free software; you can redistribute it and/or modify
96- it under the terms of the GNU General Public License as published by
97- the Free Software Foundation; either version 2 of the License, or
98- (at your option) any later version.
99- .
100- This package is distributed in the hope that it will be useful,
101- but WITHOUT ANY WARRANTY; without even the implied warranty of
102- MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
103- GNU General Public License for more details.
104- .
105- You should have received a copy of the GNU General Public License
106- along with this program. If not, see <http://www.gnu.org/licenses/>
107- .
108- On Debian systems, the complete text of the GNU General
109- Public License version 2 can be found in "/usr/share/common-licenses/GPL-2".
110-
111
112=== modified file 'debian/libautopilot-qt.install'
113--- debian/libautopilot-qt.install 2013-01-14 10:52:56 +0000
114+++ debian/libautopilot-qt.install 2013-03-14 17:09:22 +0000
115@@ -1,1 +1,2 @@
116-usr/lib/lib*.so.*
117+qt4/lib*.so* usr/lib
118+qt5/lib*.so* usr/lib
119
120=== modified file 'debian/rules'
121--- debian/rules 2013-01-17 20:20:44 +0000
122+++ debian/rules 2013-03-14 17:09:22 +0000
123@@ -1,13 +1,6 @@
124 #!/usr/bin/make -f
125-# -*- makefile -*-
126-# Sample debian/rules that uses debhelper.
127-# This file was originally written by Joey Hess and Craig Small.
128-# As a special exception, when this file is copied by dh-make into a
129-# dh-make output file, you may use that output file without restriction.
130-# This special exception was added by Craig Small in version 0.37 of dh-make.
131
132-# Uncomment this to turn on verbose mode.
133-export DH_VERBOSE=1
134+export DPKG_GENSYMBOLS_CHECK_LEVEL=4
135
136 %:
137 dh $@
138@@ -28,7 +21,4 @@
139 rm -rf qt5
140
141 override_dh_install:
142- mkdir -p $(CURDIR)/debian/libautopilot-qt/usr/lib/
143- mkdir -p $(CURDIR)/debian/libautopilot-qt5/opt/qt5/lib/
144- cp -pP qt4/*.so* $(CURDIR)/debian/libautopilot-qt/usr/lib/
145- cp -pP qt5/*.so* $(CURDIR)/debian/libautopilot-qt/usr/lib/
146+ dh_install --fail-missing

Subscribers

People subscribed via source and target branches