Merge ~mirespace/ubuntu/+source/spice:merge-spice-0.14.3-2.1-impish into ubuntu/+source/spice:debian/sid

Proposed by Miriam España Acebal
Status: Merged
Approved by: Christian Ehrhardt 
Approved revision: 1160eb2700acfe6cadb74a1b025f8fdb12411b59
Merge reported by: Bryce Harrington
Merged at revision: 1160eb2700acfe6cadb74a1b025f8fdb12411b59
Proposed branch: ~mirespace/ubuntu/+source/spice:merge-spice-0.14.3-2.1-impish
Merge into: ubuntu/+source/spice:debian/sid
Diff against target: 267 lines (+189/-3)
3 files modified
debian/changelog (+185/-0)
debian/control (+3/-2)
debian/tests/automated-tests (+1/-1)
Reviewer Review Type Date Requested Status
Christian Ehrhardt  (community) Approve
Canonical Server Pending
Miriam España Acebal Pending
Canonical Server packageset reviewers Pending
Review via email: mp+407075@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Miriam España Acebal (mirespace) wrote :

Hello team,

PPA for this merge: https://launchpad.net/~mirespace/+archive/ubuntu/merge-spice-0.14.3-2.1-impish.

The delta has 2 changes.

Usual tags are there: reconstruct/0.14.3-2ubuntu3 split/0.14.3-2ubuntu3 logical/0.14.3-2ubuntu3 new/debian old/debian old/ubuntu

Test passed:
autopkgtest [14:35:26]: test automated-tests: -----------------------]
autopkgtest [14:35:27]: test automated-tests: - - - - - - - - - - results - - - - - - - - - -
automated-tests PASS
autopkgtest [14:35:27]: @@@@@@@@@@@@@@@@@@@@ summary
automated-tests PASS

Thanks in advance!

Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

* Changelog:
  - [+] old content and logical tag match as expected
  - [+] changelog entry correct version and targeted codename
  - [-] changelog entries correct

Due to the new indent level the line referring to 973803 is too long now (81 chars)

  - [+] bug references correct (none)
  - [+] update-maintainer has been run

* Merge - Indirect Changes:
  - [+] no upstream changes to consider
  - [-] no further upstream version to consider

There is 0.14.91 but I agree that we will not merge this one now.
It is more a prep to 0.15 than something we'd want so late this cycle

  - [+] debian changes look safe

* Merge - Old Delta:
  - [n/a] dropped changes are ok to be dropped
  - [+] nothing else to drop
  - [n/a] changes forwarded upstream/debian (if appropriate)

* New Delta:
  - [+] no new patches added

* Git/Maintenance
  - [+] testcases added or not needed for this
  - [+] commits are properly split (more important on -dev than on SRUs)

* Build/Test:
  - [+] build is ok
  - [+] verified PPA package installs/uninstalls
  - [+] autopkgtest against the PPA package passes (by your tests)

So overall - yeah this LGTM except the little line length issue.
But that should be a quick change and then we can sponsor it.

review: Needs Fixing
Revision history for this message
Miriam España Acebal (mirespace) wrote :

Thanks, Christian!

I made that line smaller (and the above point also to make them look alike).

Here we go again...

Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

Length ok now, but in return we now got trailing whitespace in lin 4,5 and 7 :-/
Should I fix it on upload or do you want to modify it again?

Revision history for this message
Miriam España Acebal (mirespace) wrote :

Hoping this time is OK (installed an extension to VSCode to see the blanks)... Sorry!

Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

LGTM now, thanks
+1

review: Approve
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

To ssh://git.launchpad.net/~usd-import-team/ubuntu/+source/spice
 * [new tag] upload/0.14.3-2.1ubuntu1 -> upload/0.14.3-2.1ubuntu1

Uploading to ubuntu (via ftp to upload.ubuntu.com):
  Uploading spice_0.14.3-2.1ubuntu1.dsc: done.
  Uploading spice_0.14.3-2.1ubuntu1.debian.tar.xz: done.
  Uploading spice_0.14.3-2.1ubuntu1_source.buildinfo: done.
  Uploading spice_0.14.3-2.1ubuntu1_source.changes: done.
