Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 421683 - media-video/raspberrypi-omxplayer (new package) Raspberry Pi accelerated video player
Summary: media-video/raspberrypi-omxplayer (new package) Raspberry Pi accelerated vide...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal enhancement with 3 votes (vote)
Assignee: Miroslav Šulc
URL: https://github.com/huceke/omxplayer
Whiteboard:
Keywords:
: 442832 442834 (view as bug list)
Depends on: 421681
Blocks:
  Show dependency tree
 
Reported: 2012-06-18 02:48 UTC by Michael Weber (RETIRED)
Modified: 2020-12-21 18:24 UTC (History)
14 users (show)

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


Attachments
patch fail log (omxplayer-9999-wrapper.patch.out) (omxplayer-9999-wrapper.patch.out,2.63 KB, text/plain)
2012-07-07 12:18 UTC, Raimonds Cicans
Details
omxplayer-9999-Makefile.patch (file_421683.txt,2.01 KB, text/plain)
2012-11-12 13:01 UTC, Adam Marthaler
Details
omxplayer-9999-wrapper.patch (file_421683.txt,1.23 KB, text/plain)
2012-11-12 13:02 UTC, Adam Marthaler
Details
omxplayer-9999.ebuild (file_421683.txt,833 bytes, text/plain)
2012-11-12 13:04 UTC, Adam Marthaler
Details
omxplayer-9999.ebuild (file_421683.txt,898 bytes, text/plain)
2012-11-16 11:03 UTC, Adam Marthaler
Details
omxplayer-9999-Makefile.patch (file_421683.txt,2.04 KB, text/plain)
2012-11-16 11:07 UTC, Adam Marthaler
Details
Added freetype library to LIBS in makefile - nper c.encv@gmail.com (omxplayer-9999-Makefile.patch.txt,2.05 KB, patch)
2012-12-24 15:59 UTC, Adam Marthaler
Details | Diff
omxplayer-9999-Makefile.patch (omxplayer-9999-Makefile.patch.txt,2.05 KB, text/plain)
2012-12-24 16:02 UTC, Adam Marthaler
Details
omxplayer-9999.ebuild (file_421683.txt,1.27 KB, text/plain)
2012-12-24 23:01 UTC, Adam Marthaler
Details
omxplayer-9999-Makefile.patch (omxplayer-9999-Makefile.patch,2.23 KB, patch)
2013-04-10 22:41 UTC, Tina
Details | Diff
omxplayer-9999-wrapper.patch (omxplayer-9999-wrapper.patch,1.26 KB, patch)
2013-04-10 22:42 UTC, Tina
Details | Diff
Makefile patch eliminating libavutil and libswresample build errors (omxplayer-9999-Makefile.patch,2.21 KB, patch)
2013-08-21 07:07 UTC, Mathias Vaterlaus
Details | Diff
The build.log of the failed build (build.log,26.85 KB, text/plain)
2014-05-05 11:47 UTC, Tobias Leupold
Details
The emerge --info of the failed build (emerge_info.txt,4.39 KB, text/plain)
2014-05-05 11:47 UTC, Tobias Leupold
Details
Depend on boost and freetype2, added X use flag (raspberrypi-omxplayer-X-boost-freetype2.patch,692 bytes, patch)
2016-05-31 08:18 UTC, Ahmet Inan
Details | Diff
Fix Makefile patching for raspberrypi-omxplayer-0_p20160528 (Makefile-0_p20160528.patch,1.54 KB, patch)
2016-06-02 09:11 UTC, Ahmet Inan
Details | Diff
Fix ebuild raspberrypi-omxplayer-0_p20160528 (raspberrypi-omxplayer-0_p20160528.patch,771 bytes, patch)
2016-06-02 09:19 UTC, Ahmet Inan
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Weber (RETIRED) gentoo-dev 2012-06-18 02:48:26 UTC
This is the omxplayer on top of the VideoCoreIV GPU firmware for Raspberry Pi.

It's located in my overlay (layman -a xmw)

   Michael
Comment 1 Raimonds Cicans 2012-07-07 12:18:51 UTC
Created attachment 317476 [details]
patch fail log (omxplayer-9999-wrapper.patch.out)

Ebuild fails to apply patch: omxplayer-9999-wrapper.patch
Comment 2 Adam Marthaler 2012-11-12 00:27:08 UTC
I've submitted a bug report with revised Makefile and wrapper patches:  https://bugs.gentoo.org/show_bug.cgi?id=442832
Comment 3 Jeroen Roovers (RETIRED) gentoo-dev 2012-11-12 12:35:14 UTC
*** Bug 442832 has been marked as a duplicate of this bug. ***
Comment 4 Jeroen Roovers (RETIRED) gentoo-dev 2012-11-12 12:35:22 UTC
*** Bug 442834 has been marked as a duplicate of this bug. ***
Comment 5 Jeroen Roovers (RETIRED) gentoo-dev 2012-11-12 12:36:01 UTC
Please attach all your patches/comment on improvements here rather than opening new bug reports for packages we do not support (yet).
Comment 6 Adam Marthaler 2012-11-12 13:01:53 UTC
Created attachment 329328 [details]
omxplayer-9999-Makefile.patch

Revised Makefile patch
Comment 7 Adam Marthaler 2012-11-12 13:02:55 UTC
Created attachment 329330 [details]
omxplayer-9999-wrapper.patch

Revised wrapper patch
Comment 8 Adam Marthaler 2012-11-12 13:04:57 UTC
Created attachment 329332 [details]
omxplayer-9999.ebuild

Looks like the package also required dev-libs/boost

I've added the missing required package.
Comment 9 Adam Marthaler 2012-11-12 13:07:41 UTC
There's still a missing library in OMXAudio.cpp, I've posted the output here:  http://pastebin.com/GAgycfck
Comment 10 Adam Marthaler 2012-11-12 13:10:18 UTC
(In reply to comment #9)
> There's still a missing library in OMXAudio.cpp, I've posted the output
> here:  http://pastebin.com/GAgycfck

Sorry, the above link will expire in a month, I've reposted the output in a permanent pastebin.0
http://pastebin.com/jWruyBTQ
Comment 11 Adam Marthaler 2012-11-16 11:03:33 UTC
Created attachment 329644 [details]
omxplayer-9999.ebuild

Ok, list of things this revised ebuild fixes:

 - Add requirement for gcc-4.6 or higher - according to the developer, this packages requires it: https://github.com/huceke/omxplayer/issues/71
 - Added requirement for dev-libs/boost
 - Added requirement for media-libs/freetype
 - Changed dodoc line from README to README.md
 - And finally, added information lines about needing to upgrade the Raspberry Pi firmware if ebuild fails with a specific error.
Comment 12 Adam Marthaler 2012-11-16 11:07:31 UTC
Created attachment 329646 [details]
omxplayer-9999-Makefile.patch

Issues this Makefile patch fixes:

 - Added missing -std=c++0x flag
 - Added missing include for freetype2 library
Comment 13 Adam Marthaler 2012-11-16 11:09:19 UTC
The two above attachments should fix the existing issues.

I was able to successfully compile on my local Raspberry Pi.
Comment 14 Adam Marthaler 2012-11-16 11:10:05 UTC
(In reply to comment #13)
> The two above attachments should fix the existing issues.
> 
> I was able to successfully compile on my local Raspberry Pi.

Sorry, I meant all 3 attachments should fix all existing issues.
Comment 15 Nikolay 2012-12-24 02:05:18 UTC
I was still experiencing troubles with current files, but following resolved issue for me:

Adding freetype2 to the end of +LIBS string in omxplayer-9999-Makefile.patch,

Here is entire +LIBS string which worked for me:

+LIBS=-L/opt/vc/lib -lrt -lbcm_host -lopenmaxil -lvcos -lvchiq_arm -lGLESv2 -lkhrn_static $(shell pkg-config --libs libavcodec libavfilter libavformat libpcre freetype2)
Comment 16 John W. Newman 2012-12-24 03:15:42 UTC
For some reason, I could not get gcc-4.6 ~arm to get through the linking phase.  One of the fortran libraries kept failing.  I had USE="-fortran" .. which is probably ok for the PI anyway (globally even).  (distcc etc all disabled here).

After suffering through that, about 20 hours to fail, twice, and 30 hours to build successfully, I checked out the code and it looks like the only thing it needs gcc-4.6 for is to add the support for subtitles.  There's an error handling file that does something funny with extending the exception classes and gcc-4.5 doesn't have the same signatures.  And that piece is only used by the two files for subtitles.

So if you don't care about subtitles and are impatient, it would far faster to just hack that out of the project and continue to use gcc-4.5 stable.
Comment 17 Adam Marthaler 2012-12-24 15:59:34 UTC
Created attachment 333212 [details, diff]
Added freetype library to LIBS in makefile - nper c.encv@gmail.com

Per nc.encv@gmail.com the freetype2 library needed to be added to the LIBS line in the makefile.  Couldn't hurt, so I did it anyway.
Comment 18 Adam Marthaler 2012-12-24 16:02:24 UTC
Created attachment 333214 [details]
omxplayer-9999-Makefile.patch

Attachment name was ugly... reuploaded.  ;)
Comment 19 Adam Marthaler 2012-12-24 16:10:49 UTC
(In reply to comment #16)
> For some reason, I could not get gcc-4.6 ~arm to get through the linking
> phase.  One of the fortran libraries kept failing.  I had USE="-fortran" ..
> which is probably ok for the PI anyway (globally even).  (distcc etc all
> disabled here).
> 
> After suffering through that, about 20 hours to fail, twice, and 30 hours to
> build successfully, I checked out the code and it looks like the only thing
> it needs gcc-4.6 for is to add the support for subtitles.  There's an error
> handling file that does something funny with extending the exception classes
> and gcc-4.5 doesn't have the same signatures.  And that piece is only used
> by the two files for subtitles.
> 
> So if you don't care about subtitles and are impatient, it would far faster
> to just hack that out of the project and continue to use gcc-4.5 stable.

Subtitles are necessary for me (actually, near paramount since I watch anime with it).  Can you post your use flags so I can compare, maybe something needs to be added to the e-build use flag requirements.
Comment 20 Adam Marthaler 2012-12-24 22:52:10 UTC
(In reply to comment #16)
> For some reason, I could not get gcc-4.6 ~arm to get through the linking
> phase.  One of the fortran libraries kept failing.  I had USE="-fortran" ..
> which is probably ok for the PI anyway (globally even).  (distcc etc all
> disabled here).
> 
> After suffering through that, about 20 hours to fail, twice, and 30 hours to
> build successfully, I checked out the code and it looks like the only thing
> it needs gcc-4.6 for is to add the support for subtitles.  There's an error
> handling file that does something funny with extending the exception classes
> and gcc-4.5 doesn't have the same signatures.  And that piece is only used
> by the two files for subtitles.
> 
> So if you don't care about subtitles and are impatient, it would far faster
> to just hack that out of the project and continue to use gcc-4.5 stable.

I just had a chance to check my Pi's USE flags, and I have the fortran flag turned on.  When you can, post your use flags.
Comment 21 Adam Marthaler 2012-12-24 23:01:03 UTC
Created attachment 333230 [details]
omxplayer-9999.ebuild

Apparently the warning message about upgrading firmware was *NOT* in e-build, this has been now attached.
Comment 22 Dale Hamel 2012-12-31 08:11:28 UTC
I tried stripping out the subtitle code, but it didn't work for my application (just got errors when trying to open stream from plex when using project pyplex from github). Not sure if this is because of the subtitles, but I figure it probably is. OR I did something stupid with my hackjob to pull the subtitle code out.

Tried just using gcc 4.6.3

But updating to gcc 4.6 adds additional problems.

It pulls in glibc 2.16, which violates the latest stable version of boost. I could not get newer versions of boost from portage tree to build. 

Building all of this takes way too long on rpi, so I used crossdev toolchain pointing to sshfs root of rpi with SYSROOT. Built and installed gcc 4.6.3 successfully, but cannot run it because it is missing libiconv. Working on fixing that now. 

As for boost, I copied dead ebuilds for boost 1.50. Built and merged as described above w/ crossdev with no problems.

glibc is regenerating locales right now... after that I'll merge the binary package for libiconv and try rebuilding omx. 

If I get it to work I'll update the ebuild.

Thanks for your work everyone, stumbling across this page was like striking oil!
Comment 23 Dale Hamel 2013-01-01 23:00:18 UTC
Successfully compiled and ran it to playback a stream generated from Plex Media Server. 

The key was using the built-in ffmpeg instead of using Gentoo's. I'll be submitted a modified ebuild and patches for this if anyone is interested.
Comment 24 Jakub Daniel 2013-01-03 09:50:21 UTC
I guess everyone is interested, please commit anything that may help others :)
Comment 25 Dale Hamel 2013-01-03 19:34:12 UTC
Hey, so I have it building and streaming works as I mentioned. I am in the process of making the ebuilds.

Since I couldn't get the gentoo ffmpeg to compile I made my own ffmpeg ebuild to use the one from the omxplayer. It's working well: https://github.com/dalehamel/PlexOverlay/blob/master/media-video/omxplayer-ffmpeg/omxplayer-ffmpeg-9999.ebuild

It is a dependency of https://github.com/dalehamel/PlexOverlay/blob/master/media-video/omxplayer/omxplayer-9999.ebuild, which just needs to be modified to link to it (building is slow on the RPi).

Keep an eye on https://github.com/dalehamel/PlexOverlay, I should have cleaned up and ready to go by tonight (at work right now).
Comment 26 Dale Hamel 2013-01-04 15:19:18 UTC
As of this morning I have it building. Just adding it to a PORTDIR_OVERLAY and emerging omxplayer will automatically pull in omxplayer-ffmpeg. 

Omxplayer has been recently updated, but I have frozen support in the ebuild to the Dec 4th revision. 

I'll be looking into what needs to be done to get it to work with the latest upstream. 

Otherwise, enjoy the repo!
Comment 27 Adam Marthaler 2013-01-07 20:58:02 UTC
There's a build of omxplayer that supports built-in subtitles.  Can we merge that capability in with your build Dale?

https://github.com/huceke/omxplayer/issues/59#issuecomment-11174254
Comment 28 Tina 2013-04-10 22:39:59 UTC
to build this , one need gcc-4.7 now
also the patches dont apply anymore i tried to 'fix' them , they apply now but it wont build:
n file included from /usr/include/sys/stat.h:106:0,
                 from linux/PlatformDefs.h:37,
                 from ./system.h:227,
                 from utils/log.cpp:22:
/usr/include/bits/stat.h:72:21: error: field 'st_atim' has incomplete type
/usr/include/bits/stat.h:73:21: error: field 'st_mtim' has incomplete type
/usr/include/bits/stat.h:74:21: error: field 'st_ctim' has incomplete type
/usr/include/bits/stat.h:118:21: error: field 'st_atim' has incomplete type
/usr/include/bits/stat.h:119:21: error: field 'st_mtim' has incomplete type
/usr/include/bits/stat.h:120:21: error: field 'st_ctim' has incomplete type
In file included from ./system.h:227:0,
                 from utils/log.cpp:22:
linux/PlatformDefs.h:380:3: error: 'time_t' does not name a type
linux/PlatformDefs.h:381:3: error: 'time_t' does not name a type
linux/PlatformDefs.h:382:3: error: 'time_t' does not name a type
In file included from /usr/lib/gcc/armv6j-hardfloat-linux-gnueabi/4.7.2/include/g++-v4/bits/locale_facets_nonio.h:39:0,
                 from /usr/lib/gcc/armv6j-hardfloat-linux-gnueabi/4.7.2/include/g++-v4/locale:43,
                 from utils/StdString.h:670,
                 from DynamicDll.h:24,
                 from DynamicDll.cpp:22:
/usr/lib/gcc/armv6j-hardfloat-linux-gnueabi/4.7.2/include/g++-v4/ctime:62:11: error: '::clock_t' has not been declared
/usr/lib/gcc/armv6j-hardfloat-linux-gnueabi/4.7.2/include/g++-v4/ctime:63:11: error: '::time_t' has not been declared
/usr/lib/gcc/armv6j-hardfloat-linux-gnueabi/4.7.2/include/g++-v4/ctime:66:11: error: '::clock' has not been declared
/usr/lib/gcc/armv6j-hardfloat-linux-gnueabi/4.7.2/include/g++-v4/ctime:67:11: error: '::difftime' has not been declared
/usr/lib/gcc/armv6j-hardfloat-linux-gnueabi/4.7.2/include/g++-v4/ctime:68:11: error: '::mktime' has not been declared
/usr/lib/gcc/armv6j-hardfloat-linux-gnueabi/4.7.2/include/g++-v4/ctime:69:11: error: '::time' has not been declared
/usr/lib/gcc/armv6j-hardfloat-linux-gnueabi/4.7.2/include/g++-v4/ctime:70:11: error: '::asctime' has not been declared
/usr/lib/gcc/armv6j-hardfloat-linux-gnueabi/4.7.2/include/g++-v4/ctime:71:11: error: '::ctime' has not been declared
/usr/lib/gcc/armv6j-hardfloat-linux-gnueabi/4.7.2/include/g++-v4/ctime:72:11: error: '::gmtime' has not been declared
/usr/lib/gcc/armv6j-hardfloat-linux-gnueabi/4.7.2/include/g++-v4/ctime:73:11: error: '::localtime' has not been declared
/usr/lib/gcc/armv6j-hardfloat-linux-gnueabi/4.7.2/include/g++-v4/ctime:74:11: error: '::strftime' has not been declared


I did some searching and think this is due ffmpeg-1.x  including a modified time.h causing these conflicts. When building with <ffmpeg-1.x , there is an other error
Comment 29 Tina 2013-04-10 22:41:57 UTC
Created attachment 345156 [details, diff]
omxplayer-9999-Makefile.patch

omxplayer-9999-Makefile.patch
Comment 30 Tina 2013-04-10 22:42:42 UTC
Created attachment 345158 [details, diff]
omxplayer-9999-wrapper.patch
Comment 31 Tom Wijsman (TomWij) (RETIRED) gentoo-dev 2013-04-13 12:29:20 UTC
Experiencing `OMXReader.cpp:499:49: error: 'AV_CODEC_ID_SUBRIP' was not declared in this scope` here with =media-video/ffmpeg-0.10.6.

If we look at https://github.com/huceke/omxplayer/blob/master/Makefile.ffmpeg#L64 we can see that commit 8c51ea54897c2d8671b38efecc1422ad4ad344f9 is used for ffmpeg. That commit seems to have been made on September 20, 2013; this is a commit some months before 0.10.6 has been released and some days before 1.0 has been released. Therefore the version used is likely not in our Portage tree.

The only relevant bug I can find is people wanting to see ffmpeg as a sub module (https://github.com/huceke/omxplayer/pull/113), therefore I have filed another bug to request the code to be updated to work with newer versions (https://github.com/huceke/omxplayer/issues/155).

The only way to get it to work currently is to bundle ffmpeg unless someone feels like patching the code (please update above issue 155 in that case), but given the limited embedded environment it's a bit nasty to keep around multiple versions; so, one may opt to pull in that specific commit for media-video/ffmpeg.
Comment 32 Tina 2013-04-22 16:45:50 UTC
i havent had the time yet to test but https://github.com/huceke/omxplayer/issues/150 says it nows support ffmpeg1.2 , maybe one could try to build against that, i might try myself later this week ...
Comment 33 Mathias Vaterlaus 2013-08-21 07:03:20 UTC
Since I had 2 more problems which I fixed in the omxplayer-9999.Makefile.patch. You can read the details on https://forums.gentoo.org/viewtopic-t-967632.html. I will supply the patch in the next comment.
Comment 34 Mathias Vaterlaus 2013-08-21 07:07:38 UTC
Created attachment 356570 [details, diff]
Makefile patch eliminating libavutil and libswresample build errors

This is basically the patch supplied by the xmw overlay with fixes for libavutil and libswresample.
Comment 35 Tobias Leupold 2014-05-05 11:46:46 UTC
I just tried to build omxplayer on my Raspberry Pi. It fails with a lot of suspect errors like:

In file included from /usr/include/sys/stat.h:106:0,
                 from linux/PlatformDefs.h:37,
                 from ./system.h:227,
                 from utils/log.cpp:22:
/usr/include/bits/stat.h:72:21: error: field ‘st_atim’ has incomplete type
...
In file included from ./system.h:227:0,
                 from utils/log.cpp:22:
linux/PlatformDefs.h:380:3: error: ‘time_t’ does not name a type
...
In file included from /usr/lib/gcc/armv6j-hardfloat-linux-gnueabi/4.7.3/include/g++-v4/bits/locale_facets_nonio.h:39:0,
                 from /usr/lib/gcc/armv6j-hardfloat-linux-gnueabi/4.7.3/include/g++-v4/locale:43,
                 from ./utils/StdString.h:670,
                 from utils/log.cpp:26:
/usr/lib/gcc/armv6j-hardfloat-linux-gnueabi/4.7.3/include/g++-v4/ctime:62:11: error: ‘::clock_t’ has not been declared

(see the attached build.log and emerge --info).

Additionally, https://github.com/huceke/omxplayer says: "This tree is no longer under active delevopment. Updates will occur here: https://github.com/popcornmix/omxplayer". Perhaps the ebuild should be updated to use the new code location?
Comment 36 Tobias Leupold 2014-05-05 11:47:21 UTC
Created attachment 376422 [details]
The build.log of the failed build
Comment 37 Tobias Leupold 2014-05-05 11:47:45 UTC
Created attachment 376424 [details]
The emerge --info of the failed build
Comment 38 Michael Weber (RETIRED) gentoo-dev 2015-04-24 01:10:07 UTC
I've bumped the some ebuilds in my dev overlay (layman -s xmw), current state

# emerge -pv raspberrypi-{firmware,userland-bin} omxplayer
[ebuild   R   #] media-libs/raspberrypi-userland-bin-1_pre20150421::xmw  USE="hardfp -examples" 0 KiB
[ebuild   R   #] sys-boot/raspberrypi-firmware-1_pre20150421::xmw  0 KiB
[ebuild   R   ~] media-video/omxplayer-0_p20150421::xmw  0 KiB
Comment 39 Michael Weber (RETIRED) gentoo-dev 2015-04-24 02:14:21 UTC
maybe X? ( x11-apps/xrefresh )
Comment 40 Miroslav Šulc gentoo-dev 2016-03-08 21:12:44 UTC
as this package does not have a maintainer, i'll take it as i need it myself. i improved the ebuild and put it in the main tree:

commit 5e1dbb3f8a38fb9a91420a0f8da75efd2172a07d
Author: Miroslav Šulc <fordfrog@gentoo.org>
Date:   Tue Mar 8 22:08:32 2016 +0100

    media-video/omxplayer: adjusted and improved ebuild from bug #421683
    
    Package-Manager: portage-2.2.27


i'll leave this bug open for a while so you can report back any issues.
Comment 41 Miroslav Šulc gentoo-dev 2016-03-08 21:46:31 UTC
as suggested by steev at #gentoo-dev, renamed omxplayer to raspberrypi-omxplayer
Comment 42 Ahmet Inan 2016-05-31 08:18:52 UTC
Created attachment 435900 [details, diff]
Depend on boost and freetype2, added X use flag

With a bare bones, frame buffer-only build without X, boost and freetype2 will be missing.
Also remove dependency on xrefresh and xset if X is disabled via use flag.
Comment 43 Miroslav Šulc gentoo-dev 2016-05-31 12:35:33 UTC
(In reply to Ahmet Inan from comment #42)
> Created attachment 435900 [details, diff] [details, diff]
> Depend on boost and freetype2, added X use flag
> 
> With a bare bones, frame buffer-only build without X, boost and freetype2
> will be missing.
> Also remove dependency on xrefresh and xset if X is disabled via use flag.

thanks for the patch.

commit 7a9f821787a21db585ae1bee8c5f1facda2d3254
Author: Miroslav Šulc <fordfrog@gentoo.org>
Date:   Tue May 31 14:34:08 2016 +0200

    media-video/raspberrypi-omxplayer: applied patch by Ahmet Inan <xdsopl@googlemail.com> from bug #421683 and bumped to the latest sources
    
    Package-Manager: portage-2.3.0_rc1
Comment 44 Ahmet Inan 2016-06-02 09:11:03 UTC
Created attachment 436156 [details, diff]
Fix Makefile patching for raspberrypi-omxplayer-0_p20160528

They have added vchostif to the linker list so the previous Makefile patch will fail. Copied and hand-edited the patch to include " -lvchostif".
Comment 45 Ahmet Inan 2016-06-02 09:19:08 UTC
Created attachment 436158 [details, diff]
Fix ebuild raspberrypi-omxplayer-0_p20160528

They have added " -lvchostif" to the linker list but screwed with the order.
The previous Makefile patch takes care of removing the linker list and this ebuild creates the Makefile.include with the correct order, for static and dynamic vchostif lib. Also see https://github.com/popcornmix/omxplayer:

commit 7ad3761e9bf33a442e63e8e3d8a2429c94329294
Makefile: Explicitly link with vchostif library
Comment 46 Miroslav Šulc gentoo-dev 2016-06-02 09:32:27 UTC
(In reply to Ahmet Inan from comment #45)
> Created attachment 436158 [details, diff] [details, diff]
> Fix ebuild raspberrypi-omxplayer-0_p20160528
> 
> They have added " -lvchostif" to the linker list but screwed with the order.
> The previous Makefile patch takes care of removing the linker list and this
> ebuild creates the Makefile.include with the correct order, for static and
> dynamic vchostif lib. Also see https://github.com/popcornmix/omxplayer:
> 
> commit 7ad3761e9bf33a442e63e8e3d8a2429c94329294
> Makefile: Explicitly link with vchostif library

thanks for the fix.

commit 53eace1f6281067889b6200682253e5c43941a79
Author: Miroslav Šulc <fordfrog@gentoo.org>
Date:   Thu Jun 2 11:30:17 2016 +0200

    media-video/raspberrypi-omxplayer: applied makefile fix from Ahmet Inan <xdsopl@googlemail.com> from bug #421683 comment #44 and #45
    
    Package-Manager: portage-2.3.0_rc1