user/dev discussion of public-inbox itself
 help / color / mirror / code / Atom feed
* [PATCH] learn: support for v2 repos
@ 2018-05-16  5:33 Eric Wong
  0 siblings, 0 replies; only message in thread
From: Eric Wong @ 2018-05-16  5:33 UTC (permalink / raw)
  To: meta

Oops, I mainly rely on public-inbox-watch for spam training
and completely forgot this tool existed :x
---
 script/public-inbox-learn | 18 +++++++-----------
 1 file changed, 7 insertions(+), 11 deletions(-)

diff --git a/script/public-inbox-learn b/script/public-inbox-learn
index c51f958..7453470 100755
--- a/script/public-inbox-learn
+++ b/script/public-inbox-learn
@@ -8,8 +8,7 @@ my $usage = "$0 (spam|ham) < /path/to/message";
 use strict;
 use warnings;
 use PublicInbox::Config;
-use PublicInbox::Git;
-use PublicInbox::Import;
+use PublicInbox::InboxWritable;
 use PublicInbox::MIME;
 use PublicInbox::Address;
 use PublicInbox::Spamcheck::Spamc;
@@ -52,10 +51,8 @@ foreach my $h (qw(Cc To)) {
 if ($train eq 'spam') {
 	$pi_config->each_inbox(sub {
 		my ($ibx) = @_;
-		my $git = $ibx->git;
-		my $name = $ibx->{name};
-		my $addr = $ibx->{-primary_address};
-		my $im = PublicInbox::Import->new($git, $name, $addr, $ibx);
+		$ibx = PublicInbox::InboxWritable->new($ibx);
+		my $im = $ibx->importer(0);
 		$im->remove($mime, 'spam');
 		$im->done;
 	});
@@ -66,13 +63,12 @@ require PublicInbox::MDA if $train eq "ham";
 # n.b. message may be cross-posted to multiple public-inboxes
 foreach my $recipient (keys %dests) {
 	my $dst = $pi_config->lookup($recipient) or next;
-	my $git_dir = $dst->{mainrepo} or next;
-	my $git = PublicInbox::Git->new($git_dir);
 	# We do not touch GIT_COMMITTER_* env here so we can track
 	# who trained the message.
-	my $name = $ENV{GIT_COMMITTER_NAME} || $dst->{name};
-	my $email = $ENV{GIT_COMMITTER_EMAIL} || $recipient;
-	my $im = PublicInbox::Import->new($git, $name, $email);
+	$dst->{name} = $ENV{GIT_COMMITTER_NAME} || $dst->{name};
+	$dst->{-primary_address} = $ENV{GIT_COMMITTER_EMAIL} || $recipient;
+	$dst = PublicInbox::InboxWritable->new($dst);
+	my $im = $dst->importer(0);
 
 	if ($train eq "spam" || $train eq "rm") {
 		# This needs to be idempotent, as my inotify trainer
-- 
EW


^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2018-05-16  5:33 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-05-16  5:33 [PATCH] learn: support for v2 repos Eric Wong

Code repositories for project(s) associated with this public inbox

	https://80x24.org/public-inbox.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).