[PATCH] bug 174806

Alexander Neundorf neundorf at kde.org
Wed Mar 25 21:49:39 CET 2009


On Wednesday 25 March 2009, Michael Witten wrote:
> Hello again!
>
> I've updated the page for the following bug:
>
>     https://bugs.kde.org/show_bug.cgi?id=174806
>
> with this attachment:
>
>     https://bugs.kde.org/attachment.cgi?id=32379
>
> and this comment:
>
> Sorry for getting back WAY too late on this one. I've got excuses, but
> who cares... ;-)
>
> I've attached a tarball that contains 3 pataches: kdelibs.patch,
> kdebase.patch, kdebindings.patch.
>
> The main patch is kdelibs.patch, which cleans up some cruft and adds 3
> new CMake commands: PARSE_ARGS(), ADD_PYTHON(), and INSTALL_PYTHON().

Just as a first comment, the macros and variables should all be "namespaced", 
i.e. they should be PYTHON_ADD_SOMETHING() and also 
PYTHON_INSTALL_SOMETHING(). 
This makes it clear where they come from.
More tomorrow :-)

Alex


More information about the Kde-buildsystem mailing list