----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115271/#review48157 -----------------------------------------------------------
Are the systemd developers aware of this incompatibility? If so, do they give a reason? powerdevil/daemon/backends/upower/powerdevilupowerbackend.cpp <https://git.reviewboard.kde.org/r/115271/#comment34065> I think it would be better to read the string property into a local variable and then perform the two checks. - Kevin Krammer On Jan. 23, 2014, 6:56 p.m., Raymond Wooninck wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/115271/ > ----------------------------------------------------------- > > (Updated Jan. 23, 2014, 6:56 p.m.) > > > Review request for Solid and Lukáš Tinkl. > > > Repository: kde-workspace > > > Description > ------- > > The upcoming version of systemd (currently git master) seems to have its > version string changed. Up to version 208, the string returned was "systemd > xxx". With the current git master, only the version is returned. The version > check within powerdevil is still expecting to get "systemd xxx" back and > therefore fails with the systemd git master. > > The patch checks first if the "systemd xxx" string is returned and if not, it > checks if just "xxx" is returned. > > > Diffs > ----- > > powerdevil/daemon/backends/upower/powerdevilupowerbackend.cpp eae5436 > > Diff: https://git.reviewboard.kde.org/r/115271/diff/ > > > Testing > ------- > > Testing done on a system with systemd git master and there the suspend and > hibernate options are available again. > > > Thanks, > > Raymond Wooninck > >
_______________________________________________ Kde-hardware-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-hardware-devel
