fvogt requested changes to this revision.
fvogt added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> su:28
> +                self.state = State.GOOD
> +                call(["/usr/lib/x86_64-linux-gnu/libexec/kf5/kdesu_stub"])
> +                exit(0)

That won't work anywhere else and requires that the binary is installed when 
running tests.

> sudo:28
> +                self.state = State.GOOD
> +                call(["/usr/lib/x86_64-linux-gnu/libexec/kf5/kdesu_stub"])
> +                exit(0)

Same here.

REPOSITORY
  R299 KDESu

REVISION DETAIL
  https://phabricator.kde.org/D10716

To: jriddell, sitter, fvogt
Cc: fvogt, kde-frameworks-devel, michaelh, ngraham, bruns

Reply via email to