davidedmundson requested changes to this revision.
davidedmundson added a comment.
This revision now requires changes to proceed.


  Polkit-1 has the a concept of persistence, under a different name. A policy 
can be set to:
  
  - auth_admin
  - auth_admin_keep
  
  policy-gen-polkit1.cpp chooses between these using the persistence flag
  
    if (!action.persistence.isEmpty() && policy != QLatin1String("yes") && 
policy !=
            QLatin1String("no")) {
        policy += QLatin1String("_keep");
    }
  
  Which is saying if persistence is set to anything  and the policy is either 
"auth" or "auth_self"   append "_keep"
  
  Granted that doesn't match the current docs, but WRT 
https://phabricator.kde.org/D6198 we can delete that persistence attribute and 
it won't linger.

REPOSITORY
  R283 KAuth

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

To: elvisangelaccio, #frameworks, davidedmundson
Cc: davidedmundson

Reply via email to