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

broken qvm-device symlinks #9505

Open
ethnh opened this issue Oct 12, 2024 · 2 comments
Open

broken qvm-device symlinks #9505

ethnh opened this issue Oct 12, 2024 · 2 comments
Labels
C: core 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: enhancement Type: enhancement. A new feature that does not yet exist or improvement of existing functionality.

Comments

@ethnh
Copy link

ethnh commented Oct 12, 2024

The problem you're addressing (if any)

broken qvm-device symlinks

The solution you'd like

From dadc0d427eddaf49a6b213e7345c86e81a9c84e7 Mon Sep 17 00:00:00 2001
From: Yaroslav Bolyukin <[email protected]>
Date: Mon, 16 Sep 2024 18:54:05 +0200
Subject: [PATCH 2/6] fix: broken qvm-device symlinks

---
 Makefile | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/Makefile b/Makefile
index 7a410ade..89cc36fc 100644
--- a/Makefile
+++ b/Makefile
@@ -179,13 +179,7 @@ ifeq ($(OS),Linux)
 	$(MAKE) install -C linux/system-config
 endif
 	$(PYTHON) setup.py install -O1 --skip-build --root $(DESTDIR)
-	ln -sf qvm-device $(DESTDIR)/usr/bin/qvm-block
-	ln -sf qvm-device $(DESTDIR)/usr/bin/qvm-pci
-	ln -sf qvm-device $(DESTDIR)/usr/bin/qvm-usb
-	install -d $(DESTDIR)/usr/share/man/man1
-	ln -sf qvm-device.1.gz $(DESTDIR)/usr/share/man/man1/qvm-block.1.gz
-	ln -sf qvm-device.1.gz $(DESTDIR)/usr/share/man/man1/qvm-pci.1.gz
-	ln -sf qvm-device.1.gz $(DESTDIR)/usr/share/man/man1/qvm-usb.1.gz
+	# skipping qvm-device symlink installation, qvm-device is a binary in qubes-core-admin-client, not in qubes-core-admin!
 	$(MAKE) install -C relaxng
 	mkdir -p $(DESTDIR)/etc/qubes
 ifeq ($(BACKEND_VMM),xen)
-- 
2.45.2

The value to a user, and who that user might be

https://github.com/NixOS/nixpkgs/blob/19915ebb28f3f79ddd3b18b7bf22af1ac9c4fc88/pkgs/development/python-modules/qubes-core-admin/0002-fix-broken-qvm-device-symlinks.patch

Completion criteria checklist

(This section is for developer use only. Please do not modify it.)

@ethnh ethnh added P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. T: enhancement Type: enhancement. A new feature that does not yet exist or improvement of existing functionality. labels Oct 12, 2024
@marmarek
Copy link
Member

@andrewdavidwong andrewdavidwong added C: core pr submitted A pull request has been submitted for this issue. labels Oct 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: core 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: enhancement Type: enhancement. A new feature that does not yet exist or improvement of existing functionality.
Projects
None yet
Development

No branches or pull requests

3 participants