Bug#681589: givaro: uses system include syntax for local header

[Neil Williams]

Source: givaro
Version: 3.7.0-2
Severity: serious
Justification: breaks reverse dependencies
Tag: patch

http://gcc.gnu.org/onlinedocs/cpp/Include-Syntax.html#Include-Syntax

givaro-3.7.0/src/kernel/gmp++/gmp++.h specifies an include for one of
it's own header files (gmpxx.h) using system include syntax instead of
local include syntax.

This has two effects:

0: If /usr/include/gmp++/gmp++.h exists when givaro is built, that will
be used in preference to the (possibly modified) copy
in ./src/kernel/gmp ++/

1: linbox and other packages which use givaro_check.m4 will fail to
build from source because gmpxx.h cannot be found by the configure
check.

The syntax needs to be changed but also, if givaro used pkg-config,
there would be no need for givaro_check.m4 in the first place.

-- 


Neil Williams
=============
http://www.linux.codehelp.co.uk/

-------------- next part --------------
A non-text attachment was scrubbed...
Name: header.diff
Type: text/x-diff
Size: 353 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/debian-science-maintainers/attachments/20120714/0ef73ea3/attachment.diff>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/debian-science-maintainers/attachments/20120714/0ef73ea3/attachment.pgp>