Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

lxc, waydroid not installable / qubes-kernel-vm-support has unnecessary dependency on busybox #8377

Open
adrelanos opened this issue Jul 30, 2023 · 5 comments · May be fixed by QubesOS/qubes-linux-utils#101
Labels
affects-4.2 This issue affects Qubes OS 4.2. C: Debian/Ubuntu needs diagnosis Requires technical diagnosis from developer. Replace with "diagnosed" or remove if otherwise closed. P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. pr submitted A pull request has been submitted for this issue. T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists.

Comments

@adrelanos
Copy link
Member

Qubes OS release

R4.2

Brief summary

Attempting to install lxc or waydroid results in removal of of qubes-kernel-vm-support.

Steps to reproduce

sudo apt install lxc

Or.

Install waydroid.

Expected behavior

No such issue.

Actual behavior

https://forums.whonix.org/uploads/default/original/2X/1/17836bcc73cb49eb38f84da9dfea5a6915f23f7d.png

References

Additional information

Does qubes-kernel-vm-support really need to depend on busybox?

Would Depends: busybox | busybox-static be an acceptable alternative?

@adrelanos adrelanos added P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists. labels Jul 30, 2023
@DemiMarie DemiMarie self-assigned this Jul 30, 2023
@DemiMarie DemiMarie added this to the Release 4.2 milestone Jul 30, 2023
DemiMarie added a commit to DemiMarie/qubes-linux-utils that referenced this issue Jul 30, 2023
This ensures that installing a package that requires busybox-static,
such as waydroid or lxc, does not cause qubes-kernel-vm-support to be
removed.

Reported-by: Patrick Schleizer <adrelanos@whonix.org>
Fixes: QubesOS/qubes-issues#8377
@andrewdavidwong andrewdavidwong added C: other diagnosed Technical diagnosis has been performed (see issue comments). pr submitted A pull request has been submitted for this issue. labels Jul 31, 2023
@marmarek
Copy link
Member

Would Depends: busybox | busybox-static be an acceptable alternative?

I'm okay with this change, but I'm confused why it's necessary. busybox-static has Provides: busybox, so it shouldn't matter, no?

@marmarek
Copy link
Member

In fact, the very similar issue was motivation of including that Provides: busybox - https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=375568.

@marmarek
Copy link
Member

I just checked on my debian-12 system, the Provides: busybox does seem to work as expected:

user@d12test3:~$ dpkg -l qubes-kernel-vm-support
Desired=Unknown/Install/Remove/Purge/Hold
| Status=Not/Inst/Conf-files/Unpacked/halF-conf/Half-inst/trig-aWait/Trig-pend
|/ Err?=(none)/Reinst-required (Status,Err: uppercase=bad)
||/ Name                    Version        Architecture Description
+++-=======================-==============-============-=======================>
ii  qubes-kernel-vm-support 4.2.11+deb12u1 amd64        Qubes VM kernel and ini>
user@d12test3:~$ sudo apt install lxc
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
The following additional packages will be installed:
  arch-test binutils binutils-common binutils-x86-64-linux-gnu busybox-static
  cloud-image-utils debootstrap distro-info fakechroot genisoimage libbinutils
  libctf-nobfd0 libctf0 libdistro-info-perl libdpkg-perl libfakechroot
  libfile-fcntllock-perl libgprofng0 liblxc-common liblxc1 libpam-cgfs
  libsubid4 lxc-templates lxcfs mmdebstrap rsync uidmap uuid-runtime wget
Suggested packages:
  binutils-doc mtools squid-deb-proxy-client ubuntu-archive-keyring shunit2
  wodim cdrkit-doc debian-keyring gcc | c-compiler bzr btrfs-progs python3-lxc
  qemu-user-static apt-transport-tor dpkg-dev genext2fs perl-doc qemu-user
  squashfs-tools-ng openssh-server python3-braceexpand
The following packages will be REMOVED:
  busybox
The following NEW packages will be installed:
  arch-test binutils binutils-common binutils-x86-64-linux-gnu busybox-static
  cloud-image-utils debootstrap distro-info fakechroot genisoimage libbinutils
  libctf-nobfd0 libctf0 libdistro-info-perl libdpkg-perl libfakechroot
  libfile-fcntllock-perl libgprofng0 liblxc-common liblxc1 libpam-cgfs
  libsubid4 lxc lxc-templates lxcfs mmdebstrap rsync uidmap uuid-runtime wget
0 upgraded, 30 newly installed, 1 to remove and 7 not upgraded.
Need to get 14.6 MB of archives.
After this operation, 77.8 MB of additional disk space will be used.

As you can see, only busybox is going to be removed, but not qubes-kernel-vm-support. Same happens on Whonix Workstation 17.

So, I cannot reproduce the issue.

@DemiMarie
Copy link

Is this issue specific to Debian oldstable/Whonix 16?

@andrewdavidwong andrewdavidwong added needs diagnosis Requires technical diagnosis from developer. Replace with "diagnosed" or remove if otherwise closed. C: Debian/Ubuntu and removed diagnosed Technical diagnosis has been performed (see issue comments). C: other labels Aug 1, 2023
@adrelanos
Copy link
Member Author

Is this issue specific to Debian oldstable/Whonix 16?

Could be.

So, I cannot reproduce the issue.

Confirmed for Whonix 17.

@andrewdavidwong andrewdavidwong added the affects-4.2 This issue affects Qubes OS 4.2. label Aug 8, 2023
@andrewdavidwong andrewdavidwong removed this from the Release 4.2 milestone Aug 13, 2023
@DemiMarie DemiMarie removed their assignment Mar 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects-4.2 This issue affects Qubes OS 4.2. C: Debian/Ubuntu needs diagnosis Requires technical diagnosis from developer. Replace with "diagnosed" or remove if otherwise closed. P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. pr submitted A pull request has been submitted for this issue. T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants