Bug#847096: Bug#905230: elfutils: FTBFS on x32 due to bugs in testsuite
On Thu, Aug 02, 2018 at 11:11:24PM +0200, Thorsten Glaser wrote: > > Given #847096 it might be wise to just *always* disable the biarch stuff > since Debian does not normally use biarch *anyway*, and since we test it > on all architectures during their native build anyway. I'm not conviced that that is a good solution. Elfutils supports many different backends at the same time. You disable a test that tests part of that functionality. The only thing I've seen here so far is a setup that breaks elfutils' test suite. As far as I understand, it's perfectly possible to build the x32 version. Kurt
Bug#905230: elfutils: FTBFS on x32 due to bugs in testsuite
Thanks Kurt. I wasn't familiar with upstream's update cadence, but I now see 0.175 is imminent: https://sourceware.org/ml/elfutils-devel/2018-q4/msg00104.html -- Laurence > >
Bug#905230: elfutils: FTBFS on x32 due to bugs in testsuite
On Mon, Nov 12, 2018 at 11:15:48AM +, Laurence Parry wrote: > This seems to be blocking me from building a x32 chroot using the > documented process for doing so, because the ELF library is a required > package (iproute2 depends on libelf-dev as of 4.6.0-2; see #812774). I > appreciate that x32 is not a great priority, but it's not going to get > better if we can't even spin up a build. >_> > > The maintainer does not appear to have updated this package in the last > year, so I'm not sure when the next "regular update" will be. Would it be > possible for a developer to make a non-maintainer upload to resolve this? I'm expecting a new upstream version soon, and will do an upload of that then. Kurt
Bug#905230: elfutils: FTBFS on x32 due to bugs in testsuite
This seems to be blocking me from building a x32 chroot using the documented process for doing so, because the ELF library is a required package (iproute2 depends on libelf-dev as of 4.6.0-2; see #812774). I appreciate that x32 is not a great priority, but it's not going to get better if we can't even spin up a build. >_> The maintainer does not appear to have updated this package in the last year, so I'm not sure when the next "regular update" will be. Would it be possible for a developer to make a non-maintainer upload to resolve this? Best regards, -- Laurence "GreenReaper" Parry
Bug#905230: elfutils: FTBFS on x32 due to bugs in testsuite
Hi Helmut, > I don't think this is correct. The above configure invocation will only > run when performing a cross build. confenv will only contain your > variable when the host arch is x32 you’re completely right. Sorry. Given #847096 it might be wise to just *always* disable the biarch stuff since Debian does not normally use biarch *anyway*, and since we test it on all architectures during their native build anyway. So drop the “ifneq (,$(filter x32,$(DEB_HOST_ARCH)))” and the correspon‐ ding endif line from the patch. bye, //mirabilos -- tarent solutions GmbH Rochusstraße 2-4, D-53123 Bonn • http://www.tarent.de/ Tel: +49 228 54881-393 • Fax: +49 228 54881-235 HRB 5168 (AG Bonn) • USt-ID (VAT): DE122264941 Geschäftsführer: Dr. Stefan Barth, Kai Ebenrett, Boris Esser, Alexander Steeg * **Besuchen Sie uns auf der dmexco 2018!** 12**. **& 13. September 2018, Koelnmesse / **Halle 7,** **Stand A-031** Digital Business, Marketing und Innovation [www.tarent.de/dmexco](http://www.tarent.de/dmexco) * **Visit us at dmexco 2018!** 12th & 13th September, 2018, Koelnmesse / **Hall 7,** **Booth A-031** Digital business, marketing and innovation [www.tarent.de/dmexco](http://www.tarent.de/dmexco) *
Bug#905230: Fwd: Bug#905230: elfutils: FTBFS on x32 due to bugs in testsuite
On Wed, Aug 01, 2018 at 06:18:42PM +0200, Thorsten Glaser wrote: > +ifneq (,$(filter x32,$(DEB_HOST_ARCH))) > + confenv += utrace_cv_cc_biarch=no > +endif > + > ifneq (,$(filter parallel=%,$(DEB_BUILD_OPTIONS))) > NUMJOBS = $(patsubst parallel=%,%,$(filter > parallel=%,$(DEB_BUILD_OPTIONS))) > MAKEFLAGS += -j$(NUMJOBS) > @@ -37,12 +42,12 @@ > dh_testdir > autoreconf -fis > ifneq ($(DEB_BUILD_GNU_TYPE),$(DEB_HOST_GNU_TYPE)) > - ./configure --enable-maintainer-mode > + $(confenv) ./configure --enable-maintainer-mode > $(MAKE) $(MAKEFLAGS) > $(MAKE) clean > endif I don't think this is correct. The above configure invocation will only run when performing a cross build. confenv will only contain your variable when the host arch is x32, so this configure invocation is only run when cross building from something other than x32 to x32 and it is the native build pass, so it can never happen on x32. Helmut
Bug#905230: elfutils: FTBFS on x32 due to bugs in testsuite
Source: elfutils
Version: 0.170-0.5
Severity: important
Justification: fails to build from source (but built successfully in the past)
debian/patches/testsuite-core-files.diff essentially cleans up
a core file that is only created when the architecture to be
tested does not support unwinding as tested in run-backtrace-dwarf.sh
which made the “rmdir” after the test fail.
The change to debian/rules lets me successfully build this.
It might be related to eatmydata, or not, but it’s actually
nontrivial to disable eatmydata in my cowbuilder environment…
Please apply with the next regular upload to unstable.
Thanks!
diff -Nru elfutils-0.170/debian/changelog elfutils-0.170/debian/changelog
--- elfutils-0.170/debian/changelog 2018-06-24 20:54:50.0 +0200
+++ elfutils-0.170/debian/changelog 2018-08-01 17:53:57.0 +0200
@@ -1,3 +1,11 @@
+elfutils (0.170-0.5+x32.1) unreleased; urgency=medium
+
+ * Non-maintainer upload.
+ * Disable biarch tests on x32 to fix FTBFS.
+ * Patch testsuite to clean up core files created on some targets.
+
+ -- Thorsten Glaser Wed, 01 Aug 2018 17:53:57 +0200
+
elfutils (0.170-0.5) unstable; urgency=medium
* Non-maintainer upload acked by Kurt Roeckx.
diff -Nru elfutils-0.170/debian/patches/series
elfutils-0.170/debian/patches/series
--- elfutils-0.170/debian/patches/series2018-04-09 14:21:19.0
+0200
+++ elfutils-0.170/debian/patches/series2018-08-01 17:52:25.0
+0200
@@ -13,3 +13,4 @@
disable_werror.patch
GNU_variable_value.patch
locviews.patch
+testsuite-core-files.diff
diff -Nru elfutils-0.170/debian/patches/testsuite-core-files.diff
elfutils-0.170/debian/patches/testsuite-core-files.diff
--- elfutils-0.170/debian/patches/testsuite-core-files.diff 1970-01-01
01:00:00.0 +0100
+++ elfutils-0.170/debian/patches/testsuite-core-files.diff 2018-08-01
17:53:50.0 +0200
@@ -0,0 +1,11 @@
+# DP: fix FTBFS on x32 where unwinding is not supported
+
+--- a/tests/run-backtrace-dwarf.sh
b/tests/run-backtrace-dwarf.sh
+@@ -28,3 +28,6 @@ tempfiles dwarf.{bt,err}
+ cat dwarf.{bt,err}
+ check_native_unsupported dwarf.err dwarf
+ check_main dwarf.bt dwarf
++# running this on architectures not supporting unwinding calls abort()
++# causing core dump creation
++rm -f core
diff -Nru elfutils-0.170/debian/rules elfutils-0.170/debian/rules
--- elfutils-0.170/debian/rules 2017-09-06 10:11:35.0 +0200
+++ elfutils-0.170/debian/rules 2018-08-01 17:29:17.0 +0200
@@ -15,6 +15,7 @@
# These are used for cross-compiling and for saving the configure script
# from having to guess our platform (since we know it already)
+DEB_HOST_ARCH ?= $(shell dpkg-architecture -qDEB_HOST_ARCH)
DEB_HOST_GNU_TYPE ?= $(shell dpkg-architecture -qDEB_HOST_GNU_TYPE)
DEB_BUILD_GNU_TYPE ?= $(shell dpkg-architecture -qDEB_BUILD_GNU_TYPE)
DEB_HOST_MULTIARCH ?= $(shell dpkg-architecture -qDEB_HOST_MULTIARCH)
@@ -25,6 +26,10 @@
confflags += --build=$(DEB_BUILD_GNU_TYPE) --host=$(DEB_HOST_GNU_TYPE)
endif
+ifneq (,$(filter x32,$(DEB_HOST_ARCH)))
+ confenv += utrace_cv_cc_biarch=no
+endif
+
ifneq (,$(filter parallel=%,$(DEB_BUILD_OPTIONS)))
NUMJOBS = $(patsubst parallel=%,%,$(filter
parallel=%,$(DEB_BUILD_OPTIONS)))
MAKEFLAGS += -j$(NUMJOBS)
@@ -37,12 +42,12 @@
dh_testdir
autoreconf -fis
ifneq ($(DEB_BUILD_GNU_TYPE),$(DEB_HOST_GNU_TYPE))
- ./configure --enable-maintainer-mode
+ $(confenv) ./configure --enable-maintainer-mode
$(MAKE) $(MAKEFLAGS)
$(MAKE) clean
endif
CFLAGS="$(CFLAGS) -O3" CPPFLAGS="$(CPPFLAGS)" LDFLAGS="$(LDFLAGS)" \
- ./configure $(confflags) --prefix=/usr \
+ $(confenv) ./configure $(confflags) --prefix=/usr \
--libdir=/usr/lib/$(DEB_HOST_MULTIARCH) \
--program-prefix=eu- --disable-silent-rules
