Page MenuHomeFreeBSD

devel/freebsd-git-mfc: New port
ClosedPublic

Authored by des on Tue, Jun 30, 10:36 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Jul 7, 7:27 AM
Unknown Object (File)
Mon, Jul 6, 9:34 PM
Unknown Object (File)
Mon, Jul 6, 8:17 PM
Unknown Object (File)
Sun, Jul 5, 3:14 PM
Unknown Object (File)
Sun, Jul 5, 7:43 AM
Unknown Object (File)
Sun, Jul 5, 7:04 AM
Unknown Object (File)
Sun, Jul 5, 6:36 AM
Unknown Object (File)
Sat, Jul 4, 6:31 PM
Subscribers

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 74557
Build 71440: arc lint + arc unit

Event Timeline

des requested review of this revision.Tue, Jun 30, 10:36 PM
des created this revision.

Seems fine to me, thanks for creating the port.

devel/freebsd-git-mfc/pkg-descr
2
4
des marked 2 inline comments as done.Wed, Jul 1, 8:34 AM
fuz added inline comments.
devel/freebsd-git-mfc/Makefile
4–5

Not a huge fan of this as it goes against the rules of using USE_GITHUB for this sort of stuff, but given the large size of the system sources it might make sense.

Also consider having the script live in /usr/ports/devel/freebsd-git-mfc/files instead so we don't need to fetch it at all.

29

Might be easier to just have EXTRACT_ONLY= instead of a do-extract target. Then copy the files directly from _DISTDIR in do-install.

32

(if the previous suggestion is not taken)

devel/freebsd-git-mfc/Makefile
29

No, this port has a build stage, the script needs to transit through ${WRKSRC}.

devel/freebsd-git-mfc/Makefile
29

It does not have a build stage, you set NO_BUILD=yes.

devel/freebsd-git-mfc/Makefile
29

Sorry, I meant it has a patch stage.

LGTM, but please consider the _DISTDIR change.

devel/freebsd-git-mfc/Makefile
29

Right, that makes sense (for shebangfix). Very well then.

This revision is now accepted and ready to land.Tue, Jul 7, 11:20 AM
This revision was automatically updated to reflect the committed changes.