Summary: | =media-sound/cmus-2.5.0 fails to compile with >=dev-libs/libcdio-paranoia-0.90 | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Samuli Suominen (RETIRED) <ssuominen> |
Component: | Current packages | Assignee: | Christian Faulhammer (RETIRED) <fauli> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | cornicx, sound |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
URL: | https://github.com/cmus/cmus/issues/14 | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 452430 | ||
Attachments: |
With this you get to the actual error
build.log after fixing cdda.h include location |
Created attachment 335778 [details]
build.log after fixing cdda.h include location
and the actual error is here
Reported upstream: https://github.com/cmus/cmus/issues/14 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 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.
not fixed, still fails to compile, reopening (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' *** Bug 453286 has been marked as a duplicate of this bug. *** Upstream patch in Portage: https://github.com/cmus/cmus/commit/70c727022dcfd802de0d621255ab1a366a50c434 Fixed in 2.5.0-r1 |
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.