[PATCH] watchmaildir: limit live importer processes

2017-01-18 Thread Eric Wong
We don't want to be triggering OOM or swapping on weaker systems when we have dozens of inboxes as potential targets. --- lib/PublicInbox/WatchMaildir.pm | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/lib/PublicInbox/WatchMaildir.pm b/lib/PublicInbox/WatchMaildir.pm in

Re: [PATCH v6 4/6] builtin/tag: add --format argument for tag -v

2017-01-18 Thread Eric Wong
Eric Wong wrote: > Junio C Hamano wrote: > > > > > > http://public-inbox.org/git/20170118182831.pkhqu2np3bh2puei@LykOS.localdomain/ > > Eeep! This looks like a regression I introduced when working > around Richard Hansen's S/MIME mails the other week on git@vger: > > https://public-inbox.

[PATCH] mime: avoid SUPER usage in Email::MIME subclass

2017-01-18 Thread Eric Wong
We must call Email::Simple methods directly in our monkey patch for Email::MIME to call the intended method. Using SUPER in our subclass would instead hit a different, unintended method in Email::MIME. Reported-by: Junio C Hamano --- lib/PublicInbox/MIME.pm | 6 ++--- t/mime.t

[PATCH] learn: implement "rm" only functionality

2017-01-18 Thread Eric Wong
Do not consider this interface stable, but I just needed a way to remove mis-imported multipart messages so public-inbox-watch could pick them up again from my Maildir. --- script/public-inbox-learn | 25 ++--- 1 file changed, 14 insertions(+), 11 deletions(-) diff --git a/scr

Re: [PATCH v6 4/6] builtin/tag: add --format argument for tag -v

2017-01-18 Thread Eric Wong
Junio C Hamano wrote: > Santiago Torres writes: > > <>??? > > Eric, I've noticed that this message > > > http://public-inbox.org/git/20170118182831.pkhqu2np3bh2puei@LykOS.localdomain/ > > and all messages from Santiago appear empty when they come via > public-inbox.org; the reason I suspec

Re: [PATCH v6 4/6] builtin/tag: add --format argument for tag -v

2017-01-18 Thread Santiago Torres
On Wed, Jan 18, 2017 at 10:44:03AM -0800, Junio C Hamano wrote: > Santiago Torres writes: > Was: Thanks! Would you want me to re-roll really quick? or would you rather apply this on your side? Thanks, -Santiago. > > Eric, I've noticed that this message > > > http://public-inbox.org/git/20

Re: [PATCH v6 4/6] builtin/tag: add --format argument for tag -v

2017-01-18 Thread Junio C Hamano
Santiago Torres writes: <>??? Eric, I've noticed that this message http://public-inbox.org/git/20170118182831.pkhqu2np3bh2puei@LykOS.localdomain/ and all messages from Santiago appear empty when they come via public-inbox.org; the reason I suspect we haven't heard much complaints is because