net-p2p/transmission: Reorganize as a metaport

[mondo.debater_0q_icloud.com]

Summary

Fixes: Bugs 273841, 274054
Port reorganized as a metaport. Each component can be installed
separately or selected as an option via the metaport.
See MOVED for additional information.

Also fixes log-level typo in rc file.

Test Plan

All flavors and options tested in 13.2amd64.

[vsasjason_gmail.com]

So, initial thoughts on rework:

After carefully reading bsd.options.mk I came to following coclusions (but I may be wrong):

1. OPTIONS_FILE being crafted from <category>/<portname>, which means we can't really define any options inside flavors, but in root of Makefile only (to prevent overwriting OPTIONS_FILE by different flavors.
2. There are couple of options especially useful for slave ports (and I believe flavors): OPTIONS_EXCLUDE, OPTIONS_SLAVE.

Thus, I propose trying something like:

OPTIONS_DEFINE=DAEMON NLS UTILS ...

*_DEPENDS=<whatever needed by every component>

.if ${FLAVOR} == daemon
OPTIONS_SLAVE=DAEMON
OPTIONS_EXCLUDE=<whatever isn't applicable for daemon, GUI obviously>

<USE_RC_SUBR>, <CMAKE_*>, <*_DEPENDS>, <whatever else applicable for daemon>
.endif

.if ${FLAVOR} == <...>
.endif

Thus, we will achieve:

1. Configuring and installing net-p2p/transmission will allow user to select any components combination needed.
2. Installing net-p2p/transmission@<flavor> will allow modular installation.

So, just one port which covers everything without a need for separate metaport? I'm unsure where to put DOCS option though. It would be nice to install it with any of flavors, but there will be a conflict then...
Also, it seems reasonable to me install GUI not as bin/transmission-{gtk|qt}, but as bin/transmission (of course, then respective OPTION should be changed to RADIO and conflicts between default, gtk, and qt flavors should be registered.

[andrew_tao173.riddles.org.uk]

In D41975#957578, @vsasjason_gmail.com wrote:

So, initial thoughts on rework:

After carefully reading bsd.options.mk I came to following coclusions (but I may be wrong):

1. OPTIONS_FILE being crafted from <category>/<portname>, which means we can't really define any options inside flavors, but in root of Makefile only (to prevent overwriting OPTIONS_FILE by different flavors.

Take a look at devel/git or mail/postfix which override OPTIONS_FILE to allow different options for flavors.

[mondo.debater_0q_icloud.com]

In D41975#957578, @vsasjason_gmail.com wrote:

After carefully reading bsd.options.mk I came to following coclusions (but I may be wrong):

1. OPTIONS_FILE being crafted from <category>/<portname>, which means we can't really define any options inside flavors, but in root of Makefile only [...]

I'm unsure where to put DOCS option though. It would be nice to install it with any of flavors, but there will be a conflict then...
Also, it seems reasonable to me install GUI not as bin/transmission-{gtk|qt}, but as bin/transmission (of course, then respective OPTION should be changed to RADIO and conflicts between default, gtk, and qt flavors should be registered.

Your suggestion would go back to the problem earlier versions of the 4.0 port had, which is that going from a full install to a partial install (or vice versa) would require a wholesale uninstall and reinstall, instead of merely adding or removing the selected components. While I initially disfavored a metaport, I do have to agree that this is a meaningful benefit.

It seems ridiculous for the docs to be a separate component package, which is why I stuck them in what I figured was the most likely used component.

Prior to the 4.0 update, the default install of the metaport installed all slave ports, including both GTK and QT. The updated port has aimed to preserve this setup.

[mondo.debater_0q_icloud.com]

Updating D41975: net-p2p/transmission: Reorganize as a metaport

1. Added flavor-specific options files, fixed option descriptions.

[vsasjason_gmail.com]

In D41975#957656, @andrew_tao173.riddles.org.uk wrote:

In D41975#957578, @vsasjason_gmail.com wrote:

So, initial thoughts on rework:

After carefully reading bsd.options.mk I came to following coclusions (but I may be wrong):

1. OPTIONS_FILE being crafted from <category>/<portname>, which means we can't really define any options inside flavors, but in root of Makefile only (to prevent overwriting OPTIONS_FILE by different flavors.

Take a look at devel/git or mail/postfix which override OPTIONS_FILE to allow different options for flavors.

Could you please suggest Makefile line number for devel/git? I don't see that. But I actually see OPTIONS_EXCLUDE/OPTIONS_SLAVE, what was a proposal right after text cited by you.

[andrew_tao173.riddles.org.uk]

In D41975#957684, @vsasjason_gmail.com wrote:

In D41975#957656, @andrew_tao173.riddles.org.uk wrote:.

Take a look at devel/git or mail/postfix which override OPTIONS_FILE to allow different options for flavors.

Could you please suggest Makefile line number for devel/git? I don't see that.

https://cgit.freebsd.org/ports/tree/devel/git/Makefile#n86

[vsasjason_gmail.com]

Could you please suggest Makefile line number for devel/git? I don't see that.

https://cgit.freebsd.org/ports/tree/devel/git/Makefile#n86

Thank you, I missed that somehow. Will take a deeper look bit later.

[fuz]

LGTM but for the two minor issues I noted.

Please also check if you can move the documentation to a separate flavor of net-p2p/transmission-components. It is currently not obvious which flavour installs the documentation.

UPDATING

13

The UPDATING file is primarily for people who build from source. So please mention the complete category/portname@flavor here. The package name alone may be sightly confusing.

net-p2p/transmission-components/Makefile

2–3

PORTREVISION should be cleared when making a new port.

net-p2p/transmission-components/pkg-descr

1 ↗

(On Diff #127871)

Also consider using a separate pkg-descr for each flavour. You can use

DESCR= ${PKGDIR}/pkg-descr.${FLAVOR}

to have a separate pkg-descr for each one.

[mondo.debater_0q_icloud.com]

Updating D41975: net-p2p/transmission: Reorganize as a metaport

1. -Added docs flavor -Revised per fuz

[fuz]

LGTM, will commit later today.

[fuz]

Landed as ac3e39992e95659a5193a15d4cb873efb00fcb30.