[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#870727: marked as done (malaga FTCBFS: uses the build architecture pkg-config)



Your message dated Sun, 01 May 2022 04:18:47 +0000
with message-id <E1nl12l-0007rG-15@fasolo.debian.org>
and subject line Bug#870727: fixed in malaga 7.12-8
has caused the Debian Bug report #870727,
regarding malaga FTCBFS: uses the build architecture pkg-config
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
870727: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=870727
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message ---
Source: malaga
Version: 7.12-7
Tags: patch
User: helmutg@debian.org
Usertags: rebootstrap

malaga fails to cross build from source, because it uses the build
architecture pkg-config and thus fails finding gtk2 and glib2.0. After
switching configure.in to use PKG_CHECK_MODULES, which considers
$ac_tool_prefix, it fails building the documentation, because gtk2 and
glib2.0 are only requested for the host architecture. Since cross builds
typically skip building architecture-independent packages, moving the
documentation build to -indep overrides makes the build proceed. make
install tries to strip the resulting binaries with the build
architecture strip. This not only breaks cross building, but also
generating -dbgsym packages. It can be nullified by telling install to
strip with /bin/true. The attached patch combines all of that and makes
malaga cross build successfully. Please consider applying it.

Helmut
diff --minimal -Nru malaga-7.12/debian/changelog malaga-7.12/debian/changelog
--- malaga-7.12/debian/changelog	2016-04-07 11:51:00.000000000 +0200
+++ malaga-7.12/debian/changelog	2017-08-04 13:28:11.000000000 +0200
@@ -1,3 +1,13 @@
+malaga (7.12-7.1) UNRELEASED; urgency=medium
+
+  * Non-maintainer upload.
+  * Fix FTCBFS: Closes: #-1
+    + Add cross.patch.
+    + Do not build the documentation during arch-only.
+    + Do not strip during install.
+
+ -- Helmut Grohne <helmut@subdivi.de>  Fri, 04 Aug 2017 13:28:11 +0200
+
 malaga (7.12-7) unstable; urgency=low
 
   * QA upload.
diff --minimal -Nru malaga-7.12/debian/control malaga-7.12/debian/control
--- malaga-7.12/debian/control	2016-04-07 10:00:00.000000000 +0200
+++ malaga-7.12/debian/control	2017-08-04 13:22:23.000000000 +0200
@@ -2,7 +2,7 @@
 Maintainer: Debian QA Group <packages@qa.debian.org>
 Section: misc
 Priority: optional
-Build-Depends: libgtk2.0-dev, libglib2.0-dev, texinfo, texlive, libreadline-dev, debhelper (>= 9), autotools-dev, libcairo-dev, dpkg (>= 1.15.4) | install-info, dh-autoreconf
+Build-Depends: libgtk2.0-dev, libglib2.0-dev, texinfo, texlive, libreadline-dev, debhelper (>= 9), autotools-dev, libcairo-dev, dpkg (>= 1.15.4) | install-info, dh-autoreconf, pkg-config
 Standards-Version: 3.9.2
 
 Package: malaga-bin
diff --minimal -Nru malaga-7.12/debian/patches/cross.diff malaga-7.12/debian/patches/cross.diff
--- malaga-7.12/debian/patches/cross.diff	1970-01-01 01:00:00.000000000 +0100
+++ malaga-7.12/debian/patches/cross.diff	2017-08-04 13:28:11.000000000 +0200
@@ -0,0 +1,53 @@
+From: Helmut Grohne <helmut@subdivi.de>
+Subject: let PKG_CHECK_MODULES consider $ac_tool_prefix
+
+Index: malaga-7.12/configure.in
+===================================================================
+--- malaga-7.12.orig/configure.in
++++ malaga-7.12/configure.in
+@@ -58,12 +58,7 @@
+               use_malshow=$enableval, use_malshow=yes)
+ if test $use_malshow = yes
+ then
+-  AC_MSG_CHECKING(for GTK+ 2.8 or later)
+-  if ! $SHELL -c 'pkg-config --atleast-version=2.8 gtk+-2.0' >/dev/null 2>/dev/null
+-  then 
+-    use_malshow=no
+-  fi
+-  AC_MSG_RESULT($use_malshow)
++  PKG_CHECK_MODULES([GTK], [gtk+-2.0 >= 2.8], [use_malshow=yes], [use_malshow=no])
+   if test $use_malshow = no
+   then
+     echo 1>&2 "*** No GTK+, version 2.8 or later, found. Disabling malshow."
+@@ -72,29 +67,15 @@
+ fi
+ if test $use_malshow = yes
+ then
+-  GTK_LIBS=`pkg-config gtk+-2.0 --libs`
+-  GTK_CFLAGS=`pkg-config gtk+-2.0 --cflags`
+   MALSHOW="malshow"
+ else
+-  GTK_LIBS=""
+-  GTK_CFLAGS=""
+   MALSHOW=""
+ fi
+ 
+ dnl Check for GLib, version 2
+-AC_MSG_CHECKING(for GLib)
+-if $SHELL -c 'pkg-config glib-2.0' >/dev/null 2>/dev/null
+-then
+-  have_glib=yes
+-else
+-  have_glib=no
+-fi
+-AC_MSG_RESULT($have_glib)
+-if test $have_glib = yes
++PKG_CHECK_MODULES([GLIB], [glib-2.0], [have_glib=yes], [have_glib=no])
++if test $have_glib != yes
+ then 
+-  GLIB_LIBS=`pkg-config glib-2.0 --libs`
+-  GLIB_CFLAGS=`pkg-config glib-2.0 --cflags`
+-else
+   echo 1>&2 "*** No GLib, version 2.0 or later, found. Please install it."
+   echo 1>&2 "*** You may get GLib from <http://www.gtk.org>."
+   exit 1
diff --minimal -Nru malaga-7.12/debian/patches/series malaga-7.12/debian/patches/series
--- malaga-7.12/debian/patches/series	2016-04-07 10:00:00.000000000 +0200
+++ malaga-7.12/debian/patches/series	2017-08-04 13:27:29.000000000 +0200
@@ -4,3 +4,4 @@
 no-ncurses.diff
 link-libm.diff
 texinfo.diff