Successfully uploaded packages.

Sponsored, please track tests and migration.

Revision history for this message
Bryce Harrington (bryce) wrote :

This has successfully migrated.

  - Source Package: spice
  - Current Version: 0.14.3-2.1ubuntu1
  - Proposed Version: None
  - Old Version: tbd
  - New Version: 0.14.3-2.1ubuntu1
  - Migrated: True
  - Requester: Miriam España Acebal

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/debian/changelog b/debian/changelog
index d92d3c2..1fcd39a 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,14 @@
1spice (0.14.3-2.1ubuntu1) impish; urgency=medium
2
3 * Merge with Debian unstable. Remaining changes:
4 - d/control: Don't recommend -libav gstreamer plugins since it
5 is in universe. This now downgrades it to a suggest instead
6 of completely removing the dependency.
7 - d/t/automated-tests: avoid test fail due to build
8 errors (Closes: #973803).
9
10 -- Miriam España Acebal <miriam.espana@canonical.com> Fri, 13 Aug 2021 14:12:44 +0200
11
1spice (0.14.3-2.1) unstable; urgency=medium12spice (0.14.3-2.1) unstable; urgency=medium
213
3 * Non-maintainer upload.14 * Non-maintainer upload.
@@ -8,6 +19,21 @@ spice (0.14.3-2.1) unstable; urgency=medium
819
9 -- Salvatore Bonaccorso <carnil@debian.org> Sun, 28 Feb 2021 16:29:54 +010020 -- Salvatore Bonaccorso <carnil@debian.org> Sun, 28 Feb 2021 16:29:54 +0100
1021
22spice (0.14.3-2ubuntu3) hirsute; urgency=medium
23
24 * d/t/automated-tests: avoid test fail due to build errors (Closes: #973803)
25
26 -- Christian Ehrhardt <christian.ehrhardt@canonical.com> Thu, 05 Nov 2020 11:23:42 +0100
27
28spice (0.14.3-2ubuntu2) hirsute; urgency=medium
29
30 * Merge with Debian unstable. Remaining changes:
31 - d/control: Don't recommend -libav gstreamer plugins since it is in
32 universe. This now downgrades it to a suggest instead of completely
33 removing the dependency.
34
35 -- Christian Ehrhardt <christian.ehrhardt@canonical.com> Tue, 03 Nov 2020 13:07:45 +0100
36
11spice (0.14.3-2) unstable; urgency=medium37spice (0.14.3-2) unstable; urgency=medium
1238
13 [ Christian Ehrhardt ]39 [ Christian Ehrhardt ]
@@ -32,6 +58,40 @@ spice (0.14.3-2) unstable; urgency=medium
3258
33 -- Michael Tokarev <mjt@tls.msk.ru> Thu, 29 Oct 2020 10:57:02 +030059 -- Michael Tokarev <mjt@tls.msk.ru> Thu, 29 Oct 2020 10:57:02 +0300
3460
61spice (0.14.3-1ubuntu2) groovy; urgency=medium
62
63 * SECURITY UPDATE: multiple buffer overflows in QUIC image decoding
64 - debian/patches/CVE-2020-14355-1.patch: check we have some data to
65 start decoding quic image in subprojects/spice-common/common/quic.c.
66 - debian/patches/CVE-2020-14355-2.patch: check image size in
67 quic_decode_begin in subprojects/spice-common/common/quic.c.
68 - debian/patches/CVE-2020-14355-3.patch: check RLE lengths in
69 subprojects/spice-common/common/quic_tmpl.c.
70 - debian/patches/CVE-2020-14355-4.patch: avoid possible buffer overflow
71 in find_bucket in subprojects/spice-common/common/quic_family_tmpl.c.
72 - CVE-2020-14355
73
74 -- Marc Deslauriers <marc.deslauriers@ubuntu.com> Thu, 01 Oct 2020 07:00:18 -0400
75
76spice (0.14.3-1ubuntu1) groovy; urgency=medium
77
78 * Merge with Debian unstable (LP: #1881093). Remaining changes:
79 - d/control: Don't recommend -libav gstreamer plugins since it is in
80 universe
81 - make autopkgtests work again
82 - d/t/automated-tests: spice-common moved into dir subprojects
83 - d/t/automated-tests: option --enable-automated-tests now is always on
84 - d/t/control: make tests more debuggable by allowing stderr
85 - d/t/control: install new test dependency python-pil
86 - d/t/regression-test.py, d/t/base_test.ppm: add file dropped in release
87 tarball but needed for autopkgtests
88 - d/source/include-binaries: allow binary base_test.ppm in package
89 * Dropped changes
90 - d/p/lp-1874054-*: fix rescaling and some crashes (LP: 1874054)
91 [Upstream in 0.14.3]
92
93 -- Christian Ehrhardt <christian.ehrhardt@canonical.com> Thu, 28 May 2020 11:56:04 +0200
94
35spice (0.14.3-1) unstable; urgency=medium95spice (0.14.3-1) unstable; urgency=medium
3696
37 * new upstream version (Closes: #940057, #954629)97 * new upstream version (Closes: #940057, #954629)
@@ -43,6 +103,49 @@ spice (0.14.3-1) unstable; urgency=medium
43103
44 -- Michael Tokarev <mjt@tls.msk.ru> Tue, 14 Apr 2020 16:55:25 +0300104 -- Michael Tokarev <mjt@tls.msk.ru> Tue, 14 Apr 2020 16:55:25 +0300
45105
106spice (0.14.2-4ubuntu3) focal; urgency=medium
107
108 * d/p/lp-1874054-*: fix rescaling and some crashes (LP: #1874054)
109
110 -- Christian Ehrhardt <christian.ehrhardt@canonical.com> Tue, 21 Apr 2020 14:05:18 +0200
111
112spice (0.14.2-4ubuntu2) focal; urgency=medium
113
114 * No-change rebuild for libgcc-s1 package name change.
115
116 -- Matthias Klose <doko@ubuntu.com> Mon, 23 Mar 2020 07:26:08 +0100
117
118spice (0.14.2-4ubuntu1) focal; urgency=medium
119
120 * Merge with Debian unstable (LP: #1852439). Remaining changes:
121 - d/control: Don't recommend -libav gstreamer plugins since it is in
122 universe
123 - make autopkgtests work again
124 - d/t/automated-tests: spice-common moved into dir subprojects
125 - d/t/automated-tests: option --enable-automated-tests now is always on
126 - d/t/control: make tests more debuggable by allowing stderr
127 - d/t/control: install new test dependency python-pil
128 - d/t/regression-test.py, d/t/base_test.ppm: add file dropped in release
129 tarball but needed for autopkgtests
130 - d/source/include-binaries: allow binary base_test.ppm in package
131 * Added changes:
132 - d/t/automated-tests, d/t/control: make autopkgtests python3 compatible
133 * Dropped Changes (in Debian):
134 - d/control: Don't recommend -ugly gstreamer plugins since it is in universe
135 - d/patches: drop patches being upstream in 0.14.2
136 - new upstream 0.14.2
137 - disable failing test-listen
138 - d/libspice-server1.symbols: update for new symbols in 14.2
139 - d/p/fix-test-qxl-parsing-on-ppc64el-and-armhf.patch: avoid FTBFS due to
140 different handling of high words for constants
141 - d/control: bump build dependency to libspice-protocol-dev >=0.14.0
142 * Dropped Changes (Upstream)
143 - SECURITY UPDATE: Integer overflow and buffer overflow CVE-2017-12194
144 - SECURITY UPDATE: Denial of service CVE-2018-10873
145 - SECURITY UPDATE: off-by-one error in memslot_get_virt CVE-2019-3813
146
147 -- Christian Ehrhardt <christian.ehrhardt@canonical.com> Wed, 13 Nov 2019 15:54:00 +0100
148
46spice (0.14.2-4) unstable; urgency=medium149spice (0.14.2-4) unstable; urgency=medium
47150
48 * disable failing test-listen (Closes: #941006)151 * disable failing test-listen (Closes: #941006)
@@ -90,6 +193,42 @@ spice (0.14.2-1) unstable; urgency=medium
90193
91 -- Michael Tokarev <mjt@tls.msk.ru> Fri, 30 Aug 2019 13:54:00 +0300194 -- Michael Tokarev <mjt@tls.msk.ru> Fri, 30 Aug 2019 13:54:00 +0300
92195
196spice (0.14.2-0ubuntu2) eoan; urgency=medium
197
198 * Fixup autpkgtest (LP: #1834286)
199 These changes will make the test able to run again, but not output mismatch
200 errors (this matches the behavior before 0.14.2). Upstream discussion
201 started on how to resolve that as a next step, more details at the LP bug.
202 - d/t/automated-tests: spice-common moved into dir subprojects
203 - d/t/automated-tests: option --enable-automated-tests now is always on"
204 - d/t/automated-tests, d/t/control: make tests more debuggable by allowing
205 stderr
206 - d/t/control: install new test dependency python-pil
207 - d/t/base_test.ppm, d/t/regression-test.py: provide test resources from
208 upstream git not part of the released tarball anymore
209 - d/source/include-binaries: allow binary base_test.ppm in package
210
211 -- Christian Ehrhardt <christian.ehrhardt@canonical.com> Tue, 25 Jun 2019 12:59:01 +0200
212
213spice (0.14.2-0ubuntu1) eoan; urgency=medium
214
215 * New upstream release
216 Among many other fixes this will resolve (LP: #1814146)
217 - d/p/disable-failing-test-listen.patch: disable new test that is
218 unreliable in the build environment
219 - d/patches: drop patches being upstream in 0.14.2
220 + debian/patches/CVE-2017-12194-1.patch
221 + debian/patches/CVE-2017-12194-2.patch
222 + debian/patches/CVE-2017-12194-3.patch
223 + debian/patches/CVE-2018-10873.patch
224 + debian/patches/CVE-2019-3813.patch
225 - d/libspice-server1.symbols: update for new symbols in 14.2
226 - d/p/fix-test-qxl-parsing-on-ppc64el-and-armhf.patch: avoid FTBFS due
227 to different handling of high words for constants
228 - d/control: bump build dependency to libspice-protocol-dev >=0.14.0
229
230 -- Christian Ehrhardt <christian.ehrhardt@canonical.com> Fri, 24 May 2019 12:27:26 +0200
231
93spice (0.14.0-1.3) unstable; urgency=medium232spice (0.14.0-1.3) unstable; urgency=medium
94233
95 * Non-maintainer upload.234 * Non-maintainer upload.
@@ -112,6 +251,52 @@ spice (0.14.0-1.1) unstable; urgency=medium
112251
113 -- Salvatore Bonaccorso <carnil@debian.org> Sat, 15 Sep 2018 09:15:28 +0200252 -- Salvatore Bonaccorso <carnil@debian.org> Sat, 15 Sep 2018 09:15:28 +0200
114253
254spice (0.14.0-1ubuntu5) disco; urgency=medium
255
256 * SECURITY UPDATE: off-by-one error in memslot_get_virt
257 - debian/patches/CVE-2019-3813.patch: fix checks in server/memslot.c,
258 add tests to server/tests/test-qxl-parsing.c.
259 - CVE-2019-3813
260 * debian/tests/automated-tests: fix incorrect test name, don't fail on
261 build writing to stderr.
262
263 -- Marc Deslauriers <marc.deslauriers@ubuntu.com> Thu, 24 Jan 2019 08:58:10 -0500
264
265spice (0.14.0-1ubuntu4) cosmic; urgency=medium
266
267 * SECURITY UPDATE: Denial of service
268 - debian/patches/CVE-2018-10873.patch: fix in
269 spice-common/python_modules/demarshal.py,
270 - CVE-2018-10873
271
272 -- Leonidas S. Barbosa <leo.barbosa@canonical.com> Mon, 20 Aug 2018 13:26:02 -0300
273
274spice (0.14.0-1ubuntu3) cosmic; urgency=medium
275
276 * SECURITY UPDATE: Integer overflow and buffer overflow
277 - debian/patches/CVE-2017-12194-1.patch: fix a integer overflow
278 computing sizes in spice-common/python_modules/demarshal.py.
279 - debian/patches/CVE-2017-12194-2.patch: avoid integer overflow
280 in spice-common/python_modules/demarshal.py,
281 spice-common/python_modules/marshal.py.
282 - debian/patches/CVE-2017-12194-3.patch: add tests to verify fix.
283 - CVE-2017-12194
284
285 -- Leonidas S. Barbosa <leo.barbosa@canonical.com> Tue, 22 May 2018 14:53:01 -0300
286
287spice (0.14.0-1ubuntu2) bionic; urgency=high
288
289 * No change rebuild against openssl1.1.
290
291 -- Dimitri John Ledkov <xnox@ubuntu.com> Tue, 06 Feb 2018 17:55:31 +0000
292
293spice (0.14.0-1ubuntu1) bionic; urgency=medium
294
295 * Don't recommend -ugly or -libav gstreamer plugins since they
296 are in universe
297
298 -- Jeremy Bicha <jbicha@ubuntu.com> Wed, 01 Nov 2017 21:55:03 -0400
299
115spice (0.14.0-1) unstable; urgency=medium300spice (0.14.0-1) unstable; urgency=medium
116301
117 * New upstream release302 * New upstream release
diff --git a/debian/control b/debian/control
index d31bb2c..9813c7d 100644
--- a/debian/control
+++ b/debian/control
@@ -1,7 +1,8 @@
1Source: spice1Source: spice
2Section: misc2Section: misc
3Priority: optional3Priority: optional
4Maintainer: Debian QEMU Team <pkg-qemu-devel@lists.alioth.debian.org>4Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
5XSBC-Original-Maintainer: Debian QEMU Team <pkg-qemu-devel@lists.alioth.debian.org>
5Uploaders: Michael Tokarev <mjt@tls.msk.ru>6Uploaders: Michael Tokarev <mjt@tls.msk.ru>
6Build-Depends:7Build-Depends:
7 debhelper-compat (= 12),8 debhelper-compat (= 12),
@@ -39,10 +40,10 @@ Multi-Arch: same
39Pre-Depends: ${misc:Pre-Depends}40Pre-Depends: ${misc:Pre-Depends}
40Depends: ${misc:Depends}, ${shlibs:Depends}41Depends: ${misc:Depends}, ${shlibs:Depends}
41Recommends:42Recommends:
42 gstreamer1.0-libav,
43 gstreamer1.0-plugins-base,43 gstreamer1.0-plugins-base,
44 gstreamer1.0-plugins-good,44 gstreamer1.0-plugins-good,
45Suggests:45Suggests:
46 gstreamer1.0-libav,
46 gstreamer1.0-plugins-ugly,47 gstreamer1.0-plugins-ugly,
47Description: Implements the server side of the SPICE protocol48Description: Implements the server side of the SPICE protocol
48 The Simple Protocol for Independent Computing Environments (SPICE) is49 The Simple Protocol for Independent Computing Environments (SPICE) is
diff --git a/debian/tests/automated-tests b/debian/tests/automated-tests
index 8664d7a..c2df273 100644
--- a/debian/tests/automated-tests
+++ b/debian/tests/automated-tests
@@ -2,7 +2,7 @@
22
3set -ex3set -ex
44
5dh_auto_configure -- \5DH_COMPAT=10 dh_auto_configure -- \
6 --disable-silent-rules \6 --disable-silent-rules \
7 --enable-smartcard7 --enable-smartcard
8make -C subprojects/spice-common/common libspice-common.la libspice-common-server.la8make -C subprojects/spice-common/common libspice-common.la libspice-common-server.la

Subscribers

People subscribed via source and target branches