Re: "search --path=directory/" is lame(-ish)

Subject: Re: "search --path=directory/" is lame(-ish)

Date: Wed, 29 Oct 2014 19:14:01 +0200

To: David Edmondson, notmuch@notmuchmail.org

Cc:

From: Tomi Ollila

On Wed, Oct 29 2014, David Edmondson <dme@dme.org> wrote:

> Adding a terminal slash to a directory name when using --path causes the
> search to fail. Removing the terminal slash produces results.
>
> Given that many shells will add the terminal slash during completion,
> this is lame(-ish).

Except zsh(1) which adds it, but when pressing enter to complete command
execution, the trailing slash is removed -- at least in my configuration...

... Nevertheless, I agree that the "feature" we have is at least a bit lame ;D

Tomi

• Previous message (by thread): Re: "search --path=directory/" is lame(-ish)

Thread:

• David Edmondson—"search --path=directory/" is lame(-ish) [inbox, notmuch::bug, notmuch::fixed, unread]
  • Tomi Ollila—Re: "search --path=directory/" is lame(-ish) [inbox, unread]
  • Jani Nikula—Re: "search --path=directory/" is lame(-ish) [inbox, unread]
    • David Edmondson—Re: "search --path=directory/" is lame(-ish) [inbox, unread]
      • David Bremner—Re: "search --path=directory/" is lame(-ish) [inbox, unread]
  • David Bremner—Re: "search --path=directory/" is lame(-ish) [inbox, unread]
    • David Bremner—strip trailing '/' from path/folder query. [inbox, unread]
      • David Bremner—[PATCH 1/6] test: sanitize generated message files names [inbox, notmuch::fix, notmuch::patch, notmuch::pushed, unread]
      • David Bremner—[PATCH 2/6] test: known broken tests for trailing / in path search (infix) [inbox, notmuch::fix, notmuch::patch, notmuch::pushed, unread]
      • David Bremner—[PATCH 3/6] test/sexp: tests for path, folder, including trailing '/' (sexp) [inbox, notmuch::fix, notmuch::patch, notmuch::pushed, unread]
      • David Bremner—[PATCH 4/6] lib: drop trailing slash for path and folder searches (infix) [inbox, notmuch::fix, notmuch::patch, notmuch::pushed, unread]
      • David Bremner—[PATCH 5/6] test: add multiple path, folder sexp query tests [inbox, notmuch::fix, notmuch::patch, notmuch::pushed, unread]
      • David Bremner—[PATCH 6/6] lib: strip trailing '/' from pathnames (sexp queries). [inbox, notmuch::fix, notmuch::patch, notmuch::pushed, unread]
      • David Bremner—Re: strip trailing '/' from path/folder query. [inbox, unread]
      • David Bremner—Re: strip trailing '/' from path/folder query. [inbox, unread]
  • David Bremner—Re: "search --path=directory/" is lame(-ish) [inbox, unread]