Review Request 119142: KXMLGUI: load .rc files from DATADIR/kxmlgui5/component_name/file.rc

David Faure faure at kde.org
Sun Jul 6 15:32:58 UTC 2014


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

(Updated July 6, 2014, 3:32 p.m.)


Review request for KDE Frameworks, Alex Merry and Kevin Ottens.


Changes
-------

fixed+extended apidox


Repository: kxmlgui


Description (updated)
-------

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 :-)

REVIEW: 119142


Diffs (updated)
-----

  src/kedittoolbar.cpp 32512e9caccdbee1b07ff791fbecdcc817ca5726 
  src/kxmlguiclient.h 2331b6934f99528c5d266c92d78a8affaccad2a0 
  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/192fd6aa/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list