Review Request 124905: Win: Hide console window for binaries in LIBEXEC
Kevin Funk
kfunk at kde.org
Mon Jul 11 16:14:52 UTC 2016
> On April 18, 2016, 11:05 a.m., Gleb Popov wrote:
> > Let's do something about this. I see two solutions for this:
> >
> > - Add another macro. If we go this route, can we put it in the same .cmake file?
> > - Add a keyword parameter to the existing macro. Because macro uses `${ARGN}` it shouldn't break anything.
>
> Kevin Funk wrote:
> @dfaure: Do you think this issue should be fixed in Qt itself?
>
> I've just found http://stackoverflow.com/questions/33874243/qprocessstartdetached-but-hide-console-window which supports Patrick's remarks about "how we did it in KDE4 times".
>
> * We use `QProcess:startDetached` in KF5, in combination with "KDE_FORK_SLAVES" (=> causes console win)
> * We've been leaking QProcess and used `QProcess::start` in KDE4 times for 'kdeinit4.exe' (=> no console win).
>
> I haven't looked at the Windows implementation of QProcess yet, but maybe it's feasible to extend `QProcess::startDetached`? Ideas? Opinions?
>
> /me wonders why noone bothered fixing this in Qt itself yet (lots of discussions about the issues in forums)
>
> David Faure wrote:
> Yes this sounds like a Qt bug in startDetached() then.
Handled upstream: https://bugreports.qt.io/browse/QTBUG-53833 (Qt was already patched, fix will be in 5.8.0)
- Kevin
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124905/#review94684
-----------------------------------------------------------
On Dec. 3, 2015, 8 a.m., Kevin Funk wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124905/
> -----------------------------------------------------------
>
> (Updated Dec. 3, 2015, 8 a.m.)
>
>
> Review request for KDE Frameworks, Alex Merry and David Faure.
>
>
> Repository: kio
>
>
> Description
> -------
>
> Win: Hide console window for binaries in LIBEXEC
>
>
> Diffs
> -----
>
> src/ioslaves/http/CMakeLists.txt 76a8e2800b84c312431cc1996ac81d1ef6fb5cfc
> src/ioslaves/http/kcookiejar/CMakeLists.txt 7b4778d1f67c1ad9f9edcaa4692b39ee6fe3f365
> src/kioexec/CMakeLists.txt 91284a3a61b86770b4d1939da52d256840803608
> src/kioslave/CMakeLists.txt e02febd380b268c596e8ecc3b745b6f50993ab4e
> src/kpac/CMakeLists.txt fc5989714480ca49b5bd72e1c7b458b26bd0d9bc
>
> Diff: https://git.reviewboard.kde.org/r/124905/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Kevin Funk
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160711/fdfead39/attachment.html>
More information about the Kde-frameworks-devel
mailing list