+cross.diff
diff --minimal -Nru malaga-7.12/debian/rules malaga-7.12/debian/rules
--- malaga-7.12/debian/rules	2016-04-07 10:00:00.000000000 +0200
+++ malaga-7.12/debian/rules	2017-08-04 13:28:11.000000000 +0200
@@ -1,5 +1,7 @@
 #!/usr/bin/make -f
 
+export INSTALL=install --strip-program=true
+
 %:
 	dh $@ --with autoreconf
 
@@ -7,13 +9,11 @@
 	rm -rf build-doc
 	dh_auto_clean
 
-override_dh_auto_configure:
-	dh_auto_configure
+override_dh_auto_configure-indep:
 	mkdir -p build-doc
 	cd build-doc && ../configure --prefix=/usr
 
-override_dh_auto_build:
-	dh_auto_build
+override_dh_auto_build-indep:
 	$(MAKE) -C build-doc dvi html ps pdf
 
 override_dh_auto_install:

--- End Message ---
--- Begin Message ---
Source: malaga
Source-Version: 7.12-8
Done: Vagrant Cascadian <vagrant@debian.org>

We believe that the bug you reported is fixed in the latest version of
malaga, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 870727@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Vagrant Cascadian <vagrant@debian.org> (supplier of updated malaga package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@ftp-master.debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Format: 1.8
Date: Sat, 30 Apr 2022 20:50:59 -0700
Source: malaga
Architecture: source
Version: 7.12-8
Distribution: unstable
Urgency: medium
Maintainer: Debian QA Group <packages@qa.debian.org>
Changed-By: Vagrant Cascadian <vagrant@debian.org>
Closes: 870727 995654
Changes:
 malaga (7.12-8) unstable; urgency=medium
 .
   [ Vagrant Cascadian ]
   * QA upload.
   * debian/rules: Export FORCE_SOURCE_DATE=1 in order for texlive to
     respect SOURCE_DATE_EPOCH when generating .dvi, .pdf and .ps files.
     Closes: #995654
   * malaga-bin.preinst: Use "set -e" in script.
   * debian/control: Drop redundant Priority and Section fields.
   * Update to debhelper-compat 13.
   * debian/control: Set Rules-Requires-Root to "no".
   * libmalaga7: Switch ldconfig to use dpkg triggers.
   * debian/control: Update Standards-Version to 4.6.0.
 .
   [ Helmut Grohne ]
   * Fix FTCBFS: Closes: #870727
     + Add cross.patch.
     + Do not build the documentation during arch-only.
     + Do not strip during install.
Checksums-Sha1:
 50e18f88ff1d3f4adf0f656fbdabfd77dbf9f009 1542 malaga_7.12-8.dsc
 348cfadc21646ce33270827df5f9a7ebd880c559 12380 malaga_7.12-8.debian.tar.xz
Checksums-Sha256:
 667a418aa9831edf7af2fe1a5e6cdd7b84b62af6487ab0333db69617793858f4 1542 malaga_7.12-8.dsc
 67c67112c3ddf8166e668e95b4200956405aa8427dd44e1b3c9655a289ad7f9c 12380 malaga_7.12-8.debian.tar.xz
Files:
 d98b98c6b429a666f5f6a41ea0061ac2 1542 misc optional malaga_7.12-8.dsc
 a23fa8a7e6b5bdd894f3988076a77cd4 12380 misc optional malaga_7.12-8.debian.tar.xz

-----BEGIN PGP SIGNATURE-----

iIkEARYKADEWIQRlgHNhO/zFx+LkXUXcUY/If5cWqgUCYm4HJBMcdmFncmFudEBk
ZWJpYW4ub3JnAAoJENxRj8h/lxaqVGQA/RykjFBdzpNvsEqUEqWpz5/lMMLH/Oav
9iZqqjCmRCO4AP41lU5ppqflHkXsdrz9laduJCj2HjmSq2/n4w8rf0myDg==
=/VUK
-----END PGP SIGNATURE-----

--- End Message ---

Reply to: