* [PR PATCH] base-files+eudev: move kvm.rules from base-files to eudev
@ 2020-09-10 14:41 sgn
2020-09-11 5:34 ` CameronNemo
` (4 more replies)
0 siblings, 5 replies; 6+ messages in thread
From: sgn @ 2020-09-10 14:41 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 915 bytes --]
There is a new pull request by sgn against master on the void-packages repository
https://github.com/sgn/void-packages base-files-udev
https://github.com/void-linux/void-packages/pull/24805
base-files+eudev: move kvm.rules from base-files to eudev
Prior to 33d781ff19 (base-files: update to 0.120., 2015-03-22).
We don't requires the existence of `kvm` groups.
Since that change, we ship a udev rule with `base-files`, hence the need
of `kvm` groups.
It's more than 5 years from the change, kvm should be available
in all installation.
Simply removing "system_accounts=kvm:24" from base-files may cause
problem with those very old installation.
Let move those statements and the udev rules to eudev instead.
When moving the rules, rename it to allow partial upgrade,
otherwise, we'll run into conflicts.
A patch file from https://github.com/void-linux/void-packages/pull/24805.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-base-files-udev-24805.patch --]
[-- Type: text/x-diff, Size: 3146 bytes --]
From 4e88a824dcf0e12befe627a56acf101f45ce0ffe Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C4=90o=C3=A0n=20Tr=E1=BA=A7n=20C=C3=B4ng=20Danh?=
<congdanhqx@gmail.com>
Date: Thu, 10 Sep 2020 21:09:37 +0700
Subject: [PATCH] base-files+eudev: move kvm.rules from base-files to eudev
Prior to 33d781ff19 (base-files: update to 0.120., 2015-03-22).
We don't requires the existence of `kvm` groups.
Since that change, we ship a udev rule with `base-files`, hence the need
of `kvm` groups.
It's more than 5 years from the change, kvm should be available
in all installation.
Simply removing "system_accounts=kvm:24" from base-files may cause
problem with those very old installation.
Let move those statements and the udev rules to eudev instead.
When moving the rules, rename it to allow partial upgrade,
otherwise, we'll run into conflicts.
---
srcpkgs/base-files/template | 8 +-------
.../files/66-kvm.rules => eudev/files/67-kvm.rules} | 0
srcpkgs/eudev/template | 4 +++-
3 files changed, 4 insertions(+), 8 deletions(-)
rename srcpkgs/{base-files/files/66-kvm.rules => eudev/files/67-kvm.rules} (100%)
diff --git a/srcpkgs/base-files/template b/srcpkgs/base-files/template
index e4b0947cbaa..4cac23e01ee 100644
--- a/srcpkgs/base-files/template
+++ b/srcpkgs/base-files/template
@@ -1,7 +1,7 @@
# Template file for 'base-files'
pkgname=base-files
version=0.141
-revision=4
+revision=5
bootstrap=yes
depends="xbps-triggers"
short_desc="Void Linux base system files"
@@ -28,8 +28,6 @@ conf_files="
/etc/nsswitch.conf"
replaces="base-directories>=0"
-# New system groups
-system_groups="kvm:24"
do_install() {
# Create bin and lib dirs and symlinks.
@@ -91,8 +89,4 @@ do_install() {
# Create /proc/self/mounts -> /etc/mtab symlink.
ln -sf /proc/self/mounts ${DESTDIR}/etc/mtab
-
- # udev rules
- vmkdir usr/lib/udev/rules.d
- install -m644 ${FILESDIR}/*.rules ${DESTDIR}/usr/lib/udev/rules.d
}
diff --git a/srcpkgs/base-files/files/66-kvm.rules b/srcpkgs/eudev/files/67-kvm.rules
similarity index 100%
rename from srcpkgs/base-files/files/66-kvm.rules
rename to srcpkgs/eudev/files/67-kvm.rules
diff --git a/srcpkgs/eudev/template b/srcpkgs/eudev/template
index 42ef6369b06..56f57619895 100644
--- a/srcpkgs/eudev/template
+++ b/srcpkgs/eudev/template
@@ -4,7 +4,7 @@ _UDEV_VERSION="243" # compatible udev version provided
pkgname=eudev
version=3.2.9
-revision=1
+revision=2
build_style=gnu-configure
configure_args="--enable-hwdb --enable-manpages --disable-introspection"
hostmakedepends="automake libtool pkg-config gperf libxslt docbook-xsl"
@@ -16,6 +16,7 @@ homepage="https://github.com/gentoo/eudev"
distfiles="https://github.com/gentoo/eudev/archive/v${version}.tar.gz"
checksum=7d281276b480da3935d1acb239748c2c9db01a8043aad7e918ce57a223d8cd24
conf_files="/etc/udev/udev.conf"
+system_groups="kvm:24"
pre_configure() {
autoreconf -fi
@@ -23,6 +24,7 @@ pre_configure() {
post_install() {
mv "${DESTDIR}/etc/udev/hwdb.d" "${DESTDIR}/usr/lib/udev"
+ vinstall ${FILESDIR}/67-kvm.rules 644 usr/lib/udev/rules.d
vsv udevd
}
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: base-files+eudev: move kvm.rules from base-files to eudev
2020-09-10 14:41 [PR PATCH] base-files+eudev: move kvm.rules from base-files to eudev sgn
@ 2020-09-11 5:34 ` CameronNemo
2020-09-11 14:26 ` sgn
` (3 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: CameronNemo @ 2020-09-11 5:34 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 410 bytes --]
New comment by CameronNemo on void-packages repository
https://github.com/void-linux/void-packages/pull/24805#issuecomment-690885498
Comment:
How does this work if somebody wants to try out an alternative device manager (like mdev, which is part of the busybox-huge package)? I presume both packages will have to declare the group, but what happens when one or the other is uninstalled? Does the group stay?
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: base-files+eudev: move kvm.rules from base-files to eudev
2020-09-10 14:41 [PR PATCH] base-files+eudev: move kvm.rules from base-files to eudev sgn
2020-09-11 5:34 ` CameronNemo
@ 2020-09-11 14:26 ` sgn
2020-09-15 19:12 ` ericonr
` (2 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: sgn @ 2020-09-11 14:26 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 214 bytes --]
New comment by sgn on void-packages repository
https://github.com/void-linux/void-packages/pull/24805#issuecomment-691126278
Comment:
The problem should be fixed by #24812, I think this is not necessary anymore?
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: base-files+eudev: move kvm.rules from base-files to eudev
2020-09-10 14:41 [PR PATCH] base-files+eudev: move kvm.rules from base-files to eudev sgn
2020-09-11 5:34 ` CameronNemo
2020-09-11 14:26 ` sgn
@ 2020-09-15 19:12 ` ericonr
2020-09-15 21:37 ` CameronNemo
2020-09-19 3:42 ` [PR PATCH] [Closed]: " sgn
4 siblings, 0 replies; 6+ messages in thread
From: ericonr @ 2020-09-15 19:12 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 576 bytes --]
New comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/24805#issuecomment-692921412
Comment:
> The problem should be fixed by #24812, I think this is not necessary anymore?
I think I would still like to see it done. The way it's been implemented currently is not great.
@CameronNemo
> How does this work if somebody wants to try out an alternative device manager
Can `mdev` use udev rules files in the udev location?
> Does the group stay?
It does, with a small note that the package requiring it was removed.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: base-files+eudev: move kvm.rules from base-files to eudev
2020-09-10 14:41 [PR PATCH] base-files+eudev: move kvm.rules from base-files to eudev sgn
` (2 preceding siblings ...)
2020-09-15 19:12 ` ericonr
@ 2020-09-15 21:37 ` CameronNemo
2020-09-19 3:42 ` [PR PATCH] [Closed]: " sgn
4 siblings, 0 replies; 6+ messages in thread
From: CameronNemo @ 2020-09-15 21:37 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 391 bytes --]
New comment by CameronNemo on void-packages repository
https://github.com/void-linux/void-packages/pull/24805#issuecomment-692993308
Comment:
>Can mdev use udev rules files in the udev location?
No, mdev uses its own more restricted configuration format.
>It does, with a small note that the package requiring it was removed.
That may be manageable, although somewhat misleading.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PR PATCH] [Closed]: base-files+eudev: move kvm.rules from base-files to eudev
2020-09-10 14:41 [PR PATCH] base-files+eudev: move kvm.rules from base-files to eudev sgn
` (3 preceding siblings ...)
2020-09-15 21:37 ` CameronNemo
@ 2020-09-19 3:42 ` sgn
4 siblings, 0 replies; 6+ messages in thread
From: sgn @ 2020-09-19 3:42 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 764 bytes --]
There's a closed pull request on the void-packages repository
base-files+eudev: move kvm.rules from base-files to eudev
https://github.com/void-linux/void-packages/pull/24805
Description:
Prior to 33d781ff19 (base-files: update to 0.120., 2015-03-22).
We don't requires the existence of `kvm` groups.
Since that change, we ship a udev rule with `base-files`, hence the need
of `kvm` groups.
It's more than 5 years from the change, kvm should be available
in all installation.
Simply removing "system_accounts=kvm:24" from base-files may cause
problem with those very old installation.
Let move those statements and the udev rules to eudev instead.
When moving the rules, rename it to allow partial upgrade,
otherwise, we'll run into conflicts.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2020-09-19 3:42 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-10 14:41 [PR PATCH] base-files+eudev: move kvm.rules from base-files to eudev sgn
2020-09-11 5:34 ` CameronNemo
2020-09-11 14:26 ` sgn
2020-09-15 19:12 ` ericonr
2020-09-15 21:37 ` CameronNemo
2020-09-19 3:42 ` [PR PATCH] [Closed]: " sgn
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).