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
prev parent 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).