Source-Changes-D archive

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

Re: CVS commit: src/etc/pam.d





On Dec 31, 2024, at 5:05 PM, Julio Merino <jmmv%NetBSD.org@localhost> wrote:

On Tue, Dec 31, 2024 at 03:27:49PM -0500, Christos Zoulas wrote:
Module Name: src
Committed By: christos
Date: Tue Dec 31 20:27:49 UTC 2024

Modified Files:
src/etc/pam.d: Makefile

Log Message:
Use a suffix rule.

Thanks for this. I suppose this fixes the problem that someone else
mentioned that caused my change to pollute the source tree :( ?

I would have just commented out the skey entry instead,
since it is rarely used and we already do that for kerberos. We could now
use MKKERBEROS to enable it by default, since we have the machinery...

I thought about doing this for pam_krb5 as well, but note a commit
from June 20th of 2023 to that directory that intentionally commented
those lines out even with MKKERBEROS=yes. Not sure it's a good idea
to do it.

Happy new year!

Happy New Year!

Matt fixed the problem by adding pam.d to the SUBDIRs in the Makefile above.
Yes, we should fix https://gnats.netbsd.org/57470 before enabling kerberos by default.

christos

Attachment: signature.asc
Description: Message signed with OpenPGP



Home | Main Index | Thread Index | Old Index