Kirill A. Korinsky (2024-12-14 00:38 +0100):
> On Wed, 11 Dec 2024 22:39:38 +0100,
> Kirill A. Korinsky <kir...@korins.ky> wrote:
> >
> > On Wed, 11 Dec 2024 21:50:53 +0100,
> > Tim van der Molen <t...@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
