Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 501098 - media-video/rtmpdump - add multilib support
Summary: media-video/rtmpdump - add multilib support
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Markos Chandras (RETIRED)
URL:
Whiteboard:
Keywords: PATCH
Depends on: 501104
Blocks: gx86-multilib 489816
  Show dependency tree
 
Reported: 2014-02-12 18:11 UTC by Michael Mair-Keimberger (iamnr3)
Modified: 2015-02-11 22:35 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
rtmpdump-2.4_p20131018.ebuild.patch (rtmpdump-2.4_p20131018.ebuild.patch,2.39 KB, patch)
2014-02-12 18:11 UTC, Michael Mair-Keimberger (iamnr3)
Details | Diff
Complete ebuild for media-video/rtmpdump-2.4_p20131018-r1 (rtmpdump-2.4_p20131018-r1.ebuild,2.18 KB, text/plain)
2014-06-04 12:38 UTC, Christian Schmidt
Details
Ebuild for git head (rtmpdump-9999.ebuild,2.06 KB, text/plain)
2014-06-04 12:38 UTC, Christian Schmidt
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Mair-Keimberger (iamnr3) 2014-02-12 18:11:16 UTC
Created attachment 370232 [details, diff]
rtmpdump-2.4_p20131018.ebuild.patch

This is a patch against rtmpdump-2.4_p20131018 to add multilib support. 
Since the source package won't create an folder i had to add following src_unpack() in order to get multilib_copy_sources to work.

src_unpack(){
	mkdir -p "${S}" || die
	cd "${S}" || die
	unpack ${A}
}

It doesn't look very nice, if anyone has a better idea - i'm open for suggestions :)
Comment 1 Christian Schmidt 2014-06-04 12:38:05 UTC
Created attachment 378226 [details]
Complete ebuild for media-video/rtmpdump-2.4_p20131018-r1
Comment 2 Christian Schmidt 2014-06-04 12:38:25 UTC
Created attachment 378228 [details]
Ebuild for git head
Comment 3 Ilya Gordeev 2014-06-30 11:00:31 UTC
Building media-video/rtmpdump-2.4_p20131018 I'v got:
QA: compile
QA warning: multilib_build_binaries is deprecated. Please use the equivalent
multilib_is_native_abi function instead.
Comment 4 David Watzke 2014-12-15 19:08:48 UTC
Hi. Seems to me, that this bug is already fixed as media-video/rtmpdump-2.4_p20131018 supports multilib.

I'd just suggest adding openssl[abi_x86_32] to deps if ssl flag is enabled, because without it the build fails.

Thanks guys!
Comment 5 Pacho Ramos gentoo-dev 2014-12-15 23:57:22 UTC
Indeed, looks like [${MULTILIB_USEDEP}] for gnutls, polarssl, and openssl deps would be needed
Comment 6 Michael Mair-Keimberger (iamnr3) 2015-02-10 20:08:29 UTC
I just made a pull request on github[1] for fixing the build failure. Please have a look :)

[1]https://github.com/gentoo/gentoo-portage-rsync-mirror/pull/13
Comment 7 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2015-02-11 22:35:43 UTC
+  11 Feb 2015; Michał Górny <mgorny@gentoo.org>
+  no-emul-linux-x86/package.use.stable.mask, package.use.stable.mask:
+  Mask abi_x86_32 on media-video/rtmpdump.

+  11 Feb 2015; Michał Górny <mgorny@gentoo.org> rtmpdump-2.4_p20131018.ebuild,
+  rtmpdump-9999.ebuild:
+  Fix multilib deps. Move zlib into "ssl?" since it is used only along with SSL.
+  Add missing ||die. Bug #501098, https://github.com/gentoo/gentoo-portage-
+  rsync-mirror/pull/13 by Michael Mair-Keimberger.