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 ?
> 
> Kevin Krammer wrote:
>     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.
> 
> David Faure wrote:
>     Well, it's not the same. You want to be able to port away from Qt3support / kdelibs4support at some point, to stop linking to it.
>     
>     On the other hand, you need to keep calling kdelibs4migrator for the entire 5.x lifetime, since you can't know at which point the last users will switch. So you don't want such code in a compatibility library that you're trying to stop linking to.

Well, that is only the case for applications which used to use kdelibs in their Qt4 version.
It is just a single class for now, but if we also want to support data migration at some point this is going to need asynchronous copying, etc. adding to the complexity of a library targetted at more than just ported KDE applications, no?


- Kevin


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


On Aug. 22, 2014, 10:55 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, 10:55 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
> -----
> 
>   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 
>   src/lib/util/kdelibs4configmigrator.cpp 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/e3f4d539/attachment-0001.html>