Compilation issues on recent GNU/Linux

[Sylvain]

Hi!

Gargoyle didn't compile anymore on Debian, I just had to make a couple
minor changes:

1. clock_gettime (garglk/sysgtk.c) requires -lrt.

Well, it has been a requirement for a while AFAICS, so my guess is
that it was previously gotten indirectly through pkg-config --libs,
but not anymore.

--- gargoyle-free.orig/Jamrules 2012-05-24 21:39:10.000000000 +0200
+++ gargoyle-free/Jamrules 2012-05-24 21:39:23.000000000 +0200
@@ -88,7 +88,7 @@
PKGCONFIG = "pkg-config freetype2 gtk+-x11-2.0 gdk-x11-2.0 gobject-2.0 glib-2.0 fontconfig" ;
}
GARGLKCCFLAGS = "`$(PKGCONFIG) --cflags`" -fPIC ;
- GARGLKLIBS = "`$(PKGCONFIG) --libs`" -ljpeg -lpng -lz ;
+ GARGLKLIBS = "`$(PKGCONFIG) --libs`" -ljpeg -lpng -lz -lrt ;
LINKLIBS = -lz -lm "`$(PKGCONFIG) --libs`" ;

if $(USESDL) = yes

2. This second fix is specific to the Debian/Ubuntu package, where
SDL_sound is used from the system rather than from the bundled
copy, so I mention it for info: I needed to remove the direct
dependency on -lsmpeg et -lvorbis because the development packages
for those libs may not be installed. Possibly we may want to
handle "don't use bundled copies of dependencies" properly in the
build system rather than through distro-specific patches :)

--- gargoyle-free.orig/Jamrules 2012-05-24 21:08:24.000000000 +0200
+++ gargoyle-free/Jamrules 2012-05-24 21:10:55.000000000 +0200
@@ -94,7 +94,7 @@
if $(USESDL) = yes
{
GARGLKCCFLAGS += -I/usr/include/SDL ;
- GARGLKLIBS += -lSDL_mixer -lSDL_sound -lSDL -lsmpeg -lvorbisfile ;
+ GARGLKLIBS += -lSDL_mixer -lSDL_sound -lSDL ;
}

if $(STATIC) { LINKLIBS += $(GARGLKLIBS) ; }

Cheers!
Sylvain

[Ben Cressey]

Hi Sylvain,

I fixed the -lrt issue in r597 after I happened to try building it on the latest Fedora. I'll merge in your fix to the SDL dependencies - thanks!

At this point only the Windows and Mac targets are building their own SDL_sound. Linux targets should be linking to the distribution copy; are you seeing something else?

A couple releases back we did bundle our own copy of SDL_sound on Linux, because the packaged version clashed with SDL_mixer. That's since been fixed and I removed that logic from the build file.

Regards,

Ben

Sylvain

--
You received this message because you are subscribed to the Google Groups "garglk-dev" group.
To post to this group, send email to gargl...@googlegroups.com.
To unsubscribe from this group, send email to garglk-dev+...@googlegroups.com.
For more options, visit this group at http://groups.google.com/group/garglk-dev?hl=en.

[Sylvain]

Hi Ben,

I also used to remove the 'if $(USESDL) = yes' block at the end of
support/Jamfile, but I just noticed that is not needed anymore :)

Cheers!
Sylvain