security/sudo: Remove dependency on security/sssd security/sssd was removed from ports recently, which broke the SSSD option. Let's remove the SSSD option and let users use SSSD2 instead. PORTREVISION has been bumped so it is easier to tell which version of the sudo package supports only sssd2. PR: 283952 Sponsored by: Klara, Inc. Approved by: 0mp (mentor) Approved by: garga (maintainer)
Details
Details
- Reviewers
garga 0mp - Commits
- R11:6971a62a23da: security/sudo: Remove dependency on security/sssd
Port has been portlinted and portclippy(ed).
Tested on poudriere with SSSD2 set or unset
Diff Detail
Diff Detail
- Repository
- R11 FreeBSD ports repository
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Comment Actions
Suggestions for the commit message:
security/sudo: Remove broken option SSSD security/sssd was removed from ports recently, which broke the SSSD option. Let's remove the SSSD option and let users use SSSD2 instead. PORTREVISION has been bumped so it is easier to tell which version of the sudo package supports only sssd2. PR: 283952 Sponsored by: Klara, Inc. Approved by: 0mp (mentor)
The following comment:
On this patch SSSD2 config knob is still used. Maybe we should consider to renaming to SSSD ?
is targeted at the maintainer and should be addressed before the commit lands upstream.
Comment Actions
Otherwise, the commit looks good. Ping me again for the final review when you get the approval of the maintainer (or a timeout).