mail/spamassassin: revert previous accidential commit

Revert previous accidential commit by my mistake.

4 files changed, 68 insertions, 60 deletions

diff --git a/mail/spamassassin/Makefile b/mail/spamassassin/Makefile
index 374b68940ea..336147607fe 100644
--- a/mail/spamassassin/Makefile
+++ b/mail/spamassassin/Makefile
@@ -1,7 +1,8 @@
-# $NetBSD: Makefile,v 1.139 2020/03/21 15:29:37 taca Exp $
+# $NetBSD: Makefile,v 1.140 2020/03/21 15:36:32 taca Exp $
 
-DISTNAME=	Mail-SpamAssassin-3.4.4
-PKGNAME=	spamassassin-3.4.4
+DISTNAME=	Mail-SpamAssassin-3.4.2
+PKGNAME=	spamassassin-3.4.2
+PKGREVISION=	3
 CATEGORIES=	mail perl5
 MASTER_SITES=	${MASTER_SITE_APACHE:=spamassassin/source/}
 DISTFILES=	${DISTNAME}${EXTRACT_SUFX}
@@ -19,7 +20,7 @@ LICENSE=	apache-2.0
 
 SMF_METHODS=	spamassassin
 
-RULESARCHIVE=		Mail-SpamAssassin-rules-3.4.4.r1873061.tgz
+RULESARCHIVE=		Mail-SpamAssassin-rules-3.4.2.r1840640.tgz
 #RULESARCHIVEASC=	${RULESARCHIVE}.asc
 #RULESARCHIVESHA=	${RULESARCHIVE}.sha1
 FILES_SUBST+=		RULESARCHIVE=${RULESDIR}/${RULESARCHIVE}
@@ -183,9 +184,9 @@ SUBST_FILES.fetch=	sa-update.raw
 SUBST_VARS.fetch=	SA_UPDATE_FETCH_CMD SA_UPDATE_FETCH_PROG
 
 pre-configure: perl5-configure
-#	GNU configure needs version.h -> create it
+	# GNU configure needs version.h -> create it
 	cd ${WRKSRC}/spamc && ${PERL5} ./version.h.pl
-#	values in 'config' will be set via the SUBST framework in options.mk
+	# values in 'config' will be set via the SUBST framework in options.mk
 	${CP} ${WRKSRC}/t/config.dist ${WRKSRC}/t/config
 
 post-install:
@@ -200,7 +201,7 @@ post-install:
 		${INSTALL_DATA} "$$f" ${DESTDIR}${EGDIR} &&		\
 		${ECHO} "${EG_SUBDIR}/$$f" >> ${DYNAMIC_PLIST};		\
 	done
-#	examples are in the static PLIST
+	# examples are in the static PLIST
 	${INSTALL_DATA} ${WRKSRC}/procmailrc.example ${DESTDIR}${EGDIR}
 	${INSTALL_DATA} ${WRKSRC}/rules/local.cf ${DESTDIR}${EGDIR}
 	${INSTALL_DATA} ${FILESDIR}/netbsd_lists.cf ${DESTDIR}${EGDIR}
diff --git a/mail/spamassassin/distinfo b/mail/spamassassin/distinfo
index 558721150da..2e9b7a54d97 100644
--- a/mail/spamassassin/distinfo
+++ b/mail/spamassassin/distinfo
@@ -1,16 +1,16 @@
-$NetBSD: distinfo,v 1.73 2020/03/21 15:29:37 taca Exp $
+$NetBSD: distinfo,v 1.74 2020/03/21 15:36:32 taca Exp $
 
