unofficial mirror of notmuch@notmuchmail.org
 help / color / mirror / code / Atom feed
From: Justus Winter <4winter@informatik.uni-hamburg.de>
To: Tomi Ollila <tomi.ollila@iki.fi>,  notmuch@notmuchmail.org
Subject: Re: [RFC PATCH 1/1] add --stderr option
Date: Wed, 22 May 2013 13:15:25 +0200	[thread overview]
Message-ID: <20130522111525.9218.25144@thinkbox.jade-hamburg.de> (raw)
In-Reply-To: <m261yb4fmx.fsf@guru.guru-group.fi>

Quoting Tomi Ollila (2013-05-22 09:50:46)
> On Tue, May 21 2013, Justus Winter <4winter@informatik.uni-hamburg.de> wrote:
> 
> > Quoting Tomi Ollila (2013-05-21 20:42:30)
> >> ---
> >> 
> >> Note quickly written untested code (but compiles!), just to show an idea...
> >> 
> >> This implements (i hope) curl(1) --stderr option in notmuch(1):
> >> 
> >>        --stderr <file>
> >>               Redirect  all writes to stderr to the specified file instead. If
> >>               the file name is a plain '-', it is instead written to stdout.
> >> 
> >> This would be useful in emacs interface.
> >
> > Hm, shouldn't it be possible to bind a pipe(2) to stderr instead? I
> > mean in the process of running the notmuch binary (i.e. somewhere
> > along the lines of fork and exec)?
> 
> Yes, if emacs(1) were smarter ;/

Uh >,<

> > I've implemented this for alot, which does not use the binary but
> > directly calls into libnotmuch, but does so in a helper process. Said
> > helper has a pipe(2) on stderr and the alot process reads from it and
> > turns any line into a log message.
> 
> It is unfortunate that you have to do that -- libnotmuch should not
> emit anything to stderr... We've briefly discussed what changes
> are needed to libnotmuch what could be done there but... :)
> 
> <questionable advice>
> Instead of running separate process you could have both ends of the
> pipe in same process and check after libnotmuch call whether there 
> is data in the reading end of the pipe. I think pipe buffers like 4k
> of data. If you used socketpair(2) that buffers 100k of data by
> default in Linux systems. Still, using nonblocking fds are
> advisable if using this hack ;D
> </questionable advice>

Umm, no I don't see how that would work. I mean I'd have to dup(2) a
fd to 2, but that means not only libnotmuch will write stuff to it but
anything ever written to stderr by alot also ends up there.

It is also a means of protecting alot against any fatal errors in
libnotmuch, like segfaults and stuff like this. I'm not sure if that's
changed, but libnotmuch used to call exit once in a while taking alot
with it. With a separate subprocess you can just log this and restart
the process and you don't ever lose any mail.

But yes, it's kind of a hack.

Justus

      reply	other threads:[~2013-05-22 11:15 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-21 18:42 [RFC PATCH 1/1] add --stderr option Tomi Ollila
2013-05-21 19:21 ` Mark Walters
2013-05-21 19:55 ` Justus Winter
2013-05-22  7:50   ` Tomi Ollila
2013-05-22 11:15     ` Justus Winter [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://notmuchmail.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20130522111525.9218.25144@thinkbox.jade-hamburg.de \
    --to=4winter@informatik.uni-hamburg.de \
    --cc=notmuch@notmuchmail.org \
    --cc=tomi.ollila@iki.fi \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://yhetil.org/notmuch.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).