Review Request 119142: KXMLGUI: load .rc files from DATADIR/kxmlgui5/component_name/file.rc
Marko Käning
mk-lists at email.de
Sun Jul 6 13:21:34 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119142/#review61699
-----------------------------------------------------------
Great to see this progressing, guys! I would like to test it, but atm I can run neither kate nor kwrite due to issue 337140. Thanks for having picked this up!
- Marko Käning
On July 6, 2014, 11:47 a.m., David Faure wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119142/
> -----------------------------------------------------------
>
> (Updated July 6, 2014, 11:47 a.m.)
>
>
> Review request for KDE Frameworks, Alex Merry and Kevin Ottens.
>
>
> Repository: kxmlgui
>
>
> Description
> -------
>
> KXMLGUI: load .rc files from DATADIR/kxmlgui5/component_name/file.rc
>
> The old files (DATADIR/component_name/file.rc) can still be loaded,
> and a runtime warning is shown in such a case, telling people to install
> into KXMLGUI_INSTALL_DIR (ECM patch posted to k-f-d)
>
> This keeps DATADIR slightly cleaner, getting rid of many application subdirs
> that would just contain a rc file. Some people seem to care about the cleanliness
> of their DATADIR :-)
>
>
> Diffs
> -----
>
> src/kedittoolbar.cpp 32512e9caccdbee1b07ff791fbecdcc817ca5726
> src/kxmlguiclient.cpp f7c90b261645dfe2efeb2edbc8f32cb134fc9b7d
> src/kxmlguifactory.cpp aa657bcc24a949f4350975609f19fae8b6c9a4f0
>
> Diff: https://git.reviewboard.kde.org/r/119142/diff/
>
>
> Testing
> -------
>
> Running kwrite unmodified -> works, with runtime warning
> Porting kwrite to KXMLGUI_INSTALL_DIR -> works, runtime warning is gone
>
> I also did the same with konsole, special case since it called setXMLFile("konsole/partui.rc"), had to be ported to setComponentName("konsole") + setXMLFile("partui.rc") while porting to KXMLGUI_INSTALL_DIR. This patch is still SC/BC, it's just a porting trap -- that affects none of the other existing kf5-based apps though.
>
>
> Thanks,
>
> David Faure
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140706/0ebcc4ed/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list