Bug 228185 - dev-libs/libcdio-0.80: incorrect Makefile.am
Bug#: 228185 Product:  Gentoo Linux Version: unspecified Platform: All
OS/Version: Linux Status: RESOLVED Severity: normal Priority: P2
Resolution: FIXED Assigned To: flameeyes@gentoo.org Reported By: galtgendo@o2.pl
Component: Applications
URL: 
Summary: dev-libs/libcdio-0.80: incorrect Makefile.am
Keywords:  
Status Whiteboard: 
Opened: 2008-06-18 22:02 0000
Description:   Opened: 2008-06-18 22:02 0000 
Unless there was a change in autoconf behavior,
libcdio has an incorrect configure.ac.
I had a similar problem recently with hal.
AC_ARG_WITH and AC_ARG_ENABLE first argument should be
in the form foo-bar, not foo_bar. This doesn't hurt us (at least non-minimal
build), but is still incorrect.

There's also a very hard to notice bug in Makefile.am, one that causes two
pkg-config files to not be installed, a missing backslash.

------- Comment #1 From Rafał Mużyło 2008-06-18 22:04:04 0000 ------- 
Created an attachment (id=157509) [details]
trivial patch

This trivial patch corrects pkg-config problem.

------- Comment #2 From Rafał Mużyło 2008-06-18 22:05:16 0000 ------- 
Oh, and a minor thing:
configure: WARNING: Unrecognized options: --enable-nls

------- Comment #3 From Rafał Mużyło 2008-06-18 22:11:18 0000 ------- 
oops, those configure.ac things, are just inconsistencies, not bugs.

------- Comment #4 From Diego E. 'Flameeyes' Pettenò 2008-06-19 23:27:24 0000 ------- 
*** Bug 228351 has been marked as a duplicate of this bug. ***

------- Comment #5 From Alexis Ballier 2008-06-21 07:53:53 0000 ------- 
*** Bug 228655 has been marked as a duplicate of this bug. ***

------- Comment #6 From Diego E. 'Flameeyes' Pettenò 2008-06-21 10:30:46 0000 ------- 
Thanks Rafal, I've fixed it with a different patch so that it also takes care
of skipping over the C++ bindings.