Review Request: Make a real dynamic library out of kdgantt
C. Boemann
cbr at boemann.dk
Sun Dec 30 05:54:44 GMT 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/108019/#review24216
-----------------------------------------------------------
I think the code looks nice enough, so stuart are free to approve as far as i'm concerned.
I have one little question/nitpick
3rdparty/kdgantt/CMakeLists.txt
<http://git.reviewboard.kde.org/r/108019/#comment18557>
uhm i don't get why we can't remove it already.
- C. Boemann
On Dec. 30, 2012, 5:21 a.m., Inge Wallin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/108019/
> -----------------------------------------------------------
>
> (Updated Dec. 30, 2012, 5:21 a.m.)
>
>
> Review request for Calligra and Stuart Dickson.
>
>
> Description
> -------
>
> This is an adaption of the same patch for 2.6.
>
> This patch needs to be tested on Windows, which I hope stuart can do.
>
>
> Diffs
> -----
>
> 3rdparty/kdgantt/CMakeLists.txt 4a75084
> 3rdparty/kdgantt/kdganttconstraintmodel.h bc1ed5c
> 3rdparty/kdgantt/kdganttview.cpp a09edf1
> CMakeLists.txt 09d98be
> plan/libs/ui/CMakeLists.txt c80f891
> plan/workpackage/CMakeLists.txt 7ebedce
>
> Diff: http://git.reviewboard.kde.org/r/108019/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Inge Wallin
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20121230/60fcdc57/attachment.htm>
More information about the calligra-devel
mailing list