Jeff Breidenbach wrote:
I'm quite happy to have NMU, with no delay necessary. It especially
makes sense since I'm not set up to test on ARM. Also, upstream has some ideas. I'll see if they are amenable to direct contact.
-frename-registers made it build. Debdiff attatched and uploaded as NMU

I've realised just after I uploaded that I screwed up the comment in debian/rules (what actually happened is I corrected the comment but forgot to save the file). It should say

# gcc 4.9 ICEs when building libwebp for arm64,
# using -frename-registers works arround this.
# http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=757738

I don't think it's worth doing a second upload just to fix a comment but you might want to fix the comment but you might want to include the corrected comment in your packaging tree so it goes in the next upload.
PS. How the heck did webp become bootstrap critical?
Same way a lot of other stuff did. As far as debian is concerned a source package is either completely built or not built at all.

So almost certainly someone added webp support to a package in the strong set of packages that must be built to have a healthy archive where all source packages that produce binaries have their build-dependencies satisfiable making libwebp part of that set. I'm not sure which of your rdepends it was that's in the strong set (unfortunately i'm not finding a list of the strong set online, I guess most people who need to know about it calculate it on the fly since it changes over time anyway) but looking through your rdepends it seems some pretty important stuff has picked up dependencies on libwebp,

There is work underway to add build-profiles to ease bootstrapping by allowing source packages to be built with reduced functionality but even then afaict there is no intention to accept profile-built packages in the official archive, just ot use them as an intermediate step to break loops in bootstrapping.

diff -Nru libwebp-0.4.1/debian/changelog libwebp-0.4.1/debian/changelog
--- libwebp-0.4.1/debian/changelog      2014-07-30 23:39:36.000000000 +0000
+++ libwebp-0.4.1/debian/changelog      2014-08-12 11:16:49.000000000 +0000
@@ -1,3 +1,10 @@
+libwebp (0.4.1-1.1) unstable; urgency=medium
+
+  * NMU with maintainer's permission
+  * use -frename-registers as workaround for ICE on arm64 (Closes: #757785)
+
+ -- Peter Michael Green <plugw...@debian.org>  Tue, 12 Aug 2014 11:15:58 +0000
+
 libwebp (0.4.1-1) unstable; urgency=medium
 
   * New upstream release
diff -Nru libwebp-0.4.1/debian/rules libwebp-0.4.1/debian/rules
--- libwebp-0.4.1/debian/rules  2014-01-08 18:14:20.000000000 +0000
+++ libwebp-0.4.1/debian/rules  2014-08-11 21:43:18.000000000 +0000
@@ -6,6 +6,13 @@
 # dh-make output file, you may use that output file without restriction.
 # This special exception was added by Craig Small in version 0.37 of dh-make.
 
+# gcc 4.9 ICEs when building libwebp for arm64, use 4.8 for now
+# http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=757738
+DEB_HOST_ARCH ?= $(shell dpkg-architecture -q DEB_HOST_ARCH)
+ifeq ($(DEB_HOST_ARCH),arm64)
+  export DEB_CFLAGS_MAINT_APPEND=-frename-registers
+endif
+
 # Uncomment this to turn on verbose mode.
 export DH_VERBOSE=1
 

Reply via email to