[Kde-pim] Review Request 123353: [OS X] Make ktimetracker build and fully functional

René J.V. Bertin rjvbertin at gmail.com
Mon Apr 13 16:29:10 BST 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

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/


More information about the kde-pim mailing list