Review Request 121394: [OS X] create a dedicated Info.plist for kdevelop

Kevin Funk kfunk at kde.org
Mon Dec 8 22:24:11 UTC 2014


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

Ship it!


Alright, then I think it makes sense.


app/CMakeLists.txt
<https://git.reviewboard.kde.org/r/121394/#comment49922>

    Please rename to `Info.plist.in` (consistent with the original file)


Some other notes for a follow-up patch (feel free to commit directly):
- Please use `if (APPLE)` instead. I'm surprised `Q_WS_MAC` is set as *CMake* variable...
- Fix indenting above your hunk

- Kevin Funk


On Dec. 8, 2014, 9:05 p.m., René J.V. Bertin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/121394/
> -----------------------------------------------------------
> 
> (Updated Dec. 8, 2014, 9:05 p.m.)
> 
> 
> Review request for KDE Software on Mac OS X and KDevelop.
> 
> 
> Repository: kdevelop
> 
> 
> Description
> -------
> 
> This patch provides an Info.plist template and a change to the kdevelop/app/CMakeLists.txt to generate an appropriate info dictionary for the kdevelop app bundle.
> 
> The main interest at this point is the deactivation of "app nap" which will suspend the whole IDE if it remains (active) in the background for too long.
> The same file can also be used to define the application icon, and document types the application can handle. I intend to address those features in future updates (the latter will require changes to the code, though).
> 
> 
> Diffs
> -----
> 
>   app/CMakeLists.txt d84253d 
>   app/Info.plist.template PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/121394/diff/
> 
> 
> Testing
> -------
> 
> On OS X 10.9.4 . App Nap disabling seems to work as expected (and described in user feedback on the feature and its per-application deactivation). 
> 
> 
> Thanks,
> 
> René J.V. Bertin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20141208/b2f73d86/attachment-0001.html>


More information about the KDevelop-devel mailing list