vde2: build with --disable-python
by default
#67452
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Motivation for this change
This package explicitly depends on
python2
which will be EOLed at theend of the year[1]. This package provides python bindings for
python2
,however the latest release (also used by other distros) is from 2011[2]
and doesn't support v3. For instance, debian ships
vde2
withoutpython2
support since Debian Jessie[3].KVM and QEMU appear to build fine, also NixOS tests and ISO builds are
still functional.
By running
nix-review
against this change, onlyxen
packages failed,but those were already broken on master[4].
Finally it's also worth mentioning that the closure size of
vde2
dropsfrom 99.5M to 33.5M without
python2
according tonix path-info -S -h
.[1] https://pythonclock.org/
[2] https://github.com/virtualsquare/vde-2/releases/tag/vde-2
(vde.sourceforge.net redirects to this github page)
[3] https://packages.debian.org/en/jessie/vde2
[4] https://hydra.nixos.org/build/99185451, https://hydra.nixos.org/build/99187262
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @