Review Request 118812: [plasmashell] Show a warning if there are no Shaders and exit

[Martin Gräßlin]

> On June 18, 2014, 3 p.m., David Edmundson wrote:
> > shell/shellcorona.cpp, line 1232
> > <https://git.reviewboard.kde.org/r/118812/diff/1/?file=282339#file282339line1232>
> >
> >     just QCoreApplication::exit()
> 
> Martin Gräßlin wrote:
>     I think it makes sense to have the error code. This could allow startkde to abort the startup.
> 
> David Edmundson wrote:
>     QCoreApplication::exit(1) then.
>     
>     I meant we don't need the instance() part.
>     
>     It's possible this will fix your issue?

> It's possible this will fix your issue?

no, I had that first and thought the other way around. I also tried ::quit(), and QMetaObject::invokeMethod


- Martin


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118812/#review60395
-----------------------------------------------------------


On June 18, 2014, 1:36 p.m., Martin Gräßlin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118812/
> -----------------------------------------------------------
> 
> (Updated June 18, 2014, 1:36 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> [plasmashell] Show a warning if there are no Shaders and exit
> 
> If there are no Shaders Plasma doesn't work. If we detect this we show
> a warning (without GL) and exit.
> 
> This doesn't really work as Qt has a bug which doesn't allow to detect
> whether Shaders are supported and the exit just doesn't work.
> 
> 
> Diffs
> -----
> 
>   shell/shellcorona.h f500e837b5957e14e70ac4b24da0cdf7970a7171 
>   shell/shellcorona.cpp 4abe3432f30a8c4eb90806893f15c7c50f0e1ac2 
> 
> Diff: https://git.reviewboard.kde.org/r/118812/diff/
> 
> 
> Testing
> -------
> 
> With Mesa drivers:
> LIBGL_ALWAYS_INDIRECT=1 plasmashell
> 
> 
> Thanks,
> 
> Martin Gräßlin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140618/0d49f464/attachment.html>