Page MenuHomeFreeBSD

security/doas: Match upstream version
ClosedPublic

Authored by jrm on Feb 18 2017, 2:57 PM.

Details

Summary

security/doas: Match upstream version (non-working example)

PR: 217176

Test Plan

portlint -C: OK
testport: OK (poudriere: 10.3-RELEASE-p10, i386)
testport: OK (poudriere: 10.3-RELEASE-p10, amd64)
testport: OK (poudriere: 11.0-RELEASE-p1, i386)
testport: OK (poudriere: 11.0-RELEASE-p1, amd64)

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 7596
Build 7748: arc lint + arc unit

Event Timeline

jrm retitled this revision from to security/doas: Match upstream version (non-working example).
jrm updated this object.
jrm edited the test plan for this revision. (Show Details)

Update distinfo for new upstream GH tag.

jrm retitled this revision from security/doas: Match upstream version (non-working example) to security/doas: Match upstream version.Feb 20 2017, 11:23 PM
jrm updated this object.

Note that this does not have reviewers, so, nobody is likely to review it ;-)

security/doas/Makefile
17

remove uselessly added tab.

I was just using the visual diff to discuss changes to a patch with a submitter.

Then you should probably have the discussion here, in public :-)

Updating license variables.

License situation:

Most of the code is under OpenBSD's ISC license.
There is some code from NetBSD under a 2-clause BSD license.
The author of the FreeBSD port, jsmith@resonatingmedia.com created this license file, which includes the same NetBSD license and a copyright notice with his contact information: https://github.com/slicer69/doas/blob/master/LICENSE

I guess the two lines below depend on what jsmith@resonatingmedia.com wants to do.

#LICENSE_FILE_BSD2CLAUSE= ${WRKSRC}/LICENSE...
#LICENSE_FILE_ISCL= ${WRKSRC}/LICENSE...

jrm added a reviewer: mat.
security/doas/Makefile
11–12

Those should be uncommented.

jrm edited the test plan for this revision. (Show Details)

Uncomment the LICENSE_FILE_BSD2CLAUSE and LICENSE_FILE_ISCL lines.

mat edited edge metadata.
mat added inline comments.
security/doas/Makefile
17

The extra tab should be removed.

This revision is now accepted and ready to land.Feb 22 2017, 2:11 PM
jrm marked an inline comment as done.Feb 22 2017, 2:19 PM

I suspect I am flirting with the boundaries of your patience, but could you elaborate why it makes sense to

LICENSE_FILE_BSD2CLAUSE=${WRKSRC}/LICENSE
LICENSE_FILE_ISCL=${WRKSRC}/LICENSE

when ${WRKSRC}/LICENSE only contains what it does?

security/doas/Makefile
11–12

Unocmmented, but this seems strange to me. That license file only contains the ISCL with the porter's copyright statement.

This revision was automatically updated to reflect the committed changes.
jrm marked an inline comment as done.