Bug 452400 – =gnome-base/gvfs-1.14.2 is not compatible with >=dev-libs/libcdio-paranoia-0.90 (separated from libcdio now)

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Samuli Suominen (RETIRED) 2013-01-15 19:35:27 UTC]

The location of cdda.h and paranoia.h changed with version 0.90 in purpose

This line in this file...

daemon/gvfsbackendcdda.c:#include <cdio/paranoia.h>

...should now be:

daemon/gvfsbackendcdda.c:#include <cdio/paranoia/paranoia.h>

I've used similar hack in media-sound/pragha for now:

if has_version dev-libs/libcdio-paranoia; then
sed -i -e \
'/#include/s:cdio/paranoia.h:cdio/paranoia/paranoia.h:' \
daemon/gvfsbackendcdda.c || die
fi

I want to unmask the new libcdio in very fast pace and get it over with, I hope you understand. If nobody has reported this upstream by the time I'm done with rest of the packages, I'll report this upstream -- but I'd rather prefer if you could do it? Proper patch would be cool too.

[Comment 1 Samuli Suominen (RETIRED) 2013-01-15 19:39:36 UTC]

And make the dependency say:

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

[Comment 2 Samuli Suominen (RETIRED) 2013-01-16 05:12:27 UTC]

workaround for tree, keeping open for upstream reporting

[Comment 3 Samuli Suominen (RETIRED) 2013-01-16 11:01:10 UTC]

This is still broken at upstream git master,

http://git.gnome.org/browse/gvfs/tree/daemon/gvfsbackendcdda.c

around lines 58-59

[Comment 4 Samuli Suominen (RETIRED) 2013-01-16 11:07:37 UTC]

https://bugzilla.gnome.org/show_bug.cgi?id=691841

[Comment 5 Samuli Suominen (RETIRED) 2013-01-19 20:34:14 UTC]

There is a proper patch in the upstream bug tracking system now:

http://bugzilla-attachments.gnome.org/attachment.cgi?id=233609

But since it involves running eautoreconf, I think we are good with the workaround in the ebuild now, since this will autoresolve in later releases