Bug 413987

Summary:	media-tv/xawtv with --as-needed drops libm link
Product:	Gentoo Linux	Reporter:	Milos Ivanovic <bugs>
Component:	New packages	Assignee:	No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it <maintainer-needed>
Status:	RESOLVED WONTFIX
Severity:	normal	CC:	dark.shadow, jamesbroadhead, media-tv, peach, sachse
Priority:	Normal	Keywords:	PATCH, PMASKED
Version:	unspecified
Hardware:	All
OS:	Linux
See Also:	https://launchpad.net/bugs/821916
Whiteboard:
Package list:		Runtime testing required:	---
Attachments:	xawtv-3.95-as-needed.patch xawtv-3.95-r2.ebuild.diff ebuild for latest version xawtv-3.103.ebuild

Description Milos Ivanovic 2012-04-29 14:18:06 UTC

"ld --as-needed used to compile it and its dependencies dropped the libm link which these plugins directly needed but only linked indirectly.
The fix is to simply link them explicitly. Additionally conv-mjpeg is missing a link against libjpeg."

Above excerpt taken from https://bugs.launchpad.net/ubuntu/+source/xawtv/+bug/821916

--------------------------------------------------------------------------------

--- /usr/portage/media-tv/xawtv/xawtv-3.95-r2.ebuild
+++ /usr/local/portage/media-tv/xawtv/xawtv-3.95-r2.ebuild
@@ -84,6 +84,8 @@
 	# It tries to include FSlib.h directly, but this seems to have moved.
 	use X && has_version x11-libs/libFS && append-flags -I/usr/include/X11/fonts
 
+	append-ldflags $(no-as-needed)
+
 	econf \
 		$(use_with X x) \
 		$(use_enable xext xfree-ext) \

--------------------------------------------------------------------------------

Without the above additions (which are only temporary), we get this while trying to use the streamer binary:
dlopen: /usr/lib64/xawtv/flt-disor.so: undefined symbol: tan
dlopen: /usr/lib64/xawtv/flt-gamma.so: undefined symbol: pow

Please link the needed libraries explicitly in the build process in order to fix this efficiently.

Comment 1 Julian Ospald 2012-04-29 17:56:11 UTC

Created attachment 310431 [details, diff]
xawtv-3.95-as-needed.patch

origin: http://bazaar.launchpad.net/~jtaylor/ubuntu/oneiric/xawtv/fix-821916/revision/25/debian/patches/104_fix_plugin_links.dpatch

Comment 2 Julian Ospald 2012-04-29 17:57:34 UTC

Created attachment 310433 [details]
xawtv-3.95-r2.ebuild.diff

Comment 3 Samuli Suominen (RETIRED) gentoo-dev

2013-05-03 01:19:19 UTC

*** Bug 436708 has been marked as a duplicate of this bug. ***

Comment 4 Matteo 'The Peach' Pescarin 2013-05-04 10:21:52 UTC

As I wrote in bug #436708:

>> This bug was fixed upstream. We should probably pull a new version into
>> portage.

> JWZ already reported it a while ago:
> https://bugs.gentoo.org/show_bug.cgi?id=456100

Comment 5 James Broadhead 2014-01-12 20:41:30 UTC

Created attachment 367744 [details]
ebuild for latest version

As a starting-off point, here's an ebuild for the latest xawtv on the 3x branch. 

This drops the patches for the previous version. Interested parties should check that that is warranted before this can be merged into the main tree.

Comment 6 James Broadhead 2014-01-12 20:42:41 UTC

Created attachment 367746 [details]
xawtv-3.103.ebuild

Comment 7 jospezial 2014-10-09 17:31:39 UTC

You could make this bug depend on https://bugs.gentoo.org/show_bug.cgi?id=358013
(media-tv/xawtv-3.103 version bump)
There is an EAPI=5 ebuild.