[KDE/Mac] Review Request 119533: Make ktimetracker build on OS X, where it uses a native method from the ApplicationServices framework for idle time detection, instead of the XScreenSaver.

[René J.V. Bertin]

> On July 31, 2014, 2:10 p.m., Allen Winter wrote:
> > CMakeLists.txt, line 310
> > <https://git.reviewboard.kde.org/r/119533/diff/1/?file=294203#file294203line310>
> >
> >     remove the conditional now
> 
> Allen Winter wrote:
>     If you don't have commit access, please let us know so someone else can commit for you.
> 
> René J.V. Bertin wrote:
>     That'd be the case - I removed the conditional since it's no longer required, and uploaded a new, monolithic diff/patch file.
> 
> René J.V. Bertin wrote:
>     I don't see my changes in kdepim git/kde4.14 (and no ktimetracker at all in git/master) - has this been committed by anyone?

> On Monday September 08 2014, Christophe Giboudeaux wrote, on kde-pim:
>That's unfortunate but expected.
>The review request was created after the KDE/4.14 branch creation.
>The change was only pushed to master since there was no unittest and the 
>comment in mainwindow.cpp indicates that it cannot end in a stable branch 
>without more testing.

What comment is that?


- René J.V.


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


On July 31, 2014, 10:27 p.m., René J.V. Bertin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119533/
> -----------------------------------------------------------
> 
> (Updated July 31, 2014, 10:27 p.m.)
> 
> 
> Review request for KDE Software on Mac OS X and KDEPIM.
> 
> 
> Repository: kdepim
> 
> 
> Description
> -------
> 
> Signed-off-by: RJVB <rjvbertin at gmail.com>
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt.orig d12b90a 
>   ktimetracker/CMakeLists.txt bf7b0ef41d851055359dc2cdcb2d2d3cbce25a6d 
>   ktimetracker/idletimedetector.h 524f739e89a77d1dbd732af8f258b3276e994eba 
>   ktimetracker/idletimedetector.cpp 250dda55582f53cfc44242582dee7e07062f5358 
>   ktimetracker/ktimetrackerutility.cpp efeba3abdc94a3725f0e0d6a9b663959a06dcc1f 
>   ktimetracker/mainwindow.cpp 56d4c286213507490b31fb131f1097a4c00b7abe 
> 
> Diff: https://git.reviewboard.kde.org/r/119533/diff/
> 
> 
> Testing
> -------
> 
> Testing was done under KDE and KDEPIM 4.12.5, MacPorts. The patches from that version applied without issues on the code in git, with 2 exceptions that I could re-apply manually easily.
> 
> With these patches, ktimetracker builds and launches under OS X (•) and loads existing ktimetracker.ics files correctly. The idle time detection feature appears to work as far as I'm able to tell (I don't use it under Linux).
> 
> What remains to be done is to double-check the timing precision: I've had an impression that timing may be too fast but haven't been able to prove timing isn't correct.
> 
> The history output to clipboard feature does ignore the date range and "output only the selection" options; removing entries originating from Linux we have
> 
> "test"	0.00	0.00
> "test->test 1"	0.07	0.03
> "test->test 2"	0.00	0.06
> "test->test 3"	0.00	0.03
> "test->test 4"	0.00	0.03
> 
> and the timing output (which does honour the "output only the selection" option):
> 
> Totaux des tâches
> 20140729 13:09
> 
>  Durée    Tâche
> ----------------------------------------------
>   0:12    test
>    0:05    test 1
>    0:03    test 2
>    0:02    test 3
>    0:02    test 4
> ----------------------------------------------
>   0:12 Total
> 
> 
> •) this supposes libical is installed and built following https://trac.macports.org/ticket/44469, i.e. using cmake instead and including a patch from Ubuntu.
> 
> 
> Thanks,
> 
> René J.V. Bertin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-mac/attachments/20140908/96b5a960/attachment-0001.html>