Print

Print




On Wed, Aug 1, 2012 at 8:18 AM, Tim Jenness <[log in to unmask]> wrote:


On Wed, Aug 1, 2012 at 1:12 AM, Peter W. Draper <[log in to unmask]> wrote:

AC_PATH_XTRA doesn't do that for X_LIBS either, we add this to the _LDADD flags in the Makefile.am. So correspondindly we should add X_CFLAGS to the (I'd guess) AM_CFLAGS in Makefile.am.

Ah yes. http://code.ohloh.net reinforces that approach so I'll fix everything up.


Peter, can you have a look at skycat? I'm not sure which bit of a Makefile.in file I'm meant to add $X_CFLAGS. It currently fails on Mountain Lion with clang/clang++ version 4 because the X11 include files are missing.

PS Skycat on OSX now modifies skycat/linux/skycat.spec replacing all the .so entries with .dylib. This annoys git due to the modifications of a committed file. Should the file be in the repository and why is a linux file modified on darwin? 

PPS http://www.ohloh.net/p/Starlink/contributors is pretty good.

-- 
Tim Jenness