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