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