patch to add directory inclusion for pam config file

Bug #116846 reported by Rob Caskey
4
Affects Status Importance Assigned to Milestone
pam (Ubuntu)
Won't Fix
Undecided
Unassigned
Revision history for this message
Rob Caskey (rcaskey) wrote :
Revision history for this message
Martin Pitt (pitti) wrote :

Comments about the patch:
 * Evaluating the files in .d/ in reverse order is counterintuitive and error prone (on the side of the admin)
 * Since there is no patch system, the changelog should point out the modified files and functions (common practice: " * foo/foo.c, frobnicate(): blabla"), and explain the behavioural change of the patch in more detail. Same for the TeX transition; "Limit" is a bit misleading, what about "Build with texlive"; so -extra has never actually been necessary?
 * The changelog should close the bug (LP: #xxx) and refer to the spec

Since this patch alters the behaviour of a sensitive part of the system, it should be discussed with upstream and get their blessing (or, even better, adoption). However, see my comments in the spec, I'm not convinced that the patch in this form is sufficient for the spec.

Revision history for this message
xteejx (xteejx) wrote :

Has this been implemented yet? There has been no activity for almost 2 years, so marking Incomplete.

Changed in pam (Ubuntu):
status: New → Incomplete
Revision history for this message
Steve Langasek (vorlon) wrote :

I'm closing this bug as 'wontfix', actually, because I think pam-auth-update handles this use case and there's no need for directory inclusion in config file handling now.

Changed in pam (Ubuntu):
status: Incomplete → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.