[Bug 688408] Review Request: xfce4-cpufreq-plugin - CPU frequency scaling plugin for the Xfce4 panel

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=688408

Christoph Wickert <cwickert@xxxxxxxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |cwickert@xxxxxxxxxxxxxxxxx

--- Comment #12 from Christoph Wickert <cwickert@xxxxxxxxxxxxxxxxx> 2011-03-22 11:59:10 EDT ---
Too bad I wasn't aware of this review. Next time you package something for
Xfce, please ping the Xfce SIG.

I packaged this plugin earlier, but we agreed to not include it in Fedora for
several reasons:
* Usually the power managers take care of scaling the CPU frequency.
* The plugin does not work with Fedora's default setup. Scaling is only allowed
for root or through PolicyKit, but the plugin doesn't support the latter. How
did you make the plugin work?
* There usually is no need for changing frequencies or the governor, ondemand
will handle this best.

Some notes on the package:
1) The description is formatted a little strange
2) "Requires: hicolor-icon-theme" is not needed because the plugin requires
gtk2 which already has a dependency to hicolor-icon-theme.
3) The Requires for xfce4-panel should be versioned and the version should be
adjusted for the different Fedora releases. A plugin built for Xfce4-panel
xfce4-panel 4.8 will not work with 4.6 or 4.4, even though the srpm is the
same.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]