Review Request 121303: detect kde4 home better in kde4 migration

[David Faure]

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

Ship it!



src/lib/util/kdelibs4migration.cpp
<https://git.reviewboard.kde.org/r/121303/#comment49804>

    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.


- David Faure


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/c6b38d07/attachment.html>