[KDE/Mac] Review Request 123353: [OS X] Make ktimetracker build and fully functional

René J.V. Bertin rjvbertin at gmail.com
Mon Apr 13 15:29:10 UTC 2015


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

Review request for KDE Software on Mac OS X, KDEPIM and Thorsten Staerk.


Repository: kdepim


Description
-------

A while ago I created a RR for ktimetracker on OS X (https://git.reviewboard.kde.org/r/119533/) which was either never committed (despite a "Ship It") or which was reverted at a later stage. I therefore re-submit the patch with updates.

I have now a better fix for the failing dynamic_cast in mainwindow.cpp . The cast itself continues to fail despite ensuring that `ktimetrackerpart` is exported. It appears not to be required on OS X, however.
The idle time detection code for OS X has not been changed.


Diffs
-----

  ktimetracker/CMakeLists.txt bf7b0ef 
  ktimetracker/idletimedetector.h 524f739 
  ktimetracker/idletimedetector.cpp 250dda5 
  ktimetracker/ktimetrackerpart.h 95da43b 
  ktimetracker/ktimetrackerpart_export.h PRE-CREATION 
  ktimetracker/ktimetrackerutility.cpp efeba3a 
  ktimetracker/mainwindow.cpp 56d4c28 

Diff: https://git.reviewboard.kde.org/r/123353/diff/


Testing
-------

On OS X 10.9.5 with kdelibs 4.14.5


Thanks,

René J.V. Bertin

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-mac/attachments/20150413/8e1bfd8d/attachment.html>


More information about the kde-mac mailing list