Re: Review Request 115251: Add better runtime detection for X11 usage in KStartupInfo

2014-02-02 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115251/ --- (Updated Feb. 2, 2014, 11:06 a.m.) Status -- This change has been ma

Re: Review Request 115251: Add better runtime detection for X11 usage in KStartupInfo

2014-02-02 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115251/#review48760 --- This review has been submitted with commit 58d640cb78573b7929

Re: Review Request 115251: Add better runtime detection for X11 usage in KStartupInfo

2014-02-01 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115251/#review48742 --- Ship it! Ship It! - Alex Merry On Feb. 1, 2014, 7:40 a.m.,

Re: Review Request 115251: Add better runtime detection for X11 usage in KStartupInfo

2014-01-31 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115251/ --- (Updated Feb. 1, 2014, 8:40 a.m.) Review request for KDE Frameworks. Ch

Re: Review Request 115251: Add better runtime detection for X11 usage in KStartupInfo

2014-01-31 Thread Alex Merry
> On Jan. 31, 2014, 8:45 p.m., Alex Merry wrote: > > src/kstartupinfo.cpp, lines 458-480 > > > > > > Surely we shouldn't be using KXMessages if we don't have X11? > > > > (Also applies to the next bunch

Re: Review Request 115251: Add better runtime detection for X11 usage in KStartupInfo

2014-01-31 Thread Martin Gräßlin
> On Jan. 31, 2014, 9:45 p.m., Alex Merry wrote: > > src/kstartupinfo.cpp, lines 458-480 > > > > > > Surely we shouldn't be using KXMessages if we don't have X11? > > > > (Also applies to the next bunch

Re: Review Request 115251: Add better runtime detection for X11 usage in KStartupInfo

2014-01-31 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115251/#review48713 --- src/kstartupinfo.cpp

Re: Review Request 115251: Add better runtime detection for X11 usage in KStartupInfo

2014-01-31 Thread Martin Gräßlin
> On Jan. 30, 2014, 3:49 p.m., David Edmundson wrote: > > src/kstartupinfo.cpp, line 1068 > > > > > > Does this need a runtime check? no that should be fine. The Qt method delegates to nativeResourceForScreen i

Re: Review Request 115251: Add better runtime detection for X11 usage in KStartupInfo

2014-01-30 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115251/#review48622 --- src/kstartupinfo.cpp

Re: Review Request 115251: Add better runtime detection for X11 usage in KStartupInfo

2014-01-26 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115251/ --- (Updated Jan. 27, 2014, 8:31 a.m.) Review request for KDE Frameworks. C

Re: Review Request 115251: Add better runtime detection for X11 usage in KStartupInfo

2014-01-23 Thread Àlex Fiestas
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115251/#review48141 --- Ship it! The rest of the code is fine, +1 from me. src/ksta

Review Request 115251: Add better runtime detection for X11 usage in KStartupInfo

2014-01-23 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115251/ --- Review request for KDE Frameworks. Repository: kwindowsystem Descriptio