After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 585307 - port to PolicyKit 1.0
port to PolicyKit 1.0
Status: RESOLVED FIXED
Product: gnome-applets
Classification: Other
Component: cpufreq
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gnome-applets Maintainers
gnome-applets Maintainers
: 585309 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2009-06-10 04:31 UTC by Matthias Clasen
Modified: 2009-08-13 09:52 UTC
See Also:
GNOME target: 2.28.x
GNOME version: ---


Attachments
patch (19.49 KB, patch)
2009-06-10 04:32 UTC, Matthias Clasen
reviewed Details | Review
new patch (22.42 KB, patch)
2009-07-13 13:58 UTC, Matthias Clasen
committed Details | Review

Description Matthias Clasen 2009-06-10 04:31:07 UTC
here is the patch we are using in Fedora
Comment 1 Matthias Clasen 2009-06-10 04:32:05 UTC
Created attachment 136248 [details] [review]
patch
Comment 2 Matthias Clasen 2009-06-12 18:40:02 UTC
*** Bug 585309 has been marked as a duplicate of this bug. ***
Comment 3 Callum McKenzie 2009-06-16 03:11:52 UTC
I'm happy with the patch, but the current version of policykit built by jhbuild is 0.9. Are there any compatibility problems between policykit 0.9 and 0.9.2? i.e. can the dependence be bumped without causing other modules to fail to compile?
Comment 4 Matthias Clasen 2009-06-16 03:33:01 UTC
it is 0.92 actually, not 0.9.2

and no, there is no real compatibility here. We break api compat once to get to 1.0 here. Anyway, hold off with committing the patch until we've discussed this dependency bump on d-d-l.
Comment 5 Matthias Clasen 2009-07-13 13:58:10 UTC
Created attachment 138328 [details] [review]
new patch

The first patch didn't really work, sorry. This one has been tested
Comment 6 André Klapper 2009-08-12 10:23:34 UTC
Can the patch please get in very soon so this receives some testing for 2.28?
Comment 7 Carlos Garcia Campos 2009-08-13 09:52:32 UTC
Applied a slightly modified version of the patch. Thanks!