fvogt requested changes to this revision. fvogt added a comment. This revision now requires changes to proceed.
Unfortunately, this breaks public API and ABI by modifying KAuth::ActionReply. I'm not sure whether there has to be any compatibility for the DBus API, but I guess not (logging out and back in should suffice to make sure the backend used is the same). I wonder why you made a wrapper around `QDBusUnixFileDescriptor`, does it not work inside a `QVariant` OOTB? REPOSITORY R283 KAuth REVISION DETAIL https://phabricator.kde.org/D24245 To: volkov, fvogt, chinmoyr, cfeck, #frameworks Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns