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

List:       kde-buildsystem
Subject:    msvc: makekdewidgets
From:       Paulo.Guedes () artelecom ! pt (Paulo Jorge Guedes)
Date:       2006-05-15 13:11:51
Message-ID: F8784F641CC587479DC4FD0A08E9571D0152A6EE () SRVEXC02 ! corp ! artelecom ! pt
[Download RAW message or body]

> -----Original Message-----
> From: Peter K?mmel [mailto:syntheticpp at gmx.net]
> Sent: segunda-feira, 15 de Maio de 2006 12:11
> To: kde-buildsystem at kde.org
> Subject: Re: msvc: makekdewidgets

> > --- FindKDEWIN32.cmake    (revision 540992)
> > +++ FindKDEWIN32.cmake    (working copy)
> > @@ -20,11 +20,11 @@
> >
> >  # at first find the kdewin32 library, this has to be compiled and
> installed before kdelibs/
> >  # search for kdewin32 in the default install directory for applications
> (default of (n)make install)
> > -if (MSVC)
> > +if (MSVC AND CMAKE_BUILD_TYPE MATCHES Debug)
> >     set(_KDEWIN32_POSTFIX d)
> > -else (MSVC)
> > +else (MSVC AND CMAKE_BUILD_TYPE MATCHES Debug)
> >     set(_KDEWIN32_POSTFIX )
> > -endif (MSVC)
> > +endif (MSVC AND CMAKE_BUILD_TYPE MATCHES Debug)
> >
> >  FIND_LIBRARY(KDEWIN32_LIBRARY NAMES kdewin32${_KDEWIN32_POSTFIX}
> >    PATHS
> 
> When CMAKE_BUILD_TYPE also switches the CMAKE_DEBUG_POSTFIX macro
> then your patch looks good. 

Yes, I forgot that.

> I'll test it.

Paulo

P.S.: I think the nightly build for tomorrow will have a 0 error msvc build ;)

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

Configure | About | News | Add a list | Sponsored by KoreLogic