]> git.proxmox.com Git - proxmox.git/commit - proxmox-http/Cargo.toml
build: use weak and namespaced features
authorFabian Grünbichler <f.gruenbichler@proxmox.com>
Mon, 22 Aug 2022 09:24:04 +0000 (11:24 +0200)
committerThomas Lamprecht <t.lamprecht@proxmox.com>
Fri, 26 Aug 2022 08:35:00 +0000 (10:35 +0200)
commit289d297c7d14c5d7899be5a66e9fb7f0a1eba344
treeb26fd873659ff6a50086c35bec29b2e5c4862afd
parent1cd6a842f76d6877292a8810c4cf1248b7d2fc45
build: use weak and namespaced features

to reduce the creep of optional dependencies being exposed as features.

this requires cargo 0.63 (and debcargo built against at least that
version), but greatly reduces the number of binary packages and provides
generated, while still allowing sensible selection of optional
dependencies via the explicit feature meant for pulling them in.

diff stat for running `make  deb` after this change:
 proxmox-http/debian/control         | 226 ++++--------------------------------
 proxmox-router/debian/control       |  74 +-----------
 proxmox-schema/debian/control       |  53 ++-------
 proxmox-subscription/debian/control |  17 +--
 proxmox-sys/debian/control          |  51 +++-----
 proxmox-tfa/debian/control          | 110 ++----------------
 6 files changed, 72 insertions(+), 459 deletions(-)

the 'dep:' prefix marks something on the RHS inside the features section
as dependency, it's only allowed if the string after it is an optional
dependency an no explicit feature of the same name exists. if all
pointers to the optional dependency in the features section are marked
as such, the optional dependency itself will not be exposed as a feature
(either on the cargo or debian/control level).

the '?' suffix marks dependencies as "weak", which only enables the
optional dependency + its feature(s) if the optional dependency itself
is also enabled. it has no effect on d/control since such a relationship
is not encodable in Debian package relations, but it does affect cargo
dependency resolution and allows skipping the build of unneeded optional
dependencies in some cases.

with no packages/crates depending on the no longer exposed automatically
generated features/packages, so these are safe to remove even though
it's technically a breaking change.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
proxmox-http/Cargo.toml
proxmox-router/Cargo.toml
proxmox-schema/Cargo.toml
proxmox-subscription/Cargo.toml
proxmox-sys/Cargo.toml
proxmox-tfa/Cargo.toml