> 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 > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
