Merged: MacroLogFeature and MacroLogRequirement

[Allen Winter]

On Saturday 24 June 2006 03:29, Alexander Neundorf wrote:
> On Saturday 24 June 2006 01:44, Allen Winter wrote:
> > Howdy,
> >
> > As promised, attached is my attempt to merge Alex's MacroLogFeature() and
> > the MacroLogRequirement() I posted here a few weeks back.  Basically, you
> > pass a variable that says whether a feature is a requirement. There are
> > also new variables for Minimum_Version and Comments.
> >
> > Also attached is how one would change FindQtDBus.cmake to use the new
> > macro.
> >
> > Output will look like the following now (looks best with fixed-width font):
> > .....
> > -- Performing Test __KDE_HAVE_GCC_VISIBILITY - Success
> > -- Found KDE4 kconfig_compiler preprocessor:
> > /data/kde/trunk/KDE/kdelibs/build-gcc/bin/./kconfig_compiler.sh --
> > Missing Requirements:
> > =======================================
> > PACKAGE:     D-BUS
> > DESCRIPTION: D-BUS message bus system
> > URL:         http://www.freedesktop.org/wiki/Software_2fdbus
> > VERSION:     0.62
> > COMMENTS:    If you have dbus installed, check PKG_CONFIG_PATH so that
> > 'pkg-config --libs dbus-1' works.  See also the PORTING-TO-DBUS.txt file in
> > kdelibs/ =======================================
> > PACKAGE:     QtDBus
> > DESCRIPTION: Qt4 D-BUS Bindings
> > URL:         $SVNROOT/trunk/kdesupport
> > VERSION:     SVN
> > COMMENTS:    If you have qt-dbus installed, check PKG_CONFIG_PATH so that
> > 'pkg-config --libs dbus-qt4-1' works.  See also the PORTING-TO-DBUS.txt
> > file in kdelibs/ =======================================
> > Exiting: Missing Requirements
> > -- Configuring done
> 
> Looks good.
> Not sure about exiting with FATAL_ERROR.
> I'd say commit.
> Maybe I'll have a look how to handle the optional arguments better.
> 
Committed.
Yeah, I played around with how to handle the optional arguments a bit, but
there is probably a better way.


-- 
Let's Keep the Political Talk Out of KDE PLEASE