Review Request 130239: Exclude kconf_update from catching by uac on Windows

[Ralf Habacker]

> On Sept. 26, 2017, 8:02 nachm., Albert Astals Cid wrote:
> > cmake/modules/Win32Macros.cmake
> > Lines 61 (patched)
> > <https://git.reviewboard.kde.org/r/130239/diff/2/?file=513534#file513534line61>
> >
> >     I don't think it makes sense adding a new public-ish macro in 4.14.lots
> 
> Ralf Habacker wrote:
>     See revision 1 of this patch at https://git.reviewboard.kde.org/r/130239/diff/1/
> 
> Albert Astals Cid wrote:
>     I'm personally happier with that patch for almost dead 4.14 (it has 2 releases left), now you're going to need someone with a win32 build/knowledge to approve since honestly i have no idea what this does.

> for almost dead 4.14 (it has 2 releases left),

Looking at the number of open KF5 related windows issues I guess kmymoney and umbrello for Windows will depends on KDE4 at least a year. 

>i have no idea what this does.

>From https://stackoverflow.com/questions/20096706/how-does-windows-decide-whether-to-display-the-uac-prompt

kconf_update is "hitting a Windows Installer Detection Technology compatibility heuristic. Windows will try to detect when an application is an installer, and probably needs to be elevated."

BTW: The same issue happens also with KF5


- Ralf


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


On Sept. 26, 2017, 9:54 vorm., Ralf Habacker wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/130239/
> -----------------------------------------------------------
> 
> (Updated Sept. 26, 2017, 9:54 vorm.)
> 
> 
> Review request for kdelibs and Albert Astals Cid.
> 
> 
> Bugs: 384334
>     http://bugs.kde.org/show_bug.cgi?id=384334
> 
> 
> Repository: kdelibs
> 
> 
> Description
> -------
> 
> This patch fixes an issue that on the first start of KMyMoney (and on some computers on every start for example reported by https://forum.kde.org/viewtopic.php?f=69&t=141673&sid=ba4dcaa26018f16e19d21c58c34ad4e9) on running kconf_update the user account control pops up and forces the user to allow or deny  this application.
> 
> 
> Diffs
> -----
> 
>   cmake/modules/Win32Macros.cmake e5a3655b018c56f1d394c224b1f1496741171781 
>   kconf_update/CMakeLists.txt 6bf3bf1f923baf11aad63e10fe249d70ce1b4d33 
> 
> 
> Diff: https://git.reviewboard.kde.org/r/130239/diff/2/
> 
> 
> Testing
> -------
> 
> Compiled at https://build.opensuse.org/package/show/windows%3Amingw%3Awin32/mingw32-kdelibs4 and tested with KMyMoney snapshot for Windows downloadable from https://software.opensuse.org/package/mingw32-kmymoney-portable
> 
> 
> Thanks,
> 
> Ralf Habacker
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20170927/4b5ccdc5/attachment.htm>