Kirill A. Korinsky (2024-12-14 00:38 +0100):
> On Wed, 11 Dec 2024 22:39:38 +0100,
> Kirill A. Korinsky <kirill@korins.ky> wrote:
> > 
> > On Wed, 11 Dec 2024 21:50:53 +0100,
> > Tim van der Molen <tim@kariliq.nl> wrote:
> > > 
> > > Kirill A. Korinsky (2024-12-09 19:17 +0100):
> > > > Fixing permission at /var/spool/spamassassin/spampd looks too complicated,
> > > > because we run spamassassin's spamd at one user, and mail/spampd from
> > > > another one. Insted magic of permission or moving files in rc-script...,
> > > > I've add pkg/MESSAGE with instruction how to move the data.
> > > 
> > > I'm probably missing something, but I don't understand why you mention
> > > spamassassin's spamd. What is wrong with the following change?
> > >
> > 
> > I was wrong. Sorry. Spamassassin uses /var/db/spamassassin, and not
> > /var/spool/spamassassin.
> > 
> > > Index: pkg/PLIST
> > > ===================================================================
> > > RCS file: /cvs/ports/mail/spampd/pkg/PLIST,v
> > > diff -p -u -r1.5 PLIST
> > > --- pkg/PLIST	8 Nov 2022 11:14:52 -0000	1.5
> > > +++ pkg/PLIST	11 Dec 2024 20:01:11 -0000
> > > @@ -8,3 +8,5 @@ sbin/spampd
> > >  @owner _spampd
> > >  @group _spampd
> > >  @sample /var/spampd/
> > > +@mode 0700
> > > +@sample /var/spool/spamassassin/spampd/
> > > 
> > 
> > Well, PLIST states:
> > 
> >     @comment homedir needed to store the .spamassassin bayes DB
> >     @extraunexec rm -rf /var/spampd/.spamassassin
> > 
> > and based on that, I think that /var/spool/spamassassin/spampd/ is an
> > unexpected place that was used due to missed chunk in patch, and the
> > intended and correct place is /var/spampd, which also looks logical because
> > it keeps data in the user's homedir.
> 
> After future testing I had discovered that home folders quite a mess in
> spampd, and if someone would like to use sa-learn with spampd... he really
> needs to use absolute path without ~.
> 
> When I run it with --homedir /var/spampd it creates
> /var/spampd/.spamassassin as well.
> 
> When with --homedir /var/spampd/.spamassassin it creates
> /var/spampd/.spamassassin/.spamassassin as well.
> 
> For example keeps tx-reputation in `homedir`/tx-reputation, but creating
> `homedir`/.spamassassin and keeps some locks inside.
> 
> To add fun, it cleans up some environment variable which includes HOME:
> https://github.com/mpaperno/spampd/blob/2.62/spampd.pl#L502
> 
> For now I haven't see how to make things streigth, so I gave up on this.
> 
> Here a diff which only updates spampd to 2.62.
> 
> I had running it a few days, and no regression noticed.
> 
> Ok?

No regressions here either.

OK tim

> Index: mail/spampd/Makefile
> ===================================================================
> RCS file: /home/cvs/ports/mail/spampd/Makefile,v
> diff -u -p -r1.14 Makefile
> --- mail/spampd/Makefile	8 Nov 2022 11:14:52 -0000	1.14
> +++ mail/spampd/Makefile	9 Dec 2024 13:41:48 -0000
> @@ -2,8 +2,7 @@ COMMENT=		SMTP/LMTP proxy for Spamassass
>  
>  GH_ACCOUNT=		mpaperno
>  GH_PROJECT=		spampd
> -GH_TAGNAME=		2.61
> -REVISION=		0
> +GH_TAGNAME=		2.62
>  
>  CATEGORIES=		mail
>  
> Index: mail/spampd/distinfo
> ===================================================================
> RCS file: /home/cvs/ports/mail/spampd/distinfo,v
> diff -u -p -r1.4 distinfo
> --- mail/spampd/distinfo	7 Aug 2021 10:15:24 -0000	1.4
> +++ mail/spampd/distinfo	13 Dec 2024 23:34:50 -0000
> @@ -1,2 +1,2 @@
> -SHA256 (spampd-2.61.tar.gz) = keYPEHRepPnCe55XYZob8karmojqG4jE85+K9gfi264=
> -SIZE (spampd-2.61.tar.gz) = 246956
> +SHA256 (spampd-2.62.tar.gz) = 7+AW0RH4wglPSTby76FspueMG0hUZbJp/FxCc7nFfPg=
> +SIZE (spampd-2.62.tar.gz) = 249062
> Index: mail/spampd/patches/patch-spampd_pl
> ===================================================================
> RCS file: /home/cvs/ports/mail/spampd/patches/patch-spampd_pl,v
> diff -u -p -r1.4 patch-spampd_pl
> --- mail/spampd/patches/patch-spampd_pl	11 Mar 2022 19:34:57 -0000	1.4
> +++ mail/spampd/patches/patch-spampd_pl	13 Dec 2024 23:34:58 -0000
> @@ -1,7 +1,7 @@
>  Index: spampd.pl
>  --- spampd.pl.orig
>  +++ spampd.pl
> -@@ -429,8 +429,8 @@ sub new {
> +@@ -430,8 +430,8 @@ sub new {
>         max_servers       => 5,                     # max number of child processes (servers) to spawn
>         max_requests      => 20,                    # max requests handled by child b4 dying
>         pid_file          => '/var/run/spampd.pid', # write pid to file
> @@ -12,7 +12,7 @@ Index: spampd.pl
>         log_file          => undef,                 # log destination (undef means log to use write_to_log_hook() with stderr fallback)
>         syslog_logsock    => undef,                 # syslog socket (undef means for Sys::Syslog to decide)
>         syslog_ident      => 'spampd',              # syslog identity
> -@@ -2080,7 +2080,7 @@ Specifies what UNIX socket spampd will relay to. If th
> +@@ -2087,7 +2087,7 @@ Specifies what UNIX socket spampd will relay to. If th
>   =item B<--group> or B<-g> I<<groupname>>
>   
>   Specifies the user and/or group that the proxy will run as. Default is
> 
> 
> -- 
> wbr, Kirill