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

Bug#632102: marked as done (ftbs with --as-needed)

Your message dated Sat, 01 Jul 2023 18:05:05 +0000
with message-id <[🔎] E1qFey1-003aFI-KM@fasolo.debian.org>
and subject line Bug#1039949: Removed package(s) from unstable
has caused the Debian Bug report #632102,
regarding ftbs with --as-needed
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.)


-- 
632102: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=632102
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: pads
Version: 1.2-11
Severity: wishlist
Tags: patch
User: ubuntu-devel@lists.ubuntu.com
Usertags: origin-ubuntu oneiric ubuntu-patch
User: debian-gcc@lists.debian.org
Usertags: ld-as-needed

the package pads fails to build when using the linker flag --as-needed
This is caused by the libraries being placed inside LDFLAGS. Libraries
to link with should be placed in LIBS or LDADD so the commend line is
correctly ordered.
See the buildlogs in ubuntu oneiric which has this flag set by default:
https://launchpad.net/ubuntu/+source/pads/1.2-11

Attached patch changes both configure.in and configure althoug only
configure patching is needed currently.
If the packaging is changed to regenerate the autotools files the
patching of configure can be dropped.


Description: fix build with --as-needed
 one must use LIBS instead of LDFLAGS for libraries so autotools
 orders the command line correctly
Author: Julian Taylor <jtaylor.debian@googlemail.com>
Index: pads-1.2/configure
===================================================================
--- pads-1.2.orig/configure	2011-06-29 18:24:01.996859577 +0200
+++ pads-1.2/configure	2011-06-29 18:24:29.106859587 +0200
@@ -3679,7 +3679,7 @@
 
 done
 
-LDFLAGS="${LDFLAGS} `pcre-config --libs`"
+LIBS="${LIBS} `pcre-config --libs`"
 echo "$as_me:$LINENO: checking for pcre_compile in -lpcre" >&5
 echo $ECHO_N "checking for pcre_compile in -lpcre... $ECHO_C" >&6
 if test "${ac_cv_lib_pcre_pcre_compile+set}" = set; then
Index: pads-1.2/configure.in
===================================================================
--- pads-1.2.orig/configure.in	2011-06-29 18:24:03.446859577 +0200
+++ pads-1.2/configure.in	2011-06-29 18:24:46.386859593 +0200
@@ -81,7 +81,7 @@
 AC_CHECK_HEADERS(pcre.h,
     AC_DEFINE(HAVE_PCRE_H, 1, Have PCRE Header File),
     AC_MSG_ERROR([Cannot find PCRE header files!]))
-LDFLAGS="${LDFLAGS} `pcre-config --libs`"
+LIBS="${LIBS} `pcre-config --libs`"
 AC_CHECK_LIB(pcre, pcre_compile,
     EXTRA_LIBS="$EXTRA_LIBS -lpcre",
     AC_MSG_ERROR([Cannot find PCRE library files!]))

Attachment: signature.asc
Description: OpenPGP digital signature


--- End Message ---
--- Begin Message --- 
Version: 1.2-14+rm

Dear submitter,

as the package pads has just been removed from the Debian archive
unstable we hereby close the associated bug reports.  We are sorry
that we couldn't deal with your issue properly.

For details on the removal, please see https://bugs.debian.org/1039949

The version of this package that was in Debian prior to this removal
can still be found using https://snapshot.debian.org/.

Please note that the changes have been done on the master archive and
will not propagate to any mirrors until the next dinstall run at the
earliest.

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

Debian distribution maintenance software
pp.
Scott Kitterman (the ftpmaster behind the curtain)

--- End Message ---
Reply to: