Review Request 119669: Fix broken creation of kstartupconfigfiles

Àlex Fiestas afiestas at kde.org
Mon Aug 11 08:03:40 UTC 2014


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

Ship it!


Once issue is fix, good to go!


startkde/kstartupconfig/kdostartupconfig.cpp
<https://git.reviewboard.kde.org/r/119669/#comment44890>

    Add const if string is not modified down the line.


- Àlex Fiestas


On ago. 8, 2014, 4:09 p.m., David Edmundson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119669/
> -----------------------------------------------------------
> 
> (Updated ago. 8, 2014, 4:09 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> Fix broken creation of kstartupconfigfiles
> 
> kstartupconfigfiles contains a list of files to compare mtimes on to see
> if we need to rebuild kstartupconfig. This wasn't being created
> correctly so we failed to rebuild if any configs updated.
> 
> This was broken in Qt5 porting:
> -        const QStringList dirs =
> KGlobal::dirs()->kfsstnd_prefixes().split( KPATH_SEPARATOR, QString::SkipEmptyParts);
> +        const QStringList dirs =
> QStandardPaths::standardLocations(QStandardPaths::GenericDataLocation);
> 
> 
> Diffs
> -----
> 
>   startkde/kstartupconfig/kdostartupconfig.cpp af6062c 
> 
> Diff: https://git.reviewboard.kde.org/r/119669/diff/
> 
> 
> Testing
> -------
> 
> Checked output of kstartupconfigfiles was sane.
> Updated ksplashrc, ran kstartupconfig5 (NOT kdostartupconfig5) checked kstartupconfig was regenerated.
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140811/28ff4114/attachment.html>


More information about the Plasma-devel mailing list