Hi, here are some patches from Fedora that work on cvs HEAD, I guess the cfg patch is known by everyone and the manpage typo is also pretty obvious. The logfile patch is required on systems that use selinux. When the logfile is deleted, the selinux context is lost. Therefore the patch changes pm-utils not to remove the logfile.
Regards, Till
--- pm-utils-0.99.3/pm/functions.cfg 2007-05-29 15:40:55.000000000 +0200
+++ pm-utils-0.99.3/pm/functions 2007-05-29 15:41:22.000000000 +0200
@@ -24,7 +24,7 @@
{
cfgs="/etc/pm/config.d/*[^~]"
for cfg in $cfgs ; do
- [ -f $cfgs ] || continue
+ [ -f $cfg ] || continue
set -a
. $cfg
set +a
RedHat Bugzilla report: https://bugzilla.redhat.com/show_bug.cgi?id=286201 diff -up pm-utils-0.99.4/man/pm-pmu.sgml.manpage-typo pm-utils-0.99.4/man/pm-pmu.sgml --- pm-utils-0.99.4/man/pm-pmu.sgml.manpage-typo 2007-09-20 01:37:22.000000000 +0200 +++ pm-utils-0.99.4/man/pm-pmu.sgml 2007-09-20 01:37:06.000000000 +0200 @@ -37,7 +37,7 @@ <title>DESCRIPTION</title> <para>This manual page documents briefly the <command>&package;</command> command.</para> - <para><command>&package;</command> is a command line program to test whether the computer is running on line power</para> + <para><command>&package;</command> is a command line program to suspend a computer on machines using a Macintosh-style PMU</para> </refsect1> <refsect1> <title>EXIT CODES</title>
There is no need to remove the logfile, because the ">" after exec in the next line already truncates the logfile to zero length. Also removing is wrong, because it destroys the selinux context of the logfile. Bugzilla Reports: https://bugzilla.redhat.com/show_bug.cgi?id=237840 https://bugzilla.redhat.com/show_bug.cgi?id=238068 diff -up pm-utils-0.99.4/pm/functions.logfile pm-utils-0.99.4/pm/functions diff -up pm-utils-0.99.4/pm/functions.logfile pm-utils-0.99.4/pm/functions --- pm-utils-0.99.4/pm/functions.logfile 2007-05-15 16:55:58.000000000 +0200 +++ pm-utils-0.99.4/pm/functions 2007-09-20 00:53:51.000000000 +0200 @@ -137,7 +137,6 @@ do_suspend_hybrid() pm_main() { if [ -n "$PM_LOGFILE" ]; then - [ -f "$PM_LOGFILE" ] && rm -f "$PM_LOGFILE" exec > "$PM_LOGFILE" 2>&1 fi take_suspend_lock || exit 1
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ Pm-utils mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/pm-utils
