extragear/sdk/kdevplatform/shell

[Milian Wolff]

Andreas Pakulat, 03.02.2010:
> On 03.02.10 20:47:27, Niko Sams wrote:
> > SVN commit 1084827 by nsams:
> > 
> > Add some more NoUI ifs.
> > Store sessions additionally in a QList as in NoUI mode creating KActions
> > is not possible. Fixes at least gdbtest and duchainify.
> 
> Please revert at least the keeping of an extra list. Why is that needed?
> You can easily put Session*,0 into the hash.
> 
> Frankly, I think we should just drop the noui stuff, after all we're
> building a _GUI_ app so IMHO its just fine if all core components assume
> they do have a GUI available.  If you want a unit-test that doesn't need
> a GUI, then you'll have to revert to not creating the autotestshell.

@Niko: I once wanted to push that as well but also hit against the wall. Now 
that a friend of mine showed me Xvfb there is no need for these patches (esp. 
considering that they add unneeded "overhead" (code wise) to maintain).

So revert and use Xvfb.
-- 
Milian Wolff
mail at milianw.de
http://milianw.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20100203/f55b9062/attachment.sig>