* [PR PATCH] qemu: Add `qemuconf` to depends.
@ 2020-08-14 6:29 flexibeast
2020-08-14 11:42 ` ericonr
` (7 more replies)
0 siblings, 8 replies; 9+ messages in thread
From: flexibeast @ 2020-08-14 6:29 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 447 bytes --]
There is a new pull request by flexibeast against master on the void-packages repository
https://github.com/flexibeast/void-packages modify-qemu
https://github.com/void-linux/void-packages/pull/24267
qemu: Add `qemuconf` to depends.
The `qemu-generic` service provided by this package uses `qemuconf`, but the latter currently isn't listed in "depends".
A patch file from https://github.com/void-linux/void-packages/pull/24267.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-modify-qemu-24267.patch --]
[-- Type: text/x-diff, Size: 1093 bytes --]
From 0cfb7bcd299be68d733a3ad9fb24102e9dc7ead0 Mon Sep 17 00:00:00 2001
From: Alexis <flexibeast@gmail.com>
Date: Fri, 14 Aug 2020 12:17:26 +1000
Subject: [PATCH] qemu: Add `qemuconf` to depends.
---
srcpkgs/qemu/template | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/srcpkgs/qemu/template b/srcpkgs/qemu/template
index d1c70497473..a2456275b59 100644
--- a/srcpkgs/qemu/template
+++ b/srcpkgs/qemu/template
@@ -1,7 +1,7 @@
# Template file for 'qemu'
pkgname=qemu
version=5.1.0
-revision=1
+revision=2
short_desc="Open Source Processor Emulator"
maintainer="Helmut Pozimski <helmut@pozimski.eu>"
license="GPL-2.0-or-later, LGPL-2.1-or-later"
@@ -22,6 +22,7 @@ makedepends="libpng-devel libjpeg-turbo-devel pixman-devel snappy-devel
$(vopt_if opengl 'libepoxy-devel libdrm-devel MesaLib-devel')
$(vopt_if iscsi 'libiscsi-devel')
$(vopt_if smartcard libcacard-devel) $(vopt_if numa 'libnuma-devel')"
+depends="qemuconf"
build_options="gtk3 opengl sdl2 spice virgl smartcard numa iscsi"
build_options_default="opengl gtk3 virgl smartcard sdl2 numa iscsi"
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: qemu: Add `qemuconf` to depends.
2020-08-14 6:29 [PR PATCH] qemu: Add `qemuconf` to depends flexibeast
@ 2020-08-14 11:42 ` ericonr
2020-08-14 13:55 ` flexibeast
` (6 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: ericonr @ 2020-08-14 11:42 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 215 bytes --]
New comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/24267#issuecomment-674034667
Comment:
`qemuconf` can't depend on `qemu`, then, otherwise we'll get a cyclic dep.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: qemu: Add `qemuconf` to depends.
2020-08-14 6:29 [PR PATCH] qemu: Add `qemuconf` to depends flexibeast
2020-08-14 11:42 ` ericonr
@ 2020-08-14 13:55 ` flexibeast
2020-08-14 14:03 ` ericonr
` (5 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: flexibeast @ 2020-08-14 13:55 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 326 bytes --]
New comment by flexibeast on void-packages repository
https://github.com/void-linux/void-packages/pull/24267#issuecomment-674086750
Comment:
Oh, indeed. Would it be a bad idea to move the service into a separate package then? Installing a service without the program it needs to function seems, well, Less Than Optimal. :-)
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: qemu: Add `qemuconf` to depends.
2020-08-14 6:29 [PR PATCH] qemu: Add `qemuconf` to depends flexibeast
2020-08-14 11:42 ` ericonr
2020-08-14 13:55 ` flexibeast
@ 2020-08-14 14:03 ` ericonr
2020-08-14 14:06 ` flexibeast
` (4 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: ericonr @ 2020-08-14 14:03 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 420 bytes --]
New comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/24267#issuecomment-674090487
Comment:
I think you could just remove `qemu` as a dependency for `qemuconf`. If you install `qemuconf` through `qemu`, it will be available in the system c:
That said, I don't know if we want to keep things minimal. If so, perhaps move the service into the `qemuconf` package instead?
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: qemu: Add `qemuconf` to depends.
2020-08-14 6:29 [PR PATCH] qemu: Add `qemuconf` to depends flexibeast
` (2 preceding siblings ...)
2020-08-14 14:03 ` ericonr
@ 2020-08-14 14:06 ` flexibeast
2020-08-14 14:28 ` pullmoll
` (3 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: flexibeast @ 2020-08-14 14:06 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 216 bytes --]
New comment by flexibeast on void-packages repository
https://github.com/void-linux/void-packages/pull/24267#issuecomment-674091880
Comment:
Either makes sense to me, so i'll do whatever others decide is best. :-)
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: qemu: Add `qemuconf` to depends.
2020-08-14 6:29 [PR PATCH] qemu: Add `qemuconf` to depends flexibeast
` (3 preceding siblings ...)
2020-08-14 14:06 ` flexibeast
@ 2020-08-14 14:28 ` pullmoll
2020-08-14 14:47 ` ericonr
` (2 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: pullmoll @ 2020-08-14 14:28 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 244 bytes --]
New comment by pullmoll on void-packages repository
https://github.com/void-linux/void-packages/pull/24267#issuecomment-674101673
Comment:
Since qemu works w/o qemuconf but qemuconf requires qemu, I'd vote for moving the service to qemuconf.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: qemu: Add `qemuconf` to depends.
2020-08-14 6:29 [PR PATCH] qemu: Add `qemuconf` to depends flexibeast
` (4 preceding siblings ...)
2020-08-14 14:28 ` pullmoll
@ 2020-08-14 14:47 ` ericonr
2020-08-14 14:54 ` flexibeast
2020-08-14 14:54 ` [PR PATCH] [Closed]: " flexibeast
7 siblings, 0 replies; 9+ messages in thread
From: ericonr @ 2020-08-14 14:47 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 192 bytes --]
New comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/24267#issuecomment-674110326
Comment:
Isn't the qemu-generic service already in qemuconf?
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: qemu: Add `qemuconf` to depends.
2020-08-14 6:29 [PR PATCH] qemu: Add `qemuconf` to depends flexibeast
` (5 preceding siblings ...)
2020-08-14 14:47 ` ericonr
@ 2020-08-14 14:54 ` flexibeast
2020-08-14 14:54 ` [PR PATCH] [Closed]: " flexibeast
7 siblings, 0 replies; 9+ messages in thread
From: flexibeast @ 2020-08-14 14:54 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 270 bytes --]
New comment by flexibeast on void-packages repository
https://github.com/void-linux/void-packages/pull/24267#issuecomment-674113790
Comment:
@ericonr: You're right. i thought i'd seen that it was provided by `qemu`, but no. :-/ Sorry for the noise everyone, closing.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PR PATCH] [Closed]: qemu: Add `qemuconf` to depends.
2020-08-14 6:29 [PR PATCH] qemu: Add `qemuconf` to depends flexibeast
` (6 preceding siblings ...)
2020-08-14 14:54 ` flexibeast
@ 2020-08-14 14:54 ` flexibeast
7 siblings, 0 replies; 9+ messages in thread
From: flexibeast @ 2020-08-14 14:54 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 286 bytes --]
There's a closed pull request on the void-packages repository
qemu: Add `qemuconf` to depends.
https://github.com/void-linux/void-packages/pull/24267
Description:
The `qemu-generic` service provided by this package uses `qemuconf`, but the latter currently isn't listed in "depends".
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2020-08-14 14:54 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-14 6:29 [PR PATCH] qemu: Add `qemuconf` to depends flexibeast
2020-08-14 11:42 ` ericonr
2020-08-14 13:55 ` flexibeast
2020-08-14 14:03 ` ericonr
2020-08-14 14:06 ` flexibeast
2020-08-14 14:28 ` pullmoll
2020-08-14 14:47 ` ericonr
2020-08-14 14:54 ` flexibeast
2020-08-14 14:54 ` [PR PATCH] [Closed]: " flexibeast
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).