Bug 257607

Summary:	media-sound/cdparanoia: cdda_paranoia.h: misses cdda_interface.h include (causes media-sound/grip: header present but cannot be compiled)
Product:	Gentoo Linux	Reporter:	Diego Elio Pettenò (RETIRED) <flameeyes>
Component:	New packages	Assignee:	Gentoo Sound Team <sound>
Status:	RESOLVED FIXED
Severity:	normal	CC:	vapier
Priority:	High
Version:	unspecified
Hardware:	All
OS:	Linux
URL:	http://blog.flameeyes.eu/2009/02/04/brace-for-impact
Whiteboard:
Package list:		Runtime testing required:	---
Bug Depends on:
Bug Blocks:	257596

Description Diego Elio Pettenò (RETIRED) gentoo-dev

2009-02-04 14:04:51 UTC

Hi,

you're getting this template-based bug because the package in the subject hit a "header present but cannot be compiled" during ./configure execution. If you check my blog post in the URL, this is going to be a problem when autoconf-2.64 will be used so you should pay attention and consider fixing the issue (or let upstream know about it at least).

Thank you for your consideration. Share and enjoy!

media-sound/grip-3.3.1-r1:20081120-221521.log:configure: WARNING: cdda_paranoia.h: present but cannot be compiled

Comment 1 SpanKY gentoo-dev

2009-03-08 20:49:54 UTC

this is a bug in media-sound/cdparanoia.  it uses "struct cdrom_drive" in cdda_paranoia.h but does not do a dummy struct define or include cdda_interface.h for it.

so we end up with:
$ echo '#include <cdda_paranoia.h>' | gcc -x c -c - -o /dev/null
In file included from <stdin>:1:
/usr/include/cdda_paranoia.h:44: error: expected ‘)’ before ‘*’ token
/usr/include/cdda_paranoia.h:47: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘*’ token
/usr/include/cdda_paranoia.h:48: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘*’ token

Comment 2 Samuli Suominen (RETIRED) gentoo-dev

2009-04-30 08:35:02 UTC

We used to do this for previous releases but somehow the patch was dropped when it was bumped to latest.

*cdparanoia-3.10.2-r3 (30 Apr 2009)

  30 Apr 2009; Samuli Suominen <ssuominen@gentoo.org>
  +files/cdparanoia-3.10.2-include-cdda_interface_h.patch,
  -cdparanoia-3.10.2-r2.ebuild, +cdparanoia-3.10.2-r3.ebuild:
  Include cdda_interface.h for cdda_paranoia.h wrt #257607