Review Request 126675: Add .kra and .ora qimagio plugins

Alex Merry alex.merry at kde.org
Sun Jan 31 13:03:53 UTC 2016


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




CMakeLists.txt (line 22)
<https://git.reviewboard.kde.org/r/126675/#comment62684>

    What about making this optional (and only building those two plugins if it is found)?


- Alex Merry


On Jan. 27, 2016, 9:48 a.m., Boudewijn Rempt wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126675/
> -----------------------------------------------------------
> 
> (Updated Jan. 27, 2016, 9:48 a.m.)
> 
> 
> Review request for KDE Frameworks and Alex Merry.
> 
> 
> Repository: kimageformats
> 
> 
> Description
> -------
> 
> These qimageio plugins originally come from krita, but people who haven't got krita installed might want to view Krita or OpenRaster images as well. Both formats are really simple, just zip files with a png inside that represents the merged image. This patch does add a KF5::Archive dependency to kimageformats.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 1e7e703 
>   autotests/CMakeLists.txt 853607e 
>   autotests/read/kra/src.kra PRE-CREATION 
>   autotests/read/kra/src.png PRE-CREATION 
>   autotests/read/ora/src.ora PRE-CREATION 
>   autotests/read/ora/src.png PRE-CREATION 
>   src/imageformats/CMakeLists.txt 8cf8d54 
>   src/imageformats/kra.cpp PRE-CREATION 
>   src/imageformats/kra.desktop PRE-CREATION 
>   src/imageformats/kra.h PRE-CREATION 
>   src/imageformats/kra.json PRE-CREATION 
>   src/imageformats/ora.cpp PRE-CREATION 
>   src/imageformats/ora.desktop PRE-CREATION 
>   src/imageformats/ora.h PRE-CREATION 
>   src/imageformats/ora.json PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/126675/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Boudewijn Rempt
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160131/08e1283d/attachment.html>


More information about the Kde-frameworks-devel mailing list