Re: [pmg-devel] [PATCH pmg-api v5 05/11] postinstall: add new group for shared functionality

[Stoiko Ivanov <s.ivanov@proxmox.com>]

TIL: `systemd-sysusers` - thanks!
looks fine - a short comment/link to why the flag to `dh` is (still?)
needed would help me here

On Fri,  4 Apr 2025 15:14:32 +0200
Maximiliano Sandoval <m.sandoval@proxmox.com> wrote:

> A shared group named 'pmg' is introduced for processes that need to be
> accessible from multiple processes like spamassassin, rrdcached or the
> mail queue at /var/spool/pmg.
> 
> Signed-off-by: Maximiliano Sandoval <m.sandoval@proxmox.com>
> ---
>  debian/pmg-api.sysusers | 1 +
>  debian/postinst         | 4 ++++
>  debian/rules            | 2 +-
>  3 files changed, 6 insertions(+), 1 deletion(-)
>  create mode 100644 debian/pmg-api.sysusers
> 
> diff --git a/debian/pmg-api.sysusers b/debian/pmg-api.sysusers
> new file mode 100644
> index 00000000..a546c45f
> --- /dev/null
> +++ b/debian/pmg-api.sysusers
> @@ -0,0 +1 @@
> +g pmg             -               -
> diff --git a/debian/postinst b/debian/postinst
> index 708350ec..9af65ebf 100644
> --- a/debian/postinst
> +++ b/debian/postinst
> @@ -49,6 +49,10 @@ migrate_apt_auth_conf() {
>  }
>  
>  migrate_pmg_smtp_filter() {
> +    systemd-sysusers
> +
> +    chown :pmg /var/lib/pmg
> +
>      pmgdb update >/dev/null 2>&1 &
>  }
>  
> diff --git a/debian/rules b/debian/rules
> index 3e150795..ea8f110c 100755
> --- a/debian/rules
> +++ b/debian/rules
> @@ -13,7 +13,7 @@ include debian/rules.env
>  export REPOID=${REPOID_GENERATED}
>  
>  %:
> -	dh $@
> +	dh $@ --with installsysusers
>  
>  override_dh_installsystemd:
>  	dh_installsystemd --no-start --no-stop-on-upgrade \



_______________________________________________
pmg-devel mailing list
pmg-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pmg-devel