Re: Review Request: only install CMakeParseArguments.cmake for cmake versions that do not have it

2011-10-24 Thread Alexander Neundorf
On Monday 24 October 2011, Ralf Jung wrote: > Hi, > > > Yes, most probably. > > > > > > I guess it was first developed as a standalone tool, where you have to do > > everything yourself, including searching for Qt4 etc., and then was added > > into kdebase. > > Checking the other cmake files of

Re: Review Request: only install CMakeParseArguments.cmake for cmake versions that do not have it

2011-10-24 Thread Ralf Jung
Hi, > Yes, most probably. > > > I guess it was first developed as a standalone tool, where you have to do > everything yourself, including searching for Qt4 etc., and then was added > into kdebase. Checking the other cmake files of libkephal, it does not seem to support standalone builds anymor

Re: Review Request: facePerm is a KDM option, unrelated to the user setting his face (for other apps)

2011-10-24 Thread Ralf Jung
> On Oct. 23, 2011, 4:22 p.m., Ralf Jung wrote: > > Any news? Can I ship this (simplified) version of the original request? > > Oswald Buddenhagen wrote: > in qt review terms, you get a +1 from me. this code is pretty much > unmaintained, so it's unlikely that you'll get more feedback than