Bug#872639: cdparanoia FTCBFS: confuses ./configure with a CC that doesn't match host

2017-11-17 Thread Manuel A. Fernandez Montecelo

Control: tags -1 + pending


2017-10-31 21:33 Manuel A. Fernandez Montecelo:


This package has important rev-depends like gstreamer plugins, it would
be nice to be able to cross-build it, and it's a quite simple patch.

What do you maintainers think about it?

If you happen to be busy, would it help if we offer to NMU?


Due to the lack of feedback and uploads in 5 years, I am NMUing this
package.

.debdiff attached.


Cheers.
--
Manuel A. Fernandez Montecelo 

diff -Nru cdparanoia-3.10.2+debian/debian/changelog 
cdparanoia-3.10.2+debian/debian/changelog
--- cdparanoia-3.10.2+debian/debian/changelog   2012-07-14 08:40:47.0 
+0200
+++ cdparanoia-3.10.2+debian/debian/changelog   2017-11-17 22:00:09.0 
+0100
@@ -1,3 +1,14 @@
+cdparanoia (3.10.2+debian-11.1) unstable; urgency=medium
+
+  [ Manuel A. Fernandez Montecelo ]
+  * Non-maintainer upload.
+  * d/control: Drop obsolete "DM-Upload-Allowed: yes"
+
+  [ Helmut Grohne ]
+  * Fix FTCBFS: Pass --host= to ./configure rather than CC=. (Closes: #872639)
+
+ -- Manuel A. Fernandez Montecelo   Fri, 17 Nov 2017 22:00:09 
+0100
+
 cdparanoia (3.10.2+debian-11) unstable; urgency=low
 
   [ TANIGUCHI Takaki ]
diff -Nru cdparanoia-3.10.2+debian/debian/control 
cdparanoia-3.10.2+debian/debian/control
--- cdparanoia-3.10.2+debian/debian/control 2012-07-14 08:40:47.0 
+0200
+++ cdparanoia-3.10.2+debian/debian/control 2017-11-17 21:59:34.0 
+0100
@@ -4,7 +4,6 @@
 Maintainer: Optical Media Tools Team 

 Uploaders: TANIGUCHI Takaki ,
Rogério Brito 
-DM-Upload-Allowed: yes
 Build-Depends:
  debhelper (>= 8.1.3~),
  autotools-dev,
diff -Nru cdparanoia-3.10.2+debian/debian/rules 
cdparanoia-3.10.2+debian/debian/rules
--- cdparanoia-3.10.2+debian/debian/rules   2012-07-14 08:40:47.0 
+0200
+++ cdparanoia-3.10.2+debian/debian/rules   2017-11-17 21:53:02.0 
+0100
@@ -10,7 +10,7 @@
 
 
 ifneq ($(DEB_HOST_GNU_TYPE),$(DEB_BUILD_GNU_TYPE))
-   CROSS=CC=$(DEB_HOST_GNU_TYPE)-gcc
+   CROSS=--build=$(DEB_BUILD_GNU_TYPE) --host=$(DEB_HOST_GNU_TYPE)
 else
CROSS=
 endif


Bug#872639: cdparanoia FTCBFS: confuses ./configure with a CC that doesn't match host

2017-08-19 Thread Helmut Grohne
Source: cdparanoia
Version: 3.10.2+debian-11
Tags: patch
User: helm...@debian.org
Usertags: rebootstrap

cdparanoia fails to cross build from source, because ./configure is
confused about the (implicit) --host not matching CC. It believes we are
building for the build architecture with a cross compiler and thus fails
finding anything reasonabl. It finally gives up after failing to
discover a 16bit integer type.

cdparanoia is sufficiently recent that it should no longer be passed
CC=, but rather --host. After doing that, it cross builds. Please
consider applying the attached patch.

Helmut
diff --minimal -Nru cdparanoia-3.10.2+debian/debian/changelog 
cdparanoia-3.10.2+debian/debian/changelog
--- cdparanoia-3.10.2+debian/debian/changelog   2012-07-14 08:46:36.0 
+0200
+++ cdparanoia-3.10.2+debian/debian/changelog   2017-08-19 18:09:43.0 
+0200
@@ -1,3 +1,10 @@
+cdparanoia (3.10.2+debian-11.1) UNRELEASED; urgency=medium
+
+  * Non-maintainer upload.
+  * Fix FTCBFS: Pass --host= to ./configure rather than CC=. (Closes: -1)
+
+ -- Helmut Grohne   Sat, 19 Aug 2017 18:09:43 +0200
+
 cdparanoia (3.10.2+debian-11) unstable; urgency=low
 
   [ TANIGUCHI Takaki ]
diff --minimal -Nru cdparanoia-3.10.2+debian/debian/rules 
cdparanoia-3.10.2+debian/debian/rules
--- cdparanoia-3.10.2+debian/debian/rules   2012-07-14 08:46:36.0 
+0200
+++ cdparanoia-3.10.2+debian/debian/rules   2017-08-19 18:09:43.0 
+0200
@@ -10,7 +10,7 @@
 
 
 ifneq ($(DEB_HOST_GNU_TYPE),$(DEB_BUILD_GNU_TYPE))
-   CROSS=CC=$(DEB_HOST_GNU_TYPE)-gcc
+   CROSS=--build=$(DEB_BUILD_GNU_TYPE) --host=$(DEB_HOST_GNU_TYPE)
 else
CROSS=
 endif


Bug#872639: cdparanoia FTCBFS: confuses ./configure with a CC that doesn't match host

2017-10-31 Thread Manuel A. Fernandez Montecelo


Hi,

2017-08-19 18:15 Helmut Grohne:

Source: cdparanoia
Version: 3.10.2+debian-11
Tags: patch
User: helm...@debian.org
Usertags: rebootstrap

cdparanoia fails to cross build from source, because ./configure is
confused about the (implicit) --host not matching CC. It believes we are
building for the build architecture with a cross compiler and thus fails
finding anything reasonabl. It finally gives up after failing to
discover a 16bit integer type.

cdparanoia is sufficiently recent that it should no longer be passed
CC=, but rather --host. After doing that, it cross builds. Please
consider applying the attached patch.


This package has important rev-depends like gstreamer plugins, it would
be nice to be able to cross-build it, and it's a quite simple patch.

What do you maintainers think about it?

If you happen to be busy, would it help if we offer to NMU?


Cheers.
--
Manuel A. Fernandez Montecelo