On Wed, 1 Aug 2012, Tim Jenness wrote:
> 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.
Don't think that's relevant for skycat. What we really need is a patch to
the AC_PATH_X macro that searches /opt/X11, Tk etc. use this to locate X.
> 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?
The latter part is clearly just too much effort not too. I've removed this
file from the repository.
> PPS http://www.ohloh.net/p/Starlink/contributors is pretty good.
Hah yes, I see you are top! And I only write Tcl? And Malcolm Fortran 77.
Peter.
|