Review Request 127962: Patch QProcess::startDetached() so that console applications wouldn't show up if they are launched from a GUI app.

[Gleb Popov]

> On May 19, 2016, 4:50 p.m., Aleix Pol Gonzalez wrote:
> > Has it been submitted to Qt's gerrit? URL?
> 
> Gleb Popov wrote:
>     Nope, it wasn't. I'm a bit lazy to setup all that gerrit stuff. I'm also not sure if this would get accepted.
>     
>     We already have a bucnh of patches for Qt on Windows, which aren't going to be upstreamed, why not add another one to them?
> 
> Kevin Funk wrote:
>     Submitted it: https://codereview.qt-project.org/#/c/162585/
>     
>     I don't think it's going to be submitted as-is (because it's a behavior change), I'm waiting for the input of the TQtC guys.
>     
>     I want this in emerge nevertheless, pushing.

Hum. In my patch i've replaced `CREATE_NEW_CONSOLE` with `CREATE_NEW_WINDOW` and now i don't remember why. I guess, it should be `CREATE_NEW_CONSOLE`.


- Gleb


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


On May 19, 2016, 4:12 p.m., Gleb Popov wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127962/
> -----------------------------------------------------------
> 
> (Updated May 19, 2016, 4:12 p.m.)
> 
> 
> Review request for kdewin and Kevin Funk.
> 
> 
> Repository: emerge
> 
> 
> Description
> -------
> 
> Ran KDevelop and it doesn't spawn console windows anymore.
> 
> 
> Diffs
> -----
> 
>   portage/libs/qt5/qtbase/qtbase-20130714.patch 86d5897 
> 
> Diff: https://git.reviewboard.kde.org/r/127962/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Gleb Popov
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-windows/attachments/20160615/edb0c744/attachment.html>