ido address lookup for notmuch

[notmuch-archives.git] / 74 / eb5f302ea5d5db971b106504c6df38f1c78c11

Return-Path: <dmitry.kurochkin@gmail.com>\r
X-Original-To: notmuch@notmuchmail.org\r
Delivered-To: notmuch@notmuchmail.org\r
Received: from localhost (localhost [127.0.0.1])\r
        by olra.theworths.org (Postfix) with ESMTP id B0E43431FD0\r
        for <notmuch@notmuchmail.org>; Fri, 11 Nov 2011 08:03:09 -0800 (PST)\r
X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
X-Spam-Flag: NO\r
X-Spam-Score: -0.799\r
X-Spam-Level: \r
X-Spam-Status: No, score=-0.799 tagged_above=-999 required=5\r
        tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1,\r
        FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled\r
Received: from olra.theworths.org ([127.0.0.1])\r
        by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
        with ESMTP id AXNwszzRR27L for <notmuch@notmuchmail.org>;\r
        Fri, 11 Nov 2011 08:03:08 -0800 (PST)\r
Received: from mail-fx0-f53.google.com (mail-fx0-f53.google.com\r
        [209.85.161.53]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
        (No client certificate requested)\r
        by olra.theworths.org (Postfix) with ESMTPS id 7AF82431FB6\r
        for <notmuch@notmuchmail.org>; Fri, 11 Nov 2011 08:03:08 -0800 (PST)\r
Received: by faan15 with SMTP id n15so4719027faa.26\r
        for <notmuch@notmuchmail.org>; Fri, 11 Nov 2011 08:03:05 -0800 (PST)\r
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma;\r
        h=from:to:subject:in-reply-to:references:user-agent:date:message-id\r
        :mime-version:content-type;\r
        bh=7/zkcOLidnPxa+LT/QlQhSBD/GexL6G5GZ49gu6BZro=;\r
        b=aqoAkz6KmMIPQFTvgj28qvi0tI1XqqhkcnpPdUHpE4UWWunQF1jN0fcbkpwl9AuI2H\r
        3xl2tt27MVxrt5/w5zFXJR3ocjz3OtRXbBn+4MQo3KP+9RXsEf/X0x+/d8HhBjVHViCA\r
        Ssy68FeZ99t85cIg7wPbR4cqyKome3TPF/GlY=\r
Received: by 10.205.120.2 with SMTP id fw2mr688157bkc.10.1321027385533;\r
        Fri, 11 Nov 2011 08:03:05 -0800 (PST)\r
Received: from localhost ([91.144.186.21])\r
        by mx.google.com with ESMTPS id o7sm1621991bkw.16.2011.11.11.08.03.03\r
        (version=TLSv1/SSLv3 cipher=OTHER);\r
        Fri, 11 Nov 2011 08:03:04 -0800 (PST)\r
From: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>\r
To: Tomi Ollila <tomi.ollila@iki.fi>, Notmuch Mail <notmuch@notmuchmail.org>\r
Subject: Re: [PATCH] test: use dtach(1) instead of screen(1) in emacs tests.\r
In-Reply-To: <yf67h36ew2b.fsf@taco2.nixu.fi>\r
References: <yf6mxc3ab4t.fsf@taco2.nixu.fi>\r
        <1320963737-1666-1-git-send-email-jrollins@finestructure.net>\r
        <877h373vbs.fsf@gmail.com> <yf67h36ew2b.fsf@taco2.nixu.fi>\r
User-Agent: Notmuch/0.9+55~g81c615b (http://notmuchmail.org) Emacs/23.3.1\r
        (x86_64-pc-linux-gnu)\r
Date: Fri, 11 Nov 2011 20:02:52 +0400\r
Message-ID: <87ty6a3avn.fsf@gmail.com>\r
MIME-Version: 1.0\r
Content-Type: text/plain; charset=us-ascii\r
X-BeenThere: notmuch@notmuchmail.org\r
X-Mailman-Version: 2.1.13\r
Precedence: list\r
List-Id: "Use and development of the notmuch mail system."\r
        <notmuch.notmuchmail.org>\r
List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
        <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
List-Post: <mailto:notmuch@notmuchmail.org>\r
List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
        <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
X-List-Received-Date: Fri, 11 Nov 2011 16:03:09 -0000\r
\r
On Fri, 11 Nov 2011 13:29:48 +0200, Tomi Ollila <tomi.ollila@iki.fi> wrote:\r
> On Fri, 11 Nov 2011 12:41:11 +0400, Dmitry Kurochkin <dmitry.kurochkin@gmail.com> wrote:\r
> > Hi Tomi.\r
> > \r
> > On Thu, 10 Nov 2011 14:22:17 -0800, Jameson Graef Rollins <jrollins@finestructure.net> wrote:\r
> > > From: Tomi Ollila <tomi.ollila@iki.fi>\r
> > > \r
> > > dtach is lighter than screen and is not setuid/setgid program so\r
> > > TMPDIR does not get reset by dynamic loader when executed.\r
> > > \r
> > > Signed-off-by: Jameson Graef Rollins <jrollins@finestructure.net>\r
> > > ---\r
> > > This tweaks the original patch slightly by removing the no-longer needed\r
> > > screenrc variables.\r
> > > \r
> > >  test/test-lib.sh |    5 ++---\r
> > >  1 files changed, 2 insertions(+), 3 deletions(-)\r
> > > \r
> > > diff --git a/test/test-lib.sh b/test/test-lib.sh\r
> > > index c81c709..c0fe037 100755\r
> > > --- a/test/test-lib.sh\r
> > > +++ b/test/test-lib.sh\r
> > > @@ -50,8 +50,6 @@ TZ=UTC\r
> > >  TERM=dumb\r
> > >  export LANG LC_ALL PAGER TERM TZ\r
> > >  GIT_TEST_CMP=${GIT_TEST_CMP:-diff -u}\r
> > > -export SCREENRC=/dev/null\r
> > > -export SYSSCREENRC=/dev/null\r
> > >  \r
> > >  # Protect ourselves from common misconfiguration to export\r
> > >  # CDPATH into the environment\r
> > > @@ -844,7 +842,8 @@ test_emacs () {\r
> > >   if [ -z "$EMACS_SERVER" ]; then\r
> > >           EMACS_SERVER="notmuch-test-suite-$$"\r
> > >           # start a detached screen session with an emacs server\r
> > > -         screen -S "$EMACS_SERVER" -d -m "$TMP_DIRECTORY/run_emacs" \\r
> > > +         TERM=xterm dtach -n "$TMP_DIRECTORY/dtach-emacs-socket-$$" \\r
> > \r
> > Can you please give a more detailed explanation (and a comment) to\r
> > justify this workaround:\r
> > \r
> > * Is it actually needed for dtach or emacs?  If it is the latter, then\r
> >   it should be "dtach ... env TERM=xterm emacs ...", right?\r
> \r
> dtach doesn't care about TERM, it is emacs which does (as you noticed\r
> below). But it is bood that the TERM is same in dtach's environment.\r
> \r
> doing VAR=val command args instead of env VAR=val saves one execve(2)\r
> as the shell does the environment setting (after fork before execve)\r
\r
I do not think trying that saving one execve(2) call is what we are\r
after.  Especially in test suite.\r
\r
IMO setting TERM inside dtach makes it clear that we the workaround is\r
for emacs, not dtach (a comment should also make this clear).  Besides,\r
it is always better to minimize the context affected by a hack.\r
\r
> > \r
> > * Why should we care about systems that use terminals without\r
> >   corresponding terminfo installed?\r
> \r
> That's the issue I thought, armdragon on IRC thought etc. The 'xterm'\r
> is part of ncurses-base on Debian, at least (thanks armdragon for this\r
> info) and is it pretty much as base to every terminal emulator there is \r
> (I just launched gnome-terminal for testing, TERM was set to xterm --\r
> it may have been initially in environment but if gnome-terminal changed\r
> it before launching shell, my shell config will not change it)\r
> \r
\r
Sorry, this does not answer my question.\r
\r
My point is that if a system can not run "emacs -nw" because it lacks\r
corresponding terminfo, it is not notmuch who should work around these\r
issues.  If you are using such system by whatever reason, it is your\r
responsibility to run notmuch tests (and all other programs who need a\r
valid terminfo) with modified TERM value.\r
\r
Perhaps others disagree, but I think it is reasonable for notmuch test\r
suite to expect that "emacs -nw" works.\r
\r
> > * Why is it safe to use xterm?  Is there any standard that requires\r
> >   xterm info to be present on the system?\r
> \r
> xterm is not needed, it is just to tell emacs it can use it's display\r
> control sequences. more common alternative would have been vt100, just\r
> that if one ever wants to attach to that emacs session you he gets\r
> better experience (such as colors), using any terminal emulator, like rxvt.\r
> \r
\r
Sorry, I was misleading.  I did not mean that we are using xterm itself.\r
I am worried that we introduce dependency on xterm terminfo.  I\r
understand that it is part of ncurses-base package in Debian.  But\r
Debian is not the only system out there.  I think it is perfectly fine\r
for a system without X to lack xterm terminfo.  And by setting TERM to\r
"xterm" we break such systems.\r
\r
Using "vt100" seems like a better alternative.  But the question\r
remains: is there any standard requirement that a particular terminfo\r
must be available on a system?  If there is no such requirement, then we\r
can not override TERM value without breaking the test suite on some\r
weird (but perfectly fine) systems.\r
\r
> > I have tried running dtach with emacs in rxvt-unicode and it worked fine\r
> > without overriding TERM.\r
> \r
> you had originally suitable TERM to be passed to emacs.\r
> \r
\r
Sure. Why do not you have a valid TERM value with a corresponding\r
terminfo on your system?\r
\r
Sorry for being picky here.  I am fine with your patch except for this.\r
But TERM override feels wrong to me so far.  (Though, it is not me who\r
decides on the patches, perhaps others are fine with it and you can just\r
ignore me.)\r
\r
Regards,\r
  Dmitry\r
\r
> > \r
> > I have also tried running dtach with invalid TERM:\r
> > \r
> >   $ TERM=non-existing-term dtach -n s /bin/sh\r
> >   zsh: can't find terminal definition for non-existing-term\r
> > \r
> > No segfault, shell is started fine.\r
> \r
> Yes, dtach could not care less...\r
> \r
> > \r
> > But emacs does not work without a valid terminfo:\r
> > \r
> >   $ TERM=non-existing-term emacs -nw\r
> >   emacs: Terminal type non-existing-term is not defined.\r
> >   If that is not the actual type of terminal you have,\r
> >   use the Bourne shell command `TERM=... export TERM' (C-shell:\r
> >   `setenv TERM ...') to specify the correct type.  It may be necessary\r
> >   to do `unset TERMINFO' (C-shell: `unsetenv TERMINFO') as well.\r
> \r
> Yes, I got SIGSEGV --  strace -o oprfx -ff command args  is your friend :D\r
> \r
> > Regards,\r
> >   Dmitry\r
> \r
> Thanks for your analysis. \r
> \r
> Tomi\r
> \r
> > \r
> > > +                 "$TMP_DIRECTORY/run_emacs" \\r
> > >                   --no-window-system \\r
> > >                   --eval "(setq server-name \"$EMACS_SERVER\")" \\r
> > >                   --eval '(server-start)' \\r
> > > -- \r
> > > 1.7.7.1\r
> > > \r
> > > _______________________________________________\r
> > > notmuch mailing list\r
> > > notmuch@notmuchmail.org\r
> > > http://notmuchmail.org/mailman/listinfo/notmuch\r
> > _______________________________________________\r
> > notmuch mailing list\r
> > notmuch@notmuchmail.org\r
> > http://notmuchmail.org/mailman/listinfo/notmuch\r
> > \r