Re: CVS commit: src/usr.sbin/powerd

[Paul Goyette <paul%whooppee.com@localhost>]

The powerd(8) man page currently says

        The first location is /etc/powerd/scripts/<power_type>, where
        <power_type> is defined by the power management mechanism
        supported by the system, e.g., ``apm'' or ``acpi''.  If the
        script is not found in the first location, powerd looks in
        /etc/powerd/scripts.

This implies that the .../apm/... and .../acpi/... are optional.  Yet 
src/usr.sbin/powerd.c has not been updated for more than two years, and 
it defines only a single path for locating the scripts.

        #define _PATH_POWERD_SCRIPTS    "/etc/powerd/scripts"

        <snip>

        static const char *script_paths[] = {
                NULL,
                _PATH_POWERD_SCRIPTS
        };


On Tue, 26 Jan 2010, Jonathan A. Kollasch wrote:

> On Tue, Jan 26, 2010 at 08:37:13PM +0000, Jukka Ruohonen wrote:
> >  * Apparently there is only a single location for the scripts. Thus, remove
> >    the references to "/etc/powerd/scripts/apm" and "/etc/powerd/scripts/acpi".
>
> When did this change?  I remember fixing this in the acpi case recently.
>
>         Jonathan Kollasch

-------------------------------------------------------------------------
|   Paul Goyette   | PGP DSS Key fingerprint: |  E-mail addresses:      |
| Customer Service | FA29 0E3B 35AF E8AE 6651 |  paul at whooppee.com   |
| Network Engineer | 0786 F758 55DE 53BA 7731 | pgoyette at juniper.net |
| Kernel Developer |                          | pgoyette at netbsd.org  |
-------------------------------------------------------------------------