Re: CVS commit: pkgsrc/x11/xscreensaver

[Thomas Klausner <wiz%netbsd.org@localhost>]

Hi David!

On Fri, Feb 18, 2022 at 10:04:53PM -0500, David H. Gutteridge wrote:
> As for the PAM option, I think perhaps you missed the MESSAGE output
> that security/pam-pwauth_suid provides? It explains that it's necessary
> to place the configuration file. This isn't ideal, but it's a
> consequence of pkgsrc insisting files should not be installed directly
> under /etc. (x11/mate-screensaver, on the other hand, also has the same
> dependency, but breaks pkgsrc norms by reaching over and installing to
> /etc/pam.d directly. Then there's the whole topic of how the MESSAGE
> framework isn't ideal.)

Yes, I did miss that. Thanks for the pointer.

> Anyway, I'm not sure what you'd suggest we improve given these
> constraints? 6.02 with the PAM option enabled does work for me on
> NetBSD 9.99.93, I've just tested it.

I think the CONF_FILES mechanism can install files into /etc/pam.d.
The other question is if we want this to happen automatically though.

Given that you can't unlock the screen if you just enable the pam
option without manually installing a file, I think I'll leave the
'pam' option off by default.

Cheers,
 Thomas