Merge lp:~jtaylor/ubuntu/oneiric/buzztard/fix-803173 into lp:ubuntu/oneiric/buzztard

Proposed by Julian Taylor
Status: Merged
Merged at revision: 6
Proposed branch: lp:~jtaylor/ubuntu/oneiric/buzztard/fix-803173
Merge into: lp:ubuntu/oneiric/buzztard
Diff against target: 114 lines (+82/-1)
4 files modified
debian/changelog (+7/-0)
debian/control (+2/-1)
debian/patches/04_fix-as-needed-build.patch (+72/-0)
debian/patches/series (+1/-0)
To merge this branch: bzr merge lp:~jtaylor/ubuntu/oneiric/buzztard/fix-803173
Reviewer Review Type Date Requested Status
Chase Douglas (community) Approve
Ubuntu branches Pending
Review via email: mp+72336@code.launchpad.net

Description of the change

fix ftbs

To post a comment you must log in.
Revision history for this message
Sebastien Bacher (seb128) wrote :

Thank you for your work there

Revision history for this message
Chase Douglas (chasedouglas) wrote :

The fix looks good. When proposing package changes, please leave the distro as UNRELEASED instead of the real distro (i.e. oneiric). The uploader will flip UNRELEASED to the real distro before uploading.

We should wait a few days to see if the Debian maintainer will accept and upload a fix for this bug. If they do, then we can sync the package back to Oneiric. If not, we can upload this to Ubuntu. For now, I'm going to approve the merge proposal as I think the content is correct.

review: Approve
Revision history for this message
Julian Taylor (jtaylor) wrote :

the patch was forwarded to debian on 07 Jul 2011 with no answer yet.

Revision history for this message
Fabrice Coutadeur (fabricesp) wrote :

Merged. Thanks for your work! If Debian accepts the change, we can always request the sync later on.

