Page MenuHomeFreeBSD

[New Port] sysutils/dwatch-gource
ClosedPublic

Authored by dteske on May 12 2018, 8:13 PM.
Tags
None
Referenced Files
F82573111: D15407.id42479.diff
Tue, Apr 30, 11:52 AM
Unknown Object (File)
Mon, Apr 29, 8:49 AM
Unknown Object (File)
Mon, Apr 29, 3:25 AM
Unknown Object (File)
Mon, Apr 29, 1:23 AM
Unknown Object (File)
Sat, Apr 27, 12:11 AM
Unknown Object (File)
Sat, Apr 27, 12:10 AM
Unknown Object (File)
Sat, Apr 27, 12:06 AM
Unknown Object (File)
Sat, Apr 27, 12:06 AM
Subscribers

Details

Summary

DTrace modules for FreeBSD dwatch(1) to produce gource logs

Diff Detail

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

Event Timeline

Add IGNORE_<OS>_<Version>

Remove unnecessary GROUPS

Update to dwatch-gource-0.2 adding gwatch utility

Update to 0.3 adding syscall visualizer

Update to 0.4 adding -Q/-m flags and profile-specific gource options

sysutils/dwatch-gource/Makefile
21 ↗(On Diff #42479)

To use PORTDOCS, you must have a DOCS option.

Update FreeBSD keyword in Makefile and URL in pkg-descr

Looks good to me. I don't understand mat@'s comment. It looks stale, but if not my approval is contingent on you resolving it.

This revision is now accepted and ready to land.May 13 2018, 4:20 PM
sysutils/dwatch-gource/Makefile
21 ↗(On Diff #42479)

Good catch. I copied a port that had a DOCS option. I'll remove this line.

Remove IGNORES and PORTDOCS

This revision now requires review to proceed.May 13 2018, 6:48 PM

Removed IGNORES (I've started to MFC the dependencies) and unused PORTDOCS

sysutils/dwatch-gource/Makefile
20–21 ↗(On Diff #42489)

You are installing some doc, which is not reflected in the plist, I doubt the framework did not say anything about a file missing.

If you install some doc, you must add a DOCS option, and either use the PORTDOCS macro or add the file to the plist guarded with the %%PORTDOCS%% macro.

Fix PORTDOCS, add DOCS option (thanks mat)

Put "FreeBSD" at the beginning of COMMENT to make portlint happy

portlint didn't like "dwatch" at start and can't just capitalize
the name of a utility to satisfy it, but this makes it happy.

Update sysutils/Makefile and provide more info in pkg-descr

This revision is now accepted and ready to land.May 14 2018, 10:02 PM
This revision was automatically updated to reflect the committed changes.