Bug 452446 – =media-sound/cmus-2.5.0 fails to compile with >=dev-libs/libcdio-paranoia-0.90

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Samuli Suominen (RETIRED) 2013-01-16 08:04:42 UTC]

Created attachment 335776 [details, diff]
With this you get to the actual error

my_config says cdio, but IUSE and dep says cdda. it should be IUSE=cdio. so it's currently not even possible to enable it.

futhermore it fails to compile with dev-libs/libcdio-paranoia in many ways.

[Comment 1 Samuli Suominen (RETIRED) 2013-01-16 08:05:54 UTC]

Created attachment 335778 [details]
build.log after fixing cdda.h include location

and the actual error is here

[Comment 2 Samuli Suominen (RETIRED) 2013-01-16 19:04:50 UTC]

Reported upstream:

https://github.com/cmus/cmus/issues/14

[Comment 3 Samuli Suominen (RETIRED) 2013-01-19 22:29:19 UTC]

I've changed my_config to use 'cdda' instead of 'cdio' since that's what was in deps and IUSE.
Unsure which one the maintainer wanted here. If you want it to be "cdio", then feel free to change it again.

The failure still remains however, but now it's at least possible to hit the error :-)

[Comment 4 Samuli Suominen (RETIRED) 2013-01-19 22:32:35 UTC]

Comment on attachment 335776 [details, diff]
With this you get to the actual error

With this in src_prepare() you get to the actual error:

if has_version dev-libs/libcdio-paranoia; then
	sed -i -e 's:cdio/cdda.h:cdio/paranoia/cdda.h:' cdio.c || die
fi

This attachment had some crap that I've pushed in tree already.

[Comment 5 Samuli Suominen (RETIRED) 2013-01-21 09:57:58 UTC]

not fixed, still fails to compile, reopening

[Comment 6 Samuli Suominen (RETIRED) 2013-01-21 09:58:30 UTC]

(In reply to comment #5)
> not fixed, still fails to compile, reopening

why not read the attachment... or the bug report, or even the upstream link I gave?

cdio.c: In function 'libcdio_read_comments':
cdio.c:362:2: error: too many arguments to function 'cdio_get_cdtext'
In file included from /usr/include/cdio/cdio.h:61:0,
                 from /usr/include/cdio/paranoia/cdda.h:18,
                 from cdio.c:30:
/usr/include/cdio/disc.h:77:13: note: declared here
cdio.c:364:37: error: dereferencing pointer to incomplete type
cdio.c:365:18: error: 'CDTEXT_PERFORMER' undeclared (first use in this function)
cdio.c:365:18: note: each undeclared identifier is reported only once for each function it appears in
cdio.c:366:17: error: 'CDTEXT_TITLE' undeclared (first use in this function)
cdio.c:367:17: error: 'CDTEXT_GENRE' undeclared (first use in this function)
cdio.c:368:19: error: 'CDTEXT_MESSAGE' undeclared (first use in this function)
cdio.c:364:17: warning: variable 'field' set but not used [-Wunused-but-set-variable]
cdio.c:370:2: error: too many arguments to function 'cdio_get_cdtext'

[Comment 7 Samuli Suominen (RETIRED) 2013-01-21 13:28:59 UTC]

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

[Comment 8 Samuli Suominen (RETIRED) 2013-04-21 07:40:51 UTC]

Upstream patch in Portage:

https://github.com/cmus/cmus/commit/70c727022dcfd802de0d621255ab1a366a50c434

Fixed in 2.5.0-r1