Thanks!
Fabrice

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2011-03-28 21:26:12 +0000
3+++ debian/changelog 2011-08-21 14:22:24 +0000
4@@ -1,3 +1,10 @@
5+buzztard (0.5.0-4ubuntu1) oneiric; urgency=low
6+
7+ * debian/patches/04_fix-as-needed-build.patch:
8+ - fix build with ld --as-needed (LP: #803173)
9+
10+ -- Julian Taylor <jtaylor.debian@googlemail.com> Sun, 21 Aug 2011 16:12:21 +0200
11+
12 buzztard (0.5.0-4) unstable; urgency=low
13
14 * Team upload.
15
16=== modified file 'debian/control'
17--- debian/control 2011-03-28 21:26:12 +0000
18+++ debian/control 2011-08-21 14:22:24 +0000
19@@ -1,7 +1,8 @@
20 Source: buzztard
21 Section: sound
22 Priority: optional
23-Maintainer: Maintainers of GStreamer packages <pkg-gstreamer-maintainers@lists.alioth.debian.org>
24+Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
25+XSBC-Original-Maintainer: Maintainers of GStreamer packages <pkg-gstreamer-maintainers@lists.alioth.debian.org>
26 Uploaders: Sebastian Dröge <slomo@debian.org>
27 Build-Depends: cdbs,
28 debhelper (>= 7),
29
30=== added file 'debian/patches/04_fix-as-needed-build.patch'
31--- debian/patches/04_fix-as-needed-build.patch 1970-01-01 00:00:00 +0000
32+++ debian/patches/04_fix-as-needed-build.patch 2011-08-21 14:22:24 +0000
33@@ -0,0 +1,72 @@
34+Description: fix build with ld --as-needed
35+ when building with ld --as-needed libraries must be placed
36+ after object files needing them on the command line so the symbols
37+ of the libraries are registered as needed.
38+Author: Julian Taylor <jtaylor.debian@googlemail.com>
39+Bug-Ubuntu: https://bugs.launchpad.net/ubuntu/+source/buzztard/+bug/803173
40+Bug-Debian: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=632956
41+Index: buzztard-0.5.0/docs/reference/bt-cmd/Makefile.in
42+===================================================================
43+--- buzztard-0.5.0.orig/docs/reference/bt-cmd/Makefile.in 2009-07-31 09:47:38.000000000 +0200
44++++ buzztard-0.5.0/docs/reference/bt-cmd/Makefile.in 2011-07-07 13:34:50.473977021 +0200
45+@@ -332,9 +332,9 @@
46+ $(BT_CFLAGS)
47+
48+ GTKDOC_LIBS = \
49++ $(top_builddir)/src/ui/cmd/libbtcmd.la \
50+ $(top_builddir)/src/lib/core/libbuzztard-core.la \
51+ $(top_builddir)/src/lib/ic/libbuzztard-ic.la \
52+- $(top_builddir)/src/ui/cmd/libbtcmd.la \
53+ $(BASE_DEPS_LIBS) \
54+ $(BT_LIBS)
55+
56+Index: buzztard-0.5.0/docs/reference/bt-edit/Makefile.in
57+===================================================================
58+--- buzztard-0.5.0.orig/docs/reference/bt-edit/Makefile.in 2009-07-31 09:47:38.000000000 +0200
59++++ buzztard-0.5.0/docs/reference/bt-edit/Makefile.in 2011-07-07 13:34:50.483977021 +0200
60+@@ -337,9 +337,9 @@
61+ $(BT_CFLAGS)
62+
63+ GTKDOC_LIBS = \
64++ $(top_builddir)/src/ui/edit/libbtedit.la \
65+ $(top_builddir)/src/lib/core/libbuzztard-core.la \
66+ $(top_builddir)/src/lib/ic/libbuzztard-ic.la \
67+- $(top_builddir)/src/ui/edit/libbtedit.la \
68+ $(BASE_DEPS_LIBS) \
69+ $(GUI_DEPS_LIBS) \
70+ $(BT_LIBS)
71+Index: buzztard-0.5.0/src/ui/edit/Makefile.in
72+===================================================================
73+--- buzztard-0.5.0.orig/src/ui/edit/Makefile.in 2009-07-31 09:47:41.000000000 +0200
74++++ buzztard-0.5.0/src/ui/edit/Makefile.in 2011-07-07 13:34:50.483977021 +0200
75+@@ -324,10 +324,11 @@
76+ top_srcdir = @top_srcdir@
77+ with_desktop_dir = @with_desktop_dir@
78+ xvfb_path = @xvfb_path@
79+-buzztard_edit_LDADD = $(BASE_DEPS_LIBS) $(GUI_DEPS_LIBS) $(BT_LIBS) $(LIBINTL) \
80++buzztard_edit_LDADD = \
81+ $(top_builddir)/src/lib/core/libbuzztard-core.la \
82+ $(top_builddir)/src/lib/ic/libbuzztard-ic.la \
83+- libbtedit.la
84++ libbtedit.la \
85++ $(BASE_DEPS_LIBS) $(GUI_DEPS_LIBS) $(BT_LIBS) $(LIBINTL)
86+
87+ buzztard_edit_SOURCES = \
88+ bt-edit.c bt-edit.h
89+Index: buzztard-0.5.0/src/ui/cmd/Makefile.in
90+===================================================================
91+--- buzztard-0.5.0.orig/src/ui/cmd/Makefile.in 2011-07-07 13:35:37.383977049 +0200
92++++ buzztard-0.5.0/src/ui/cmd/Makefile.in 2011-07-07 13:35:48.613977057 +0200
93+@@ -288,9 +288,10 @@
94+ top_srcdir = @top_srcdir@
95+ with_desktop_dir = @with_desktop_dir@
96+ xvfb_path = @xvfb_path@
97+-buzztard_cmd_LDADD = $(BASE_DEPS_LIBS) $(BT_LIBS) $(LIBINTL) \
98++buzztard_cmd_LDADD = \
99+ $(top_builddir)/src/lib/core/libbuzztard-core.la \
100+- libbtcmd.la
101++ libbtcmd.la \
102++ $(BASE_DEPS_LIBS) $(BT_LIBS) $(LIBINTL)
103+
104+ buzztard_cmd_SOURCES = \
105+ bt-cmd.c bt-cmd.h
106
107=== modified file 'debian/patches/series'
108--- debian/patches/series 2010-04-21 13:30:20 +0000
109+++ debian/patches/series 2011-08-21 14:22:24 +0000
110@@ -1,3 +1,4 @@
111 01_COPYING-LGPL.patch
112 02_var_args-copy.patch
113 03_omf-dir.patch
114+04_fix-as-needed-build.patch

Subscribers

People subscribed via source and target branches

to all changes: