Feature request - Linux compile/install scripts about xlnt

[tfussell]

One of my major goals with this library is making it easy to use for other developers. It seems like implementing these changes would help ensure that that is the case. I'll get to work on this soon.

[Malvineous]

Great, many thanks! This will make it much easier to deploy xlnt with applications that use it.

[tfussell]

I think I've finished this feature (still need to add declspec macros for Windows). I never really used pkg-config before, it's pretty nice. You can now configure a make project and install the library like this:

mkdir build
cmake ../cmake
make install

cmake now takes three options:
SHARED (default=ON)
BUILD_TESTS (default=OFF)
AUTORUN_TESTS (default=OFF)

building a static library would then look like:

cmake -D SHARED=0 ../cmake

set the install prefix like this:

cmake -D CMAKE_INSTALL_PREFIX=/usr/local ../cmake

uninstall installed files with:

make uninstall

Trying out pkg-config, it seems to be working:

pkg-config --libs xlnt

yields

-L/usr/local/lib -lxlnt

and

pkg-config --cflags xlnt

yields

-I/usr/local/include

Let me know how this works for you. I'll need to document these new features somewhere.

[Malvineous]

Wow that's fantastic! That works really well. Being able to switch the tests on and off works really well too.

I do get one error when compiling my own code from the installed library:

/usr/include/xlnt/cell/types.hpp:29:27: fatal error: xlnt_config.hpp: No such file or directory

This file is located in /usr/include/xlnt/xlnt_config.hpp but the #include line in types.hpp uses double-quotes rather than angle brackets, so the compiler looks in the current directory instead of the include path.

I imagine changing the line to #include <xlnt/xlnt_config.hpp> would do the trick.

Otherwise, it works spectacularly well! Great job :-)

[Malvineous]

Just FYI I have created an Arch Linux user-package for xlnt just to make it that little bit easier to install for Arch Linux users such as myself :-)

[tfussell]

That looks useful. Thanks.