Re: [PR PATCH] [Updated] incus: reverse dependency; add man pages

[duskmoss <duskmoss@users.noreply.github.com>]

[-- Attachment #1: Type: text/plain, Size: 1175 bytes --]

There is an updated pull request by duskmoss against master on the void-packages repository

https://github.com/duskmoss/void-packages incus-updates
https://github.com/void-linux/void-packages/pull/52384

incus: reverse dependency; add man pages 
incus-client is intended to be installed without the daemon to use to control remove incus servers. incus the daemon can't me used without the client, at least to authorize another client's to access the API.

I moved incus-user to the daemon package because it's a daemon and not part of the client.

readme: I also removed the information about subuid and subgid because void sets these up out of the box.

readme: I changed "and" to "or" because the two groups grant you access to the daemons in different privelege levels.


#### Testing the changes
- I tested the changes in this PR: **briefly**

#### Local build testing
- I built this PR locally for my native architecture:
x86_64-glibc
- I built this PR locally for these architectures:
x86_64-musl
(cross) aarch64-musl 
(cross) armv7l
(cross) armv6l-musl



A patch file from https://github.com/void-linux/void-packages/pull/52384.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-incus-updates-52384.patch --]
[-- Type: text/x-diff, Size: 3789 bytes --]

From 88d4aa6725b6b3b0642ef552d42c9f69bcd25a23 Mon Sep 17 00:00:00 2001
From: Moss Hope <duskmoss@proton.me>
Date: Fri, 27 Sep 2024 15:56:48 -0700
Subject: [PATCH] incus: reverse dependency; add man pages

    incus-client is intended to be installed without the daemon to use to
    control remote incus servers. incus the daemon can't be used without
    the client, at least to authorize another client to access the API.

    I moved incus-user to the daemon package because it's a daemon and
    not part of the client.

    I added incus's generated manpages to the incus-client package.

    readme: I changed "and" to "or" because the two groups grant you
    access to the daemons in different privelege levels.
---
 srcpkgs/incus/files/README.voidlinux |  4 ++--
 srcpkgs/incus/template               | 32 +++++++++++++++++++++++-----
 2 files changed, 29 insertions(+), 7 deletions(-)

diff --git a/srcpkgs/incus/files/README.voidlinux b/srcpkgs/incus/files/README.voidlinux
index 2d1757447dda8e..31a55a3415ece8 100644
--- a/srcpkgs/incus/files/README.voidlinux
+++ b/srcpkgs/incus/files/README.voidlinux
@@ -1,6 +1,6 @@
 Users wishing to interact with incus system daemons should belong to
-the `_incus-admin` and `_incus` groups. The root user should also be
-assigned a range of subordinate user and group IDs to be mapped in
+either the `_incus-admin` or `_incus` group. The root user should also
+be assigned a range of subordinate user and group IDs to be mapped in
 containers. For example, the command
 
 	usermod --add-subuids 1000000-1065535 \
diff --git a/srcpkgs/incus/template b/srcpkgs/incus/template
index a7c9857c58e33b..781c638db738cb 100644
--- a/srcpkgs/incus/template
+++ b/srcpkgs/incus/template
@@ -1,7 +1,7 @@
 # Template file for 'incus'
 pkgname=incus
 version=6.5.0
-revision=1
+revision=2
 build_style=go
 build_helper=qemu
 go_import_path=github.com/lxc/incus/v6
@@ -11,7 +11,8 @@ make_check_args="-skip TestConvertNetworkConfig"
 hostmakedepends="pkg-config"
 makedepends="lxc-devel acl-devel cowsql-devel raft-devel
  libcap-devel libuv-devel sqlite-devel eudev-libudev-devel"
-depends="lxc acl acl-progs rsync squashfs-tools xz dnsmasq iptables attr-progs"
+depends="lxc acl acl-progs rsync squashfs-tools xz dnsmasq iptables attr-progs
+ ${pkgname}-client-${version}_${revision}"
 short_desc="Powerful system container and virtual machine manager"
 maintainer="dkwo <npiazza@disroot.org>"
 license="Apache-2.0"
@@ -38,6 +39,7 @@ post_install() {
 		vinstall "${f}" 700 usr/libexec/incus && rm "${f}"
 	done
 	vsv incus
+	vsv incus-user
 
 	# avoid conflict with lxd, lxd-lts
 	mv ${DESTDIR}/usr/bin/{fuidshift,fuidshift-incus}
@@ -46,24 +48,44 @@ post_install() {
 		chmod 700 ${DESTDIR}/usr/bin/${_tool}
 	done
 
-	# generate shell completions
+	# path to run incus commands in build
 	local _incus=${DESTDIR}/usr/bin/incus
+
+	# generate shell completions
 	for _shell in bash fish zsh; do
 		vtargetrun ${_incus} completion ${_shell} > scripts/${_shell}-completion
 	done
 
+	# generate man pages
+	mkdir man
+	vtargetrun ${_incus} manpage --all --format=man man
+
+	local _manpage _prefix _suffix
+	# rename pages with multiple dots
+	for _manpage in man/*.*.*; do
+	[ -f "${_manpage}" ] || continue
+		_prefix="${_manpage%.*}"
+		_suffix="${_manpage##*.}"
+		mv "${_manpage}" "${_prefix//./-}.${_suffix}"
+	done
+
 	vdoc "${FILESDIR}/README.voidlinux"
 }
 
 incus-client_package() {
 	short_desc+=" - client"
-	depends="${sourcepkg}>=${version}_${revision}"
 	pkg_install() {
 		vmove usr/bin/incus
-		vsv incus-user
+
+		# install shell completions
 		for shell in bash fish zsh; do
 			vcompletion scripts/${shell}-completion ${shell} incus
 		done
+
+		# install manpages
+		for _manpage in man/*; do
+			vman "$_manpage"
+		done
 	}
 }