Review Request 119895: New class for 5.2 to help user to migrate config files and ui file to new standardpath

[Kevin Krammer]

On Aug. 22, 2014, 9:39 vorm., Laurent Montel wrote:
> > Just a general question: do we really want a porting class in core addons?
> 
> Laurent Montel wrote:
>     Kdelibs4Migration is already in this addons.
>     Where do you want to put it ? In which module ?

I just found it strange.
To me it is like having Qt3Support in QtCore. But I don't know what the goal of KDE core addons is, so providing compatibility stuff might very well be part of its scope.


- Kevin


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


On Aug. 22, 2014, 9:05 vorm., Laurent Montel wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119895/
> -----------------------------------------------------------
> 
> (Updated Aug. 22, 2014, 9:05 vorm.)
> 
> 
> Review request for KDE Frameworks, David Faure and Kevin Krammer.
> 
> 
> Repository: kcoreaddons
> 
> 
> Description
> -------
> 
> Class helps user to migrate config file and ui file to new location
> 
> 
> Diffs
> -----
> 
>   src/lib/util/kdelibs4configmigrator.cpp PRE-CREATION 
>   autotests/CMakeLists.txt 75d1293 
>   autotests/data/appui1rc PRE-CREATION 
>   autotests/data/appuirc PRE-CREATION 
>   autotests/data/foo1rc PRE-CREATION 
>   autotests/data/foorc PRE-CREATION 
>   autotests/kdelibs4configmigratortest.cpp PRE-CREATION 
>   src/lib/CMakeLists.txt 26eb5a1 
>   src/lib/util/kdelibs4configmigrator.h PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/119895/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Laurent Montel
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140822/d674229c/attachment.html>