Page MenuHomeFreeBSD

security/pam_krb5: Update to 4.8
ClosedPublic

Authored by woodsb02 on Feb 16 2019, 4:01 AM.
Tags
None
Referenced Files
F80138143: D19207.id53982.diff
Thu, Mar 28, 11:00 AM
Unknown Object (File)
Jan 24 2024, 3:56 AM
Unknown Object (File)
Jan 17 2024, 5:16 AM
Unknown Object (File)
Dec 25 2023, 6:58 PM
Unknown Object (File)
Dec 25 2023, 6:58 PM
Unknown Object (File)
Dec 25 2023, 6:58 PM
Unknown Object (File)
Dec 25 2023, 6:46 PM
Unknown Object (File)
Dec 23 2023, 1:27 PM
Subscribers

Details

Summary

security/pam_krb5: Update to 4.8

  • Add LICENSE details
  • Pet portlint (Makefile order)

Approved by: cy (maintainer)

Test Plan

poudriere testport success on 13amd64
Runtime testing also worked fine

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

cy requested changes to this revision.Feb 16 2019, 4:57 AM

Looks good except the update, license and portlint changes need to be three separate commits. I will approve if you keep the three functional commits separate, clean easy to follow logs.

security/pam_krb5/Makefile
5 ↗(On Diff #53982)

Separate commit, please.

16 ↗(On Diff #53982)

Separate commit, please.

This revision now requires changes to proceed.Feb 16 2019, 4:57 AM
security/pam_krb5/Makefile
28 ↗(On Diff #53982)

That variable is not a framework variable, so it should go after the options, see Chapter 15. Order of Variables in Port Makefiles.

That being said, it is also never used, so it should probably be removed.

As discussed, I've found the time this morning and will commit.

security/pam_krb5/Makefile
28 ↗(On Diff #53982)

It's been removed and committed to my git repo.

This revision was not accepted when it landed; it landed in state Needs Revision.Feb 23 2019, 4:46 PM
Closed by commit rP493708: Add LICENSE details. (authored by cy). · Explain Why
This revision was automatically updated to reflect the committed changes.

Done. That's the beauty of git svn.