newmat10-dev installs SickConfig.h and includes it in the headers which can cause conflicts

Bug #204338 reported by Tully
6
Affects Status Importance Assigned to Milestone
newmat (Ubuntu)
Fix Released
Undecided
Philippe Coval

Bug Description

Binary package hint: libnewmat10-dev

I'm using the package libnewmat10-dev 1.10.4-2build1 and have run into a problem compiling against it. The problem stems from the inclusion of config.h into the installed headers. (Line 10 of include.h) This was discovered as a problem when I had another gnu autotools install that installed and included it's own config.h. The names didn't conflict since they were in subdirctories, but the #defines did and prevented compilation.

It is my impression that the config.h is only necessary for compiling the library and not for compiling programs to link against the library, for if they need it they will have their own config.h which will be generated with it's own configure script.

If possible I would like to remove line 10 of include.h and possibly not have config.h included in the package installation. I have commented out line 10 in my installed version and everything compiles. This either needs to be done after compiling the library since the library needs the config.h. Another solution that is more work, but wouldn't require modifying things after compile is to include config.h in the cpp files instead of the header files so that it will not be included by external programs using newmat headers.

I"m running Gutsy 7.10.

Related branches

Philippe Coval (rzr)
Changed in newmat:
assignee: nobody → rzr
Revision history for this message
Philippe Coval (rzr) wrote :

Hi,

I am preparring a new version for debian, can you check if it's ok for you ?
http://mentors.debian.net/debian/pool/main/n/newmat/

the compiled package is in my PPA, are you still using gutsy ?

http://rzr.online.fr/q/apt

Regards

Revision history for this message
Tully (tully.foote) wrote :

I am still running Gutsy.

I replaced my newmat10 and newmat10-dev with the ones posted here http://ppa.launchpad.net/rzr/ubuntu/pool/main/n/newmat/
libnewmat10-dev_1.10.4-4ubuntu0~rzr8_i386.deb
libnewmat10ldbl_1.10.4-4ubuntu0~rzr8_i386.deb

and had no problems compiling and running my code.

Thanks for fixing this,

Daniel T Chen (crimsun)
Changed in newmat:
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.