Merge ~sergiodj/ubuntu/+source/libvpx:fix-ftbfs-gcc-11 into ubuntu/+source/libvpx:ubuntu/devel

Proposed by Sergio Durigan Junior
Status: Merged
Approved by: Sergio Durigan Junior
Approved revision: f1b8bbca2223c38ceb28598835237412a834cdc4
Merged at revision: f1b8bbca2223c38ceb28598835237412a834cdc4
Proposed branch: ~sergiodj/ubuntu/+source/libvpx:fix-ftbfs-gcc-11
Merge into: ubuntu/+source/libvpx:ubuntu/devel
Diff against target: 43 lines (+12/-1)
3 files modified
debian/changelog (+7/-0)
debian/control (+2/-1)
debian/rules (+3/-0)
Reviewer Review Type Date Requested Status
Lucas Kanashiro (community) Approve
Canonical Server packageset reviewers Pending
Review via email: mp+407095@code.launchpad.net

Description of the change

This MP works around the LTO problems that are causing libvpx to FTBFS with GCC 11.

I made a mistake and added the package to the lto-disabled-list, but libvpx is in main and therefore the problem should be fixed/worked around in the package itself. That's why I'm now filing this MP (after having removed libvpx from lto-disabled-list).

I have verified that this problem is still happening with upstream's git HEAD, so I've filed a report against them here:

https://bugs.chromium.org/p/webm/issues/detail?id=1736

There's a PPA with the proposed change here:

https://launchpad.net/~sergiodj/+archive/ubuntu/ftbfs-impish/+packages

dep8 tests are still happy:

autopkgtest [11:14:57]: @@@@@@@@@@@@@@@@@@@@ summary
encode-testimage PASS

To post a comment you must log in.
Revision history for this message
Lucas Kanashiro (lucaskanashiro) wrote :

Thanks for this MP Sergio, the changes LGTM, +1.

review: Approve
Revision history for this message
Sergio Durigan Junior (sergiodj) wrote :

On Monday, August 16 2021, Lucas Kanashiro wrote:

> Thanks for this MP Sergio, the changes LGTM, +1.

Thanks for the review, Lucas. Uploaded:

$ git push pkg upload/1.9.0-1ubuntu1
Enumerating objects: 18, done.
Counting objects: 100% (18/18), done.
Delta compression using up to 8 threads
Compressing objects: 100% (13/13), done.
Writing objects: 100% (13/13), 1.61 KiB | 550.00 KiB/s, done.
Total 13 (delta 9), reused 0 (delta 0)
To ssh://git.launchpad.net/ubuntu/+source/libvpx
 * [new tag] upload/1.9.0-1ubuntu1 -> upload/1.9.0-1ubuntu1

$ dput libvpx_1.9.0-1ubuntu1_source.changes
Trying to upload package to ubuntu
Checking signature on .changes
gpg: /home/sergio/work/libvpx/libvpx_1.9.0-1ubuntu1_source.changes: Valid signature from 106DA1C8C3CBBF14
Checking signature on .dsc
gpg: /home/sergio/work/libvpx/libvpx_1.9.0-1ubuntu1.dsc: Valid signature from 106DA1C8C3CBBF14
Uploading to ubuntu (via ftp to upload.ubuntu.com):
  Uploading libvpx_1.9.0-1ubuntu1.dsc: done.
  Uploading libvpx_1.9.0-1ubuntu1.debian.tar.xz: done.
  Uploading libvpx_1.9.0-1ubuntu1_source.buildinfo: done.
  Uploading libvpx_1.9.0-1ubuntu1_source.changes: done.
Successfully uploaded packages.

--
Sergio
GPG key ID: E92F D0B3 6B14 F1F4 D8E0 EB2F 106D A1C8 C3CB BF14

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 d32bede..f559ac1 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,10 @@
6+libvpx (1.9.0-1ubuntu1) impish; urgency=medium
7+
8+ * d/rules: Disable LTO and fix FTBFS when building with GCC 11.
9+ (LP: #1939640)
10+
11+ -- Sergio Durigan Junior <sergio.durigan@canonical.com> Wed, 11 Aug 2021 22:25:45 -0400
12+
13 libvpx (1.9.0-1) unstable; urgency=medium
14
15 * Team upload
16diff --git a/debian/control b/debian/control
17index 4203206..decace7 100644
18--- a/debian/control
19+++ b/debian/control
20@@ -1,7 +1,8 @@
21 Source: libvpx
22 Section: video
23 Priority: optional
24-Maintainer: Debian Multimedia Maintainers <debian-multimedia@lists.debian.org>
25+Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
26+XSBC-Original-Maintainer: Debian Multimedia Maintainers <debian-multimedia@lists.debian.org>
27 Uploaders: Sebastian Dröge <slomo@debian.org>,
28 Ondřej Nový <onovy@debian.org>,
29 Build-Depends: debhelper-compat (= 13),
30diff --git a/debian/rules b/debian/rules
31index e186702..cb1b2ed 100755
32--- a/debian/rules
33+++ b/debian/rules
34@@ -5,6 +5,9 @@ export DEB_BUILD_MAINT_OPTIONS = hardening=+all
35 DEB_CFLAGS_MAINT_APPEND = -Wall
36 DEB_CXXFLAGS_MAINT_APPEND = -Wall
37
38+# The build fails with GCC 11 when using LTO.
39+export DEB_BUILD_MAINT_OPTIONS = optimize=-lto
40+
41 include /usr/share/dpkg/architecture.mk
42
43 configure_flags += \

Subscribers

People subscribed via source and target branches