2012/5/18 mathog <mathog@...1176...>:
On 18-May-2012 13:58, Ted Gould wrote:
What's important to realize here is that we *really* don't want to include libraries in the Inkscape codebase. It's a bad habit that we really need to kick :-) The libraries that are in the Inkscape codebase today are ones that were developed by Inkscape developers along side, or ones that we had to patch significantly.
The problem here is that I had to modify the code in libemf somewhat to get it to work.
In this case it's best to submit these modifications upstream.
Also it isn't a very common library, so very few machines are going to have it installed.
The presence of this library should not be required to compile Inkscape - the autoconf script should detect whether it's available.
Perhaps "library" really isn't the right term here. The 10 or so files live in one directory, but it doesn't need to build to a .a or .so or .dll, just to a .o (or .obj) which is linked into the one big inkscape library.
This will increase link time for no good reason. If you really want to do this, just copy the approach used for libavoid, libcroco, and lib2geom, which discards all of the original autoconf files and builds the library as a local .a file.
Regards, Krzysztof