Review Request: Move kdchart from plugins/chartshape to 3rdparty/
Boudewijn Rempt
boud at valdyas.org
Sun Dec 16 12:11:57 GMT 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107747/#review23524
-----------------------------------------------------------
Ship it!
Looks fine to me, a nice cleanup.
- Boudewijn Rempt
On Dec. 15, 2012, 11:59 p.m., Inge Wallin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107747/
> -----------------------------------------------------------
>
> (Updated Dec. 15, 2012, 11:59 p.m.)
>
>
> Review request for Calligra.
>
>
> Description
> -------
>
> This patch moves the next 3rd party library, namely kdchart, to 3rdparty/.
>
> To test this, in addition to applying this patch you must type
>
> git mv plugins/chartchape/kdchart 3rdparty
>
> And if you want, also:
>
> git mv plugins/chartchape/kdchartpathces 3rdparty
>
> but that will not result in any code changes.
>
>
> Diffs
> -----
>
> 3rdparty/CMakeLists.txt a167fa3
> CMakeLists.txt 518954e
> plan/CMakeLists.txt 8a8931f
> plan/libs/models/CMakeLists.txt 18c7af3
> plan/libs/ui/CMakeLists.txt 4546a20
> plugins/chartshape/CMakeLists.txt 1acbd63
> plugins/chartshape/tests/CMakeLists.txt e67a0cb
> plugins/reporting/chart/CMakeLists.txt 1d80ca9
>
> Diff: http://git.reviewboard.kde.org/r/107747/diff/
>
>
> Testing
> -------
>
> make test
>
> and
>
> calligrawords allcharts.odt
>
>
> Thanks,
>
> Inge Wallin
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20121216/db98b484/attachment.htm>
More information about the calligra-devel
mailing list