Review Request 121303: detect kde4 home better in kde4 migration

Xuetian Weng wengxt at gmail.com
Thu Dec 4 08:51:29 UTC 2014



> On Dec. 4, 2014, 8:42 a.m., David Faure wrote:
> > src/lib/util/kdelibs4migration.cpp, line 52
> > <https://git.reviewboard.kde.org/r/121303/diff/1/?file=331217#file331217line52>
> >
> >     These 3 lines could be removed then - if KDE4_DEFAULT_HOME doesn't exist, there isn't much point in pointing to it.
> >     
> >     But ok it doesn't change anything in practice, kdeHomeFound() tests for exists anyway.

Emm. but the reason that KDE4_DEFAULT_HOME being introduced: https://git.reviewboard.kde.org/r/120480/diff/ is that it would be better to leave it non-empty. (in case user install some kde4 app later).


- Xuetian


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


On Dec. 4, 2014, 8:22 a.m., Xuetian Weng wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/121303/
> -----------------------------------------------------------
> 
> (Updated Dec. 4, 2014, 8:22 a.m.)
> 
> 
> Review request for KDE Frameworks, David Faure and Laurent Montel.
> 
> 
> Repository: kcoreaddons
> 
> 
> Description
> -------
> 
> some really old distribution might use ~/.kde for kde3 and ~/.kde4 for kde4, and in some other case ~/.kde might be existed for random reason.
> 
> This change will change the check order to KDE4_DEFAULT_HOME (from cmake option), .kde4, .kde. So distro can always specify their correct one as the first one. And check ~/.kde4 before ~/.kde seems to be a saner option because they must use "4" for a good reason.
> 
> 
> Diffs
> -----
> 
>   src/lib/util/kdelibs4migration.cpp d8d564a 
> 
> Diff: https://git.reviewboard.kde.org/r/121303/diff/
> 
> 
> Testing
> -------
> 
> return ~/.kde4 first even ~/.kde exists.
> 
> 
> Thanks,
> 
> Xuetian Weng
> 
>

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


More information about the Kde-frameworks-devel mailing list