search

ambs / Text-BibTeX

Text::BibTeX
6 stars 9 forks source link

Buildsystem: export C-header files #15

Open Tratoschek opened 8 years ago

Tratoschek commented 8 years ago

If I get it right, atm only libbtparse.so is installed and not the header files. This effect of this is, that the binaries included in Text-BibTeX will work, but other programs that use btparse directly cannot be compiled against Text-BibTeX. Nevertheless due to the fact, that libbtparse.so is installed, it prevents the systemwide installation of old ptparse.

It would be nice, if you include an (maybe optional) export of btparse.h and the creation of a pkg-config file (the old btparse seems to create libbtparse.a and libbtparse.la as well, I get this from this list).

In fact distributions like Debian seems to patch this in the appropriate way already.

keinstein commented 6 years ago

The header that is provided by Debian is not C++ compatible. Try the following file ptparse-error.cc:

extern "C" {
#include "btparse.h"
}

It fails to compile with g++.

ambs commented 6 years ago

anyone can test #27, and see if solves anything?