On Thu, Nov 12, 2009 at 10:13:22AM +0100, Fabian Groffen wrote:
> On 12-11-2009 10:11:13 +0100, Stefan Manegold wrote:
> > On Thu, Nov 12, 2009 at 09:06:23AM +0000, Fabian wrote:
> > > Update of /cvsroot/monetdb/MonetDB
> > > In directory 23jxhf1.ch3.sourceforge.com:/tmp/cvs-serv25980
> > > 
> > > Modified Files:
> > >       Tag: Nov2009
> > >   configure.ag 
> > > Log Message:
> > > Yippy yay for messing with this at this stage of the development cycle... 
> > >  Revert my yesterday's commit here as it not only breaks compilation on 
> > > OSX 10.6 (the original problem), but also Solaris 10.  We need to 
> > > facilitate people doing cross-compilation in some other way.
> > 
> > Might simply using AC_CHECK_FUNCS(fdatasync) be an option, here?
> 
> That's the whole problem, for some reason configure then thinks the
> function is available, which it is not on OSX 10.6.

The reason is that with compilation, only, a non-declared triggers only a
warning with most compilers:
from
http://monetdb.cwi.nl/testing/projects/monetdb/Stable/MonetDB/.GNU.32.32.d.1-Darwin9.8.0..2009.11.12_00-09-02/config.out_log.html
========
configure:39146: checking for fdatasync
configure:39177: gcc -c -O2 -Wall -Wextra -O6 -fomit-frame-pointer              
            -finline-functions -falign-loops=4 -falign-jumps=4 
-falign-functions=4 -fexpensive-optimizations                     
-funroll-loops -frerun-cse-after-loop -frerun-loop-opt -ftree-vectorize  
-std=c99 conftest.c >&5
conftest.c: In function 'main':
conftest.c:111: warning: implicit declaration of function 'fdatasync'
configure:39183: $? = 0
configure:39199: result: yes
========
(and, provided I recall correctly, we cannot turn warnings into errors
with/during configure, as then too many configure checks would fail ...)

I was hoping that AC_CHECK_FUNCS would do more than just compile, e.g., also
(try to) link ...

> I now try a version using AC_TRY_LINK, as I only used AC_TRY_COMPILE in
> yesterday's patch.

That might indeed be the best idea, then ...

Stefan

> ------------------------------------------------------------------------------
> Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
> trial. Simplify your report design, integration and deployment - and focus on 
> what you do best, core application coding. Discover what's new with
> Crystal Reports now.  http://p.sf.net/sfu/bobj-july
> _______________________________________________
> Monetdb-checkins mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/monetdb-checkins
> 
> 

-- 
| Dr. Stefan Manegold | mailto:[email protected] |
| CWI,  P.O.Box 94079 | http://www.cwi.nl/~manegold/  |
| 1090 GB Amsterdam   | Tel.: +31 (20) 592-4212       |
| The Netherlands     | Fax : +31 (20) 592-4312       |

------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
_______________________________________________
Monetdb-developers mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/monetdb-developers

Reply via email to