Merge lp:~scarneiro/ubuntu/oneiric/gupnp-ui/fix-for-749152 into lp:ubuntu/oneiric/gupnp-ui

Proposed by Sebastian Carneiro
Status: Merged
Merged at revision: 5
Proposed branch: lp:~scarneiro/ubuntu/oneiric/gupnp-ui/fix-for-749152
Merge into: lp:ubuntu/oneiric/gupnp-ui
Diff against target: 52 lines (+11/-2)
4 files modified
configure (+1/-0)
configure.ac (+1/-1)
debian/changelog (+7/-0)
debian/control (+2/-1)
To merge this branch: bzr merge lp:~scarneiro/ubuntu/oneiric/gupnp-ui/fix-for-749152
Reviewer Review Type Date Requested Status
Marc Deslauriers Approve
Ubuntu branches Pending
Review via email: mp+68552@code.launchpad.net

Description of the change

In configure and configure.ac the libgupnp libraries are added to LIBS variable, to avoid issues with indirect linking when using ld --no-add-needed.

To post a comment you must log in.
6. By Sebastian Carneiro

Maintainer fields updated.

Revision history for this message
Marc Deslauriers (mdeslaur) wrote :

Looks good. ACK!
Thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'configure'
--- configure 2009-04-01 10:52:03 +0000
+++ configure 2011-07-20 14:42:35 +0000
@@ -20909,6 +20909,7 @@
20909else20909else
20910 LIBGUPNP_UI_CFLAGS=$pkg_cv_LIBGUPNP_UI_CFLAGS20910 LIBGUPNP_UI_CFLAGS=$pkg_cv_LIBGUPNP_UI_CFLAGS
20911 LIBGUPNP_UI_LIBS=$pkg_cv_LIBGUPNP_UI_LIBS20911 LIBGUPNP_UI_LIBS=$pkg_cv_LIBGUPNP_UI_LIBS
20912 LIBS="$LIBS $LIBGUPNP_UI_LIBS"
20912 { echo "$as_me:$LINENO: result: yes" >&520913 { echo "$as_me:$LINENO: result: yes" >&5
20913echo "${ECHO_T}yes" >&6; }20914echo "${ECHO_T}yes" >&6; }
20914 :20915 :
2091520916
=== modified file 'configure.ac'
--- configure.ac 2009-04-01 10:52:03 +0000
+++ configure.ac 2011-07-20 14:42:35 +0000
@@ -10,7 +10,7 @@
10AC_STDC_HEADERS10AC_STDC_HEADERS
11AC_PROG_LIBTOOL11AC_PROG_LIBTOOL
1212
13PKG_CHECK_MODULES(LIBGUPNP_UI, gupnp-1.0 >= 0.3 gtk+-2.0)13PKG_CHECK_MODULES(LIBGUPNP_UI, gupnp-1.0 >= 0.3 gtk+-2.0, LIBS="$LIBS $LIBGUPNP_UI_LIBS")
1414
15PKG_CHECK_MODULES(GTHREAD, gthread-2.0)15PKG_CHECK_MODULES(GTHREAD, gthread-2.0)
1616
1717
=== modified file 'debian/changelog'
--- debian/changelog 2009-09-26 13:39:55 +0000
+++ debian/changelog 2011-07-20 14:42:35 +0000
@@ -1,3 +1,10 @@
1gupnp-ui (0.1.1-3ubuntu1) oneiric; urgency=low
2
3 * configure, configure.ac: add libgupnp libraries to LIBS in
4 order to solve FTBFS with ld --no-add-needed (LP: #749152)
5
6 -- Sebastian Carneiro <scarneiro@fibertel.com.ar> Wed, 20 Jul 2011 09:50:52 -0300
7
1gupnp-ui (0.1.1-3) unstable; urgency=low8gupnp-ui (0.1.1-3) unstable; urgency=low
29
3 * Rebuild against new libgupnp10 * Rebuild against new libgupnp
411
=== modified file 'debian/control'
--- debian/control 2009-09-26 13:39:55 +0000
+++ debian/control 2011-07-20 14:42:35 +0000
@@ -1,7 +1,8 @@
1Source: gupnp-ui1Source: gupnp-ui
2Section: libs2Section: libs
3Priority: optional3Priority: optional
4Maintainer: Ross Burton <ross@debian.org>4Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
5XSBC-Original-Maintainer: Ross Burton <ross@debian.org>
5Build-Depends: debhelper (>= 5), cdbs, libgupnp-1.0-dev (>= 0.13), libgtk2.0-dev, uuid-dev6Build-Depends: debhelper (>= 5), cdbs, libgupnp-1.0-dev (>= 0.13), libgtk2.0-dev, uuid-dev
6Standards-Version: 3.8.37Standards-Version: 3.8.3
78

Subscribers

People subscribed via source and target branches