On 17.08.08 01:38:25, Alexander Neundorf wrote: > On Saturday 16 August 2008, Andreas Pakulat wrote: > > On 15.08.08 21:56:42, Alexander Neundorf wrote: > > > On Friday 15 August 2008, Andreas Pakulat wrote: > > > > On 15.08.08 00:13:49, Alexander Neundorf wrote: > > > > > Con: > > > > > -with the patch there are now two ways how to do the same thing (and > > > > > the old way has to stay, in order to stay source compatible) > > > > > > > > Well, while I agree its better to leave it in, the variable isn't > > > > documented and hence is IMHO an internal variable - > > > > > > Ooops, this is a bug, this variable was/is intended to be public. > > > > I've added documentation for the "new" way including a small example, > > should I also mention the variable or are we deprecating the variable? > > Hmm. Now we have a nicer way, but without documenting this variable we have > no > documented way how to do this for KDE 4.0.x and 4.1.x. > So I'm afraid we should make it public right now that we deprecated it :-/
Ok, done and comitted. Including a backport of the KDE_MIN_VERSION to 4.1 branch. Andreas -- Your love life will be happy and harmonious. _______________________________________________ Kde-buildsystem mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-buildsystem
