Review Request 129004: Offer to install kipi if it is not installed

Aleix Pol Gonzalez aleixpol at kde.org
Fri Sep 23 17:07:10 UTC 2016


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



I'd say it would be much simpler to do `QDesktopServices::openUrl("appstream:/photolayoutseditor.desktop")`, which will open Discover/GnomeSoftware/whatever.


app/kipiexportaction.cpp (line 63)
<https://git.reviewboard.kde.org/r/129004/#comment66924>

    ?



app/kipiinterface.cpp (line 238)
<https://git.reviewboard.kde.org/r/129004/#comment66925>

    ?



app/kipiinterface.cpp (line 360)
<https://git.reviewboard.kde.org/r/129004/#comment66926>

    Allocate on the stack, you're leaking the database instance?


- Aleix Pol Gonzalez


On Sept. 23, 2016, 7:04 p.m., Jonathan Riddell wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129004/
> -----------------------------------------------------------
> 
> (Updated Sept. 23, 2016, 7:04 p.m.)
> 
> 
> Review request for Gwenview and Aleix Pol Gonzalez.
> 
> 
> Repository: gwenview
> 
> 
> Description
> -------
> 
> Use Appstream and Packagekit to offer to install Kipi plugins if they are not already installed
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 82b219a 
>   app/CMakeLists.txt ed77d7c 
>   app/kipiexportaction.cpp cd74b6a 
>   app/kipiinterface.h 1c06cd4 
>   app/kipiinterface.cpp 04dccb3 
> 
> Diff: https://git.reviewboard.kde.org/r/129004/diff/
> 
> 
> Testing
> -------
> 
> compile without appstream/packagekit, works same as before
> compile with them it adds a menu entry to install it.  This pops up a progressdialog while packagekit installs it.  on install the menu gets reloaded with new plugins
> (note this doesn't work on neon currently because libkipi-data installs the dummy test plugins so it considers kipi-plugins to be installed, delete those before trying)
> 
> 
> Thanks,
> 
> Jonathan Riddell
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/gwenview-devel/attachments/20160923/8edee5e9/attachment.html>


More information about the Gwenview-devel mailing list