[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: isnan() and isinf() checks
From:       Alexander Neundorf <neundorf () kde ! org>
Date:       2007-04-12 21:30:16
Message-ID: 200704122330.17301.neundorf () kde ! org
[Download RAW message or body]

On Thursday 12 April 2007 20:58, Hasso Tepper wrote:
> Alexander Neundorf wrote:
> > Using "-lm" is no good idea, as Christian already stated.
>
> Mkay. Better now?
>
> > Please take a look at kdelibs/cmake/modules/MacroPushRequiredVars.cmake,
> > it offers two macros:
> >
> > # Usage:
> > # MACRO_PUSH_REQUIRED_VARS()
> > # SET(CMAKE_REQUIRED_DEFINITIONS
> >       ${CMAKE_REQUIRED_DEFINITIONS} -DSOME_MORE_DEF)
> > # CHECK_FUNCTION_EXISTS(...)
> > # MACRO_POP_REQUIRED_VARS()
>
> Thanks for info.
>
> > Can you please check whether check_symbol_exists() (from
> > Modules/CheckSymbolExists.cmake) does what you need instead of using
> > check_c_source_compiles() directly ?
>
> Indeed. I didn't realise that it does work for macros as well. Updated
> patch is attached.
>

Looks good, at least from the cmake side :-)

Alex
-- 
Work: alexander.neundorf AT jenoptik.com - http://www.jenoptik-los.de
Home: neundorf AT kde.org                - http://www.kde.org
      alex AT neundorf.net               - http://www.neundorf.net
[prev in list] [next in list] [prev in thread] [next in thread]