Review Request 119596: Fix the upstart version check if the version is not a valid float
Harald Sitter
sitter at kde.org
Mon Aug 4 11:45:46 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119596/#review63760
-----------------------------------------------------------
Ship it!
Ship It!
- Harald Sitter
On Aug. 3, 2014, 9:28 p.m., Philip Muskovac wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119596/
> -----------------------------------------------------------
>
> (Updated Aug. 3, 2014, 9:28 p.m.)
>
>
> Review request for Plasma.
>
>
> Repository: powerdevil
>
>
> Description
> -------
>
> Recently upstart started using X.Y.Z versioning which breaks the version check as e.g. 1.13.1 is not a valid float anymore and gets translated to 0 by toFloat()
>
> Now the check only keeps the X.Y part of the version before the conversion.
>
> This needs to be applied in powerdevil and kde-workspace KDE/4.11.
>
>
> Diffs
> -----
>
> daemon/backends/upower/powerdevilupowerbackend.cpp ea1741f
>
> Diff: https://git.reviewboard.kde.org/r/119596/diff/
>
>
> Testing
> -------
>
> Tested on kubuntu utopic(dev) with powerdevil 5.0.0, kde-workspace 4.11.10 and upstart 1.13.1
>
>
> Thanks,
>
> Philip Muskovac
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140804/95b8d67d/attachment.html>
More information about the Plasma-devel
mailing list