Page MenuHomeFreeBSD

libpam: Move to a new "pam" package
ClosedPublic

Authored by ivy on Wed, Nov 5, 11:30 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 7, 5:41 PM
Unknown Object (File)
Fri, Nov 7, 12:25 PM
Unknown Object (File)
Fri, Nov 7, 7:15 AM
Unknown Object (File)
Fri, Nov 7, 2:25 AM
Unknown Object (File)
Thu, Nov 6, 5:59 PM
Unknown Object (File)
Thu, Nov 6, 11:52 AM
Unknown Object (File)
Thu, Nov 6, 9:22 AM
Unknown Object (File)
Thu, Nov 6, 8:01 AM
Subscribers

Details

Summary

OpenPAM is a discrete, largely self-contained system component.
Users may not need PAM for many use-cases (e.g. jails, containers),
so move it to its own package.

Use LIB_PACKAGE to create a separate pam-lib package for libpam,
so that applications that support PAM don't need to bring in all
the PAM modules if PAM isn't actually in use.

Add pam to the minimal sets, since this is a core system component that
people expect to be installed. This means all supported installation
methods will install the PAM modules by default, so don't add explicit
dependencies on the PAM modules from things that use PAM (e.g. runtime),
allowing custom/embedded systems to omit these easily.

This change adds a new package to the system so, until we have a proper
policy on how to handle this in release/stable branches, it should not
be MFC'd.

MFC after: never

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 68440
Build 65323: arc lint + arc unit

Event Timeline

ivy requested review of this revision.Wed, Nov 5, 11:30 PM

fix the UPDATING text to match the use of LIB_PACKAGE

lib/libpam/Makefile.inc
26

Is aything below lib/libpam not in the pam package?

lib/libpam/pam.d/Makefile
21
25
lib/libpam/Makefile.inc
26

no (at least not that i can find) but i prefer to use ?= as it's less error-prone. for example, if someone adds tests and forgets to change = to ?= then the tests will end up in the wrong package.

lib/libpam/pam.d/Makefile
21

i don't think this is right: ${MK_AT} and ${MK_FTP} are both "yes", and assigning to CONFGROUPS.yes twice with = will mean only one group is built.

This revision is now accepted and ready to land.Sun, Nov 9, 1:27 PM
This revision was automatically updated to reflect the committed changes.