Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 452462 - ~media-tv/xbmc-11.0 not compatible with new >=libcdio-0.90
Summary: ~media-tv/xbmc-11.0 not compatible with new >=libcdio-0.90
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Xbox project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 452430 454552
  Show dependency tree
 
Reported: 2013-01-16 10:01 UTC by Samuli Suominen (RETIRED)
Modified: 2013-02-18 19:35 UTC (History)
2 users (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Samuli Suominen (RETIRED) gentoo-dev 2013-01-16 10:01:25 UTC
found this,

https://projects.archlinux.org/svntogit/community.git/tree/trunk/cdio.patch?h=packages/xbmc

and headers cdda.h and paranoia.h now lives under cdio/paranoia/ instead of directly in cdio/

please test and if paranoia.h or cdda.h is being used, the dep goes like:

|| ( dev-libs/libcdio-paranoia <dev-libs/libcdio-0.90[-minimal] )

but I really don't know. xbmc requires such dependencies it won't emerge on my devbox, but that's another story

thanks!
Comment 1 Ben Kohler gentoo-dev 2013-01-29 18:22:11 UTC
xbmc-12.0_rc3 builds fine for me even with no paranoia.h / cdda.h present at all.

# emerge -C libcdio-paranoia
<.. successful unmerge>
# qlist -ICv libcdio
dev-libs/libcdio-0.90
# updatedb
# locate cdda.h
# locate paranoia.h
# emerge =xbmc-12.0_rc3
<successful emerge>
#
Comment 2 Samuli Suominen (RETIRED) gentoo-dev 2013-01-29 18:29:59 UTC
The bundled FFmpeg in the _rc3 source code is at least broken:

$ grep -r cdda.h *
lib/ffmpeg/libavdevice/libcdio.c:#include <cdio/cdda.h>
lib/ffmpeg/configure:    check_lib2 "cdio/cdda.h cdio/paranoia.h" cdio_cddap_open "-lcdio_paranoia -lcdio_cdda -lcdio"
$ grep -r paranoia.h *
lib/ffmpeg/libavdevice/libcdio.c:#include <cdio/paranoia.h>
lib/ffmpeg/configure:    check_lib2 "cdio/cdda.h cdio/paranoia.h" cdio_cddap_open "-lcdio_paranoia -lcdio_cdda -lcdio"

But it's propably not used by the ebuild. So having plain 'dev-libs/libcdio' dependency is fine if the bundled FFmpeg is not used.
Comment 3 Ben Kohler gentoo-dev 2013-02-01 16:21:42 UTC
Current xbmc does not even have the option to use internal ffmpeg, so this can be closed, right?
Comment 4 Samuli Suominen (RETIRED) gentoo-dev 2013-02-01 16:28:29 UTC
(In reply to comment #3)
> Current xbmc does not even have the option to use internal ffmpeg, so this
> can be closed, right?

Still not convinced it works without seeing a build log (or a word from the maintainer) which implies libcdio is actually picked up and used.

But oh well
Comment 5 Ben Kohler gentoo-dev 2013-02-01 20:37:07 UTC
You are right after all, xbmc-11 does use cdda.h and requires the patch from archlinux to build against libcdio-0.90.  xbmc-12 is fine, though.
Comment 6 SpanKY gentoo-dev 2013-02-18 19:35:44 UTC
xbmc-12 works fine