Page MenuHomeFreeBSD

Add new port devel/dep
ClosedPublic

Authored by sam_samgwydir.com on Dec 4 2017, 6:45 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 7, 10:09 PM
Unknown Object (File)
Wed, Nov 6, 10:46 AM
Unknown Object (File)
Tue, Nov 5, 2:15 PM
Unknown Object (File)
Tue, Nov 5, 12:10 PM
Unknown Object (File)
Thu, Oct 31, 3:56 PM
Unknown Object (File)
Tue, Oct 22, 12:27 AM
Unknown Object (File)
Thu, Oct 17, 4:09 PM
Unknown Object (File)
Thu, Oct 17, 1:50 PM

Details

Reviewers
None
Commits
rP458984: devel/dep: create port
Summary

Go dependency management tool

https://github.com/golang/dep

Test Plan

Installed a used as a dependency for sysutils/circonus-agent.

Diff Detail

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

Event Timeline

Does it pass poudriere testport?

devel/dep/Makefile
1 ↗(On Diff #36198)

You're missing # $FreeBSD$ at the top.

2 ↗(On Diff #36198)

Use DISTVERSION

7 ↗(On Diff #36198)

This cannot have passed portlint. Remove dep is a, the dot at the end, and capitalize prototype.

9 ↗(On Diff #36198)

Missing LICENSE_FILE

11 ↗(On Diff #36198)

Why? You USES=go which adds it already.

devel/dep/pkg-descr
1 ↗(On Diff #36198)

Please make it at least a paragraph or two. It should be more than just a repetition of COMMENT.

devel/dep/Makefile
7 ↗(On Diff #36198)

Agreed on everything else but capitalize prototype?

devel/dep/Makefile
7 ↗(On Diff #36198)

Yes, COMMENTs always start with a captial letter.

https://www.freebsd.org/doc/en/books/porters-handbook/makefile-comment.html

COMMENT=        Prototype dependency management tool for Go
sam_samgwydir.com marked 8 inline comments as done.
sam_samgwydir.com marked an inline comment as not done.

Completed your requests -- lengthened pkg-descr but I have someone writing a better one up for me. That should be here soon.

This passes poudriere testport.

This revision was not accepted when it landed; it landed in state Needs Review.Jan 14 2018, 5:25 PM
Closed by commit rP458984: devel/dep: create port (authored by swills). · Explain Why
This revision was automatically updated to reflect the committed changes.