Gwenview ui file in 15.08

David Edmundson david at davidedmundson.co.uk
Sat Oct 10 11:11:51 UTC 2015


On Wed, Oct 7, 2015 at 10:13 PM, Luigi Toscano <luigi.toscano at tiscali.it>
wrote:

> Hi Ivan,
> the change you committed few months ago to Gwenview:
>
> https://quickgit.kde.org/?p=gwenview.git&a=commit&h=78e4e57af3a5476dd949797c559a3d25d8ce50a1
>
> broke the loading of the menu, as the ui file still has the old name and
> there
> is no explicit setting of the .ui file name.
>
> There is a bug for this and two review requests:
> https://bugs.kde.org/show_bug.cgi?id=351431
>
> The older RR reverts your change:
> https://git.reviewboard.kde.org/r/125216/
>
> The newer one renames the .ui file on installation and fixes the directory.
> https://git.reviewboard.kde.org/r/125524/
>
> Could you please comment on the proper way to solve this? (if you can do
> before tomorrow, October 8, this can go in for 15.08.2).
>

I'm not sure we can revert, it changes the file path of one of the config
files used too.

Second patch looks OK, and I've given it some testing, and it seems to be
fine.
I'll merge it now, given it's fairly major would it be possible to have a
respin of gwenview for Applications 15.08.2?

David
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/release-team/attachments/20151010/0d9e6991/attachment.html>


More information about the release-team mailing list