Hi Cyril.

...

I do not understand the code in deep but still feel this patch is questionable. It works but I'd expect a wrapper around XKeysymToString so no direct "-lX11" is needed in tests and demos utilities. Let this be a question for the maintainer :-)

This should be covered by the configure script. If you run configure, it looks for X11 devel headers and if found it enables build againts X11 libraries (and generally any libraries it should link with).

Now if there are no X11 libs detected the X11 backend should be compiled with stubs functions that only prints "X11 not compiled in" and aborts.

So generally whole library could be compiled without X11 (or any other library) but some demos may not run (these that opens a window and draws into it).

Check output from 'gfxprim-config --libs-backends' which should print linker flags for drawing backends (i.e. X11, SDL, Framebuffer) and is used to fill the LDLIBS_LOADERS make variable.

mvancura@prisera:~/gfxprim$ gfxprim-config --libs-backends -lGP_backends -lX11 -lXext

Moreover something seems fishy here as test for loaders are not linked agains backends at all. So this rather seems to be bug in the build system or something similar.

Could you send a log from verbose build, i.e. output from:

VERBOSE=1 make

BTW: This kind of questions would be better discussed on GFXprim mailing list (although I would be most likely the only one answering them there).

Moved to the mailing list. Both the patch in question and requested log are attached.

Details:

git commit 5381c2dcc328739b9d82e462da5e2135a8d23c2a (current HEAD of master)

machine is debian squeeze, libx11-6, libx11-data and libx11-dev installed

With the attached patch, I can build gfxprim succesfully and spiv also works then.

Milan Vancura