Merged: MacroLogFeature and MacroLogRequirement

[Alexander Neundorf]

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.

Bye
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