-SHA1 (Mail-SpamAssassin-3.4.4.tar.gz) = 98ce46e464a3dd9d3bf83b8744eebcb109292a6d
-RMD160 (Mail-SpamAssassin-3.4.4.tar.gz) = 414fdfcb544afd44f3e17dbced145f26fe827e2e
-SHA512 (Mail-SpamAssassin-3.4.4.tar.gz) = b6efa1c733ddf810b189ec69445faeae6488ee2671f87f56b49ec3bf85690bf7950aa5ce251c1f1371b2bbe4fb88dbce0a162c9a24a48ed5e6584f9019611552
-Size (Mail-SpamAssassin-3.4.4.tar.gz) = 3274482 bytes
-SHA1 (Mail-SpamAssassin-rules-3.4.4.r1873061.tgz) = ee022f1cc18a66d49c1954936c70c36049c37dcf
-RMD160 (Mail-SpamAssassin-rules-3.4.4.r1873061.tgz) = fb8a8c71dbd6621ca7044600bf9a52f9398b6fcb
-SHA512 (Mail-SpamAssassin-rules-3.4.4.r1873061.tgz) = cc2f6949db4662cdcaf5dcef922e69d18320a591deb7fb98c1fb729d91d37f5164052ab6cd2e294657334874fbfc0cccdefc750910e0453cb8da0b3f263b3ede
-Size (Mail-SpamAssassin-rules-3.4.4.r1873061.tgz) = 315930 bytes
-SHA1 (patch-Makefile.PL) = 8c866690cd40109dbeb31e475603255daba075e1
+SHA1 (Mail-SpamAssassin-3.4.2.tar.gz) = f24c471d6594f60f1a1146a11bcb1c2f5215de03
+RMD160 (Mail-SpamAssassin-3.4.2.tar.gz) = dd3cbd744a642478d5773e2fb524461b2cd864e3
+SHA512 (Mail-SpamAssassin-3.4.2.tar.gz) = 85e3d78bb885ad1d0bf2066d1bc919d6ad5e9f86795069397e7c28cc1ba02870566ec014c08c81f68e7ed03b7f60d2de0b9730b3415b35d848abde2c8920a28f
+Size (Mail-SpamAssassin-3.4.2.tar.gz) = 3185452 bytes
+SHA1 (Mail-SpamAssassin-rules-3.4.2.r1840640.tgz) = c93006e1572297f816a0e186a98cbbae246a4945
+RMD160 (Mail-SpamAssassin-rules-3.4.2.r1840640.tgz) = d1f5d207176e51e37c346b2b6ce893b50341011d
+SHA512 (Mail-SpamAssassin-rules-3.4.2.r1840640.tgz) = 38b5f4dc6e6776937e787123c265ecd9a0a2f60aca1b57d6ed4a8f78cf81550478eddd0829b1255e9e8ce64421e06cc13ae82f1a597e893b65f0d07ba8c53a7f
+Size (Mail-SpamAssassin-rules-3.4.2.r1840640.tgz) = 284758 bytes
+SHA1 (patch-Makefile.PL) = f598b173e73130b55714413d5fc55e29ca6a3c4f
 SHA1 (patch-README) = 5d2aaecc4791e4f76df1078c17036cc23a39a8d0
 SHA1 (patch-ae) = e6e83c1de1002b8db647779d17740e67103b69d8
-SHA1 (patch-sa-update) = 12f996b23ae05e1a5919a314e82dc66e28b8ef4c
+SHA1 (patch-sa-update) = 59cba1287051042fc7f510f5e5ef462e2ee8d034
 SHA1 (patch-spamc_libspamc.c) = 757b845df445414d4ba0c2fb039dbc6d9e68b85f
 SHA1 (patch-spamd_netbsd-rc-script.sh) = 192fc1876ee30a4475c0efd9be6340e87d9fa2f4
diff --git a/mail/spamassassin/patches/patch-Makefile.PL b/mail/spamassassin/patches/patch-Makefile.PL
index bfdad7a7095..47eb7cb0c75 100644
--- a/mail/spamassassin/patches/patch-Makefile.PL
+++ b/mail/spamassassin/patches/patch-Makefile.PL
@@ -1,13 +1,13 @@
-$NetBSD: patch-Makefile.PL,v 1.4 2020/03/21 15:29:38 taca Exp $
+$NetBSD: patch-Makefile.PL,v 1.5 2020/03/21 15:36:33 taca Exp $
 
   - enable build of qmail-spamc
   - some files (local.cf, *.pre) are managed by pkgsrc, disable
     their installation.
   - pkgsrc creates B_DATADIR, so there is no need to re-set permissions
 
---- Makefile.PL.orig	2020-01-25 02:51:33.000000000 +0000
-+++ Makefile.PL
-@@ -139,6 +139,7 @@ my %makefile = (
+--- Makefile.PL.orig	2015-04-28 15:57:01.000000000 -0400
++++ Makefile.PL	2015-09-09 14:59:06.000000000 -0400
+@@ -139,6 +139,7 @@
        'spamassassin.raw' => 'spamassassin',
        'sa-learn.raw'     => 'sa-learn',
        'sa-update.raw'    => 'sa-update',
@@ -15,31 +15,38 @@ $NetBSD: patch-Makefile.PL,v 1.4 2020/03/21 15:29:38 taca Exp $
        'sa-compile.raw'   => 'sa-compile',
        'sa-awl.raw'       => 'sa-awl',
        'sa-check_spamd.raw' => 'sa-check_spamd',
-@@ -1120,16 +1121,17 @@ qmail/qmail-spamc$(EXE_EXT): spamc/qmail
+@@ -1120,15 +1121,16 @@
  
  conf__install:
  	-$(MKPATH) $(B_CONFDIR)
--	$(PERL) -MFile::Copy -e "copy(q[rules/local.cf], q[$(B_CONFDIR)/local.cf]) unless -f q[$(B_CONFDIR)/local.cf]"
--	$(PERL) -MFile::Copy -e "copy(q[rules/init.pre], q[$(B_CONFDIR)/init.pre]) unless -f q[$(B_CONFDIR)/init.pre]"
--	$(PERL) -MFile::Copy -e "copy(q[rules/v310.pre], q[$(B_CONFDIR)/v310.pre]) unless -f q[$(B_CONFDIR)/v310.pre]"
--	$(PERL) -MFile::Copy -e "copy(q[rules/v312.pre], q[$(B_CONFDIR)/v312.pre]) unless -f q[$(B_CONFDIR)/v312.pre]"
--	$(PERL) -MFile::Copy -e "copy(q[rules/v320.pre], q[$(B_CONFDIR)/v320.pre]) unless -f q[$(B_CONFDIR)/v320.pre]"
--	$(PERL) -MFile::Copy -e "copy(q[rules/v330.pre], q[$(B_CONFDIR)/v330.pre]) unless -f q[$(B_CONFDIR)/v330.pre]"
--	$(PERL) -MFile::Copy -e "copy(q[rules/v340.pre], q[$(B_CONFDIR)/v340.pre]) unless -f q[$(B_CONFDIR)/v340.pre]"
--	$(PERL) -MFile::Copy -e "copy(q[rules/v341.pre], q[$(B_CONFDIR)/v341.pre]) unless -f q[$(B_CONFDIR)/v341.pre]"
--	$(PERL) -MFile::Copy -e "copy(q[rules/v342.pre], q[$(B_CONFDIR)/v342.pre]) unless -f q[$(B_CONFDIR)/v342.pre]"
--	$(PERL) -MFile::Copy -e "copy(q[rules/v343.pre], q[$(B_CONFDIR)/v343.pre]) unless -f q[$(B_CONFDIR)/v343.pre]"
-+# manage local.cf and *.pre through pkgsrc .mk files
-+#	$(PERL) -MFile::Copy -e "copy(q[rules/local.cf], q[$(B_CONFDIR)/local.cf]) unless -f q[$(B_CONFDIR)/local.cf]"
-+#	$(PERL) -MFile::Copy -e "copy(q[rules/init.pre], q[$(B_CONFDIR)/init.pre]) unless -f q[$(B_CONFDIR)/init.pre]"
-+#	$(PERL) -MFile::Copy -e "copy(q[rules/v310.pre], q[$(B_CONFDIR)/v310.pre]) unless -f q[$(B_CONFDIR)/v310.pre]"
-+#	$(PERL) -MFile::Copy -e "copy(q[rules/v312.pre], q[$(B_CONFDIR)/v312.pre]) unless -f q[$(B_CONFDIR)/v312.pre]"
-+#	$(PERL) -MFile::Copy -e "copy(q[rules/v320.pre], q[$(B_CONFDIR)/v320.pre]) unless -f q[$(B_CONFDIR)/v320.pre]"
-+#	$(PERL) -MFile::Copy -e "copy(q[rules/v330.pre], q[$(B_CONFDIR)/v330.pre]) unless -f q[$(B_CONFDIR)/v330.pre]"
-+#	$(PERL) -MFile::Copy -e "copy(q[rules/v340.pre], q[$(B_CONFDIR)/v340.pre]) unless -f q[$(B_CONFDIR)/v340.pre]"
-+#	$(PERL) -MFile::Copy -e "copy(q[rules/v341.pre], q[$(B_CONFDIR)/v341.pre]) unless -f q[$(B_CONFDIR)/v341.pre]"
-+#	$(PERL) -MFile::Copy -e "copy(q[rules/v342.pre], q[$(B_CONFDIR)/v342.pre]) unless -f q[$(B_CONFDIR)/v342.pre]"
-+#	$(PERL) -MFile::Copy -e "copy(q[rules/v343.pre], q[$(B_CONFDIR)/v343.pre]) unless -f q[$(B_CONFDIR)/v343.pre]"
+-	$(PERL) -MFile::Copy -e "copy(q[rules/local.cf], q{$(B_CONFDIR)/local.cf}) unless -f q{$(B_CONFDIR)/local.cf}"
+-	$(PERL) -MFile::Copy -e "copy(q[rules/init.pre], q{$(B_CONFDIR)/init.pre}) unless -f q{$(B_CONFDIR)/init.pre}"
+-	$(PERL) -MFile::Copy -e "copy(q[rules/v310.pre], q{$(B_CONFDIR)/v310.pre}) unless -f q{$(B_CONFDIR)/v310.pre}"
+-	$(PERL) -MFile::Copy -e "copy(q[rules/v312.pre], q{$(B_CONFDIR)/v312.pre}) unless -f q{$(B_CONFDIR)/v312.pre}"
+-	$(PERL) -MFile::Copy -e "copy(q[rules/v320.pre], q{$(B_CONFDIR)/v320.pre}) unless -f q{$(B_CONFDIR)/v320.pre}"
+-	$(PERL) -MFile::Copy -e "copy(q[rules/v330.pre], q{$(B_CONFDIR)/v330.pre}) unless -f q{$(B_CONFDIR)/v330.pre}"
+-	$(PERL) -MFile::Copy -e "copy(q[rules/v340.pre], q{$(B_CONFDIR)/v340.pre}) unless -f q{$(B_CONFDIR)/v340.pre}"
+-	$(PERL) -MFile::Copy -e "copy(q[rules/v341.pre], q{$(B_CONFDIR)/v341.pre}) unless -f q{$(B_CONFDIR)/v341.pre}"
+-	$(PERL) -MFile::Copy -e "copy(q[rules/v342.pre], q{$(B_CONFDIR)/v342.pre}) unless -f q{$(B_CONFDIR)/v342.pre}"
++	# manage local.cf and *.pre through pkgsrc .mk files
++	# $(PERL) -MFile::Copy -e "copy(q[rules/local.cf], q{$(B_CONFDIR)/local.cf}) unless -f q{$(B_CONFDIR)/local.cf}"
++	# $(PERL) -MFile::Copy -e "copy(q[rules/init.pre], q{$(B_CONFDIR)/init.pre}) unless -f q{$(B_CONFDIR)/init.pre}"
++	# $(PERL) -MFile::Copy -e "copy(q[rules/v310.pre], q{$(B_CONFDIR)/v310.pre}) unless -f q{$(B_CONFDIR)/v310.pre}"
++	# $(PERL) -MFile::Copy -e "copy(q[rules/v312.pre], q{$(B_CONFDIR)/v312.pre}) unless -f q{$(B_CONFDIR)/v312.pre}"
++	# $(PERL) -MFile::Copy -e "copy(q[rules/v320.pre], q{$(B_CONFDIR)/v320.pre}) unless -f q{$(B_CONFDIR)/v320.pre}"
++	# $(PERL) -MFile::Copy -e "copy(q[rules/v330.pre], q{$(B_CONFDIR)/v330.pre}) unless -f q{$(B_CONFDIR)/v330.pre}"
++	# $(PERL) -MFile::Copy -e "copy(q[rules/v340.pre], q{$(B_CONFDIR)/v340.pre}) unless -f q{$(B_CONFDIR)/v340.pre}"
++	# $(PERL) -MFile::Copy -e "copy(q[rules/v341.pre], q{$(B_CONFDIR)/v341.pre}) unless -f q{$(B_CONFDIR)/v341.pre}"
++	# $(PERL) -MFile::Copy -e "copy(q[rules/v342.pre], q{$(B_CONFDIR)/v342.pre}) unless -f q{$(B_CONFDIR)/v342.pre}"
+   
  
  data__install:
- 	-$(MKPATH) $(B_DATADIR)
+  	-$(MKPATH) $(B_DATADIR)
+ 	$(PERL) -e "map unlink, <$(B_DATADIR)/*>"
+ 	$(PREPROCESS) $(FIXVARS) -m$(PERM_RW) -Irules -O$(B_DATADIR) $(DATAFILES)
+-	$(CHMOD) $(PERM_RWX) $(B_DATADIR)
++	# no change of directory created by pkgsrc
++	# $(CHMOD) $(PERM_RWX) $(B_DATADIR)
+ 
+ text_html_doc: made-doc-stamp
+ 	$(NOOP)
diff --git a/mail/spamassassin/patches/patch-sa-update b/mail/spamassassin/patches/patch-sa-update
index fd1341ead2a..ec12b5e69d3 100644
--- a/mail/spamassassin/patches/patch-sa-update
+++ b/mail/spamassassin/patches/patch-sa-update
@@ -1,10 +1,10 @@
-$NetBSD: patch-sa-update,v 1.4 2020/03/21 15:29:38 taca Exp $
+$NetBSD: patch-sa-update,v 1.5 2020/03/21 15:36:33 taca Exp $
 
 Prepare for substitution.
 
---- sa-update.raw.orig	2020-01-25 02:51:16.000000000 +0000
-+++ sa-update.raw
-@@ -357,10 +357,10 @@ my $GPGPath;
+--- sa-update.raw.orig	2014-02-07 08:36:38.000000000 +0000
++++ sa-update.raw	2014-04-26 13:56:38.000000000 +0100
+@@ -340,10 +340,10 @@
  if ($GPG_ENABLED || $opt{'import'}) {
    # find GPG in the PATH
    # bug 4958: for *NIX it's "gpg", in Windows it's "gpg.exe"
@@ -17,16 +17,16 @@ Prepare for substitution.
      dbg("gpg: found $GPGPath");
  
      # bug 5030: if GPGPath has a space, put it in quotes
-@@ -1570,10 +1570,8 @@ sub http_get {
-       }
-     }
-   } else {
--    foreach my $try_prog ('curl', 'wget', 'fetch') {
--      $cmd = Mail::SpamAssassin::Util::find_executable_in_env_path($try_prog);
--      if (defined $cmd && $cmd ne '') { $ext_prog = $try_prog; last }
--    }
-+      $ext_prog = '@SA_UPDATE_FETCH_PROG@';
-+      $cmd = '@SA_UPDATE_FETCH_CMD@';
-   }
+@@ -1459,10 +1459,8 @@
+       !$out_fname_exists ? 'new' : $force_reload ? 'replace' : 'update');
+ 
+   my($ext_prog, $cmd, @args);
+-  foreach my $try_prog ('curl', 'wget', 'fetch') {
+-    $cmd = Mail::SpamAssassin::Util::find_executable_in_env_path($try_prog);
+-    if (defined $cmd && $cmd ne '') { $ext_prog = $try_prog; last }
+-  }
++  $ext_prog = '@SA_UPDATE_FETCH_PROG@';
++  $cmd = '@SA_UPDATE_FETCH_CMD@';
  
    if (defined $ext_prog && $ext_prog eq 'curl') {
+     push(@args, qw(-s -L -O --remote-time -g --max-redirs 2