[pbs-devel] applied: [PATCH proxmox] build: use weak and namespaced features

Thomas Lamprecht t.lamprecht at proxmox.com
Fri Aug 26 13:40:59 CEST 2022


Am 22/08/2022 um 11:24 schrieb Fabian Grünbichler:
> 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 at proxmox.com>
> ---
> sent as patch to wait for cargo 0.63 regressions before applying
> 
>  proxmox-http/Cargo.toml         | 26 ++++++++++++++------------
>  proxmox-router/Cargo.toml       |  2 +-
>  proxmox-schema/Cargo.toml       |  4 ++--
>  proxmox-subscription/Cargo.toml |  2 +-
>  proxmox-sys/Cargo.toml          |  4 ++--
>  proxmox-tfa/Cargo.toml          |  6 +++---
>  6 files changed, 23 insertions(+), 21 deletions(-)
> 
>

applied, thanks!





More information about the pbs-devel mailing list