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

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 ;/

> 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>

>
> Justus

Tomi

  reply	other threads:[~2013-05-22  7:51 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 [this message]
2013-05-22 11:15     ` Justus Winter

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=m261yb4fmx.fsf@guru.guru-group.fi \
    --to=tomi.ollila@iki.fi \
    --cc=4winter@informatik.uni-hamburg.de \
    --cc=notmuch@notmuchmail.org \
    /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).