Review Request 110943: Adds basic lzo support

David Hallas david at davidhallas.dk
Mon Mar 14 14:28:43 UTC 2016


> On Jan. 15, 2016, 12:09 p.m., Elvis Angelaccio wrote:
> > Hi David. Sorry for the delay.
> > 
> > Are you still interested in working on LZO support?
> > Your patch looks promising, but should be ported to the KDE Frameworks 5.
> 
> David Hallas wrote:
>     Hi Elvis,
>     
>     yes certainly. Can you give me some pointers where to start with porting the code to Frameworks 5?
>     
>     Thanks!
> 
> Elvis Angelaccio wrote:
>     > Can you give me some pointers where to start with porting the code to Frameworks 5?
>     
>     Your best bet is looking at how the Ark plugins are currently implemented. In the Ark repository, start from the `plugins/cliplugin-example` folder and then look at some actual plugin. Basically you are supposed to:
>     
>     1. Update your `CMakeLists.txt` file, until you are able to run cmake without errors
>     2. Add the `K_PLUGIN_FACTORY` macro in your `cliplugin.cpp`
>     3. Port `cliplugin.cpp` to Qt5/KF5. Since there is not much code there, it shouldn't be too hard. For instance, you will have to replace `kDebug()` with `qCDebug(ARK)`, and so on. Please refer to the official porting guide: https://community.kde.org/Frameworks/Porting_Notes
>     
>     If you have questions or problems, feel free to ask here or on IRC (in the #kde-utils channel).
>     
>     Good luck!
> 
> Ragnar Thomsen wrote:
>     I think it is better to enable support for lzop-compressed tars in the libarchive plugin than to add and maintain an additional cliplugin.
> 
> Elvis Angelaccio wrote:
>     > I think it is better to enable support for lzop-compressed tars in the libarchive plugin than to add and maintain an additional cliplugin.
>     
>     If that is possible, yes it would be better. David, what's the status of your patch?
> 
> David Hallas wrote:
>     I have attached the current patch, it is still work in progress, but it can be picked up and completed with a little dedicated effor. Unfortunately I have not had time to finish it yet :(
> 
> Ragnar Thomsen wrote:
>     I have enabled support for tar.lzo archives in the libarchive plugin (commit 17df27dab76bad4ce5bf78e1a1240408d972ef88). This will be available in Ark 16.04. I think we can discard this RR?

Yes - if you have already added support then there is no reason for this review request - please close it :)


- David


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


On June 10, 2013, 7:22 p.m., David Hallas wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/110943/
> -----------------------------------------------------------
> 
> (Updated June 10, 2013, 7:22 p.m.)
> 
> 
> Review request for KDE Utils.
> 
> 
> Bugs: 211669
>     http://bugs.kde.org/show_bug.cgi?id=211669
> 
> 
> Repository: ark
> 
> 
> Description
> -------
> 
> This patch adds basic lzo support using lzop. I want to get some early feedback before completing the code
> 
> 
> Diffs
> -----
> 
>   plugins/CMakeLists.txt 3619412 
>   plugins/clilzopplugin/CMakeLists.txt PRE-CREATION 
>   plugins/clilzopplugin/cliplugin.h PRE-CREATION 
>   plugins/clilzopplugin/cliplugin.cpp PRE-CREATION 
>   plugins/clilzopplugin/kerfuffle_clilzop.desktop.cmake PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/110943/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> David Hallas
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-utils-devel/attachments/20160314/0f1f77ed/attachment.html>


More information about the Kde-utils-devel mailing list