Needs porting to policykit-1

Bug #435709 reported by Martin Pitt
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
screen-resolution-extra (Ubuntu)
Fix Released
High
Martin Pitt

Bug Description

Binary package hint: screen-resolution-extra

s-r-e is one of two remaining packages in the Ubuntu default install which still uses the deprecated policykit 0.9 API. Please port to the new polkit-1 API.

This generally means to change the backend code to the new API, completely throw away everything PolicyKit related from the frontends (polkit daemon invokes authorization dialogs on its own now), and moving around the .policy file.

Please see https://wiki.ubuntu.com/DesktopTeam/PolicyKitOneMigration for the current status and links to API docs and migration guide.

Please see http://bazaar.launchpad.net/%7Ejockey-hackers/jockey/trunk/revision/562 for an example of another Python project (Jockey).

ProblemType: Bug
Architecture: amd64
CheckboxSubmission: 6923ba68f90064b9e2e670997b655a8d
CheckboxSystem: c8e8edcc4d15e0d55af04774be77e330
Date: Thu Sep 24 09:49:38 2009
DistroRelease: Ubuntu 9.10
Package: screen-resolution-extra 0.9
PackageArchitecture: all
ProcEnviron:
 PATH=(custom, user)
 LANG=de_DE.UTF-8
 SHELL=/bin/bash
ProcVersionSignature: Ubuntu 2.6.31-10.35-generic
SourcePackage: screen-resolution-extra
Uname: Linux 2.6.31-10-generic x86_64

Revision history for this message
Martin Pitt (pitti) wrote :
Changed in screen-resolution-extra (Ubuntu):
status: New → Triaged
Revision history for this message
Martin Pitt (pitti) wrote :

Oh, and I forgot: The current package does not depend on policykit at all. Please add a policykit-1 dependency if a package provides PK-protected services.

Martin Pitt (pitti)
Changed in screen-resolution-extra (Ubuntu):
assignee: nobody → Martin Pitt (pitti)
importance: Undecided → High
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package screen-resolution-extra - 0.10

---------------
screen-resolution-extra (0.10) karmic; urgency=low

  * policy-dontzap.py, policyui.py: Add missing hashbang lines (files are
    executable).
  * debian/control: Update Vcs-Bzr: to ubuntu-desktop branch.
  * Port to polkit-1: (LP: #435709):
    - screenresolution-mechanism.policy.in: auth_self_keep_always →
      auth_admin_keep.
    - setup.{py,cfg}: Install policy file into share/polkit-1/actions.
    - Drop ScreenResolution/policykit.py, and its usage in policyui.py and
      policy-dontzap.py; client-side handling of PK is not necessary any more.
    - screenresolution-mechanism.py: Drop PolicyKitAuthentication class, it's
      way too complicated.
    - screenresolution-mechanism.py, _check_permission(): Call polkit-1
      instead of old PolicyKit.
  * screenresolution-mechanism.py: Actually enforce privilege check by calling
    _check_permission() in the service methods.
  * debian/control: Add policykit-1 dependency.
  * debian/control: Bump Standards-Version to 3.8.3 (no changes necessary).

 -- Martin Pitt <email address hidden> Thu, 08 Oct 2009 14:02:54 +0200

Changed in screen-resolution-extra (Ubuntu):
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.