Created attachment 211829 [details]
This links the package with the godplugin. Netdata is in the process of migrating all python plugins to go (so default on option to future proof and help in the transition).
QA: portlint -AC
testport option on current amd64
Build info is available at https://gitlab.com/swills/freebsd-ports/pipelines/124236653
I'm a bit confused about what this is supposed to. Are you trying to provide a "simple" way of installing the go-plugins?
This makes the netdata port install netdata-go as dependency. Right now the python plugin is included in the netdata repo upstream by default. Meaning it is always installed when installing netdata. This is not the case for the god.plugin. And as netdata is in the progress to convert all python plugins to go plugins, it makes sense to enable the option by default.
It's more consistent to follow upstream rather than diverging and there are a lot cases where you don't need the go plugins at all also go is a pretty large dependency. Until upstream decides to fully change I think it's better to display a notice like other ports does (like when python extensions are available in the tree) for addtional funtionality via other ports.
We could do that! I don't really have any hidden agenda's although if we don't want to diverge from upstream we should actually enable it. On there github README, https://github.com/netdata/go.d.plugin, they explicitly state it should be shipped with netdata. "Install Shipped with Netdata." But at least I would like to have the option toggler thingy :-D, not necessarily enabled by default.