From: Jani Nikula <jani@nikula.org>
To: Chris Gray <chrismgray@gmail.com>, notmuch@notmuchmail.org
Subject: Re: [PATCH] emacs: Use notmuch-command variable in process-lines.
In-Reply-To: <87hb1m6cag.fsf@gmail.com>
References: <87hb1m6cag.fsf@gmail.com>
User-Agent: Notmuch/0.10+51~gef3ae74 (http://notmuchmail.org) Emacs/23.3.1
	(i686-pc-linux-gnu)
Date: Tue, 29 Nov 2011 21:15:13 +0200
Message-ID: <87zkfesptq.fsf@nikula.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Precedence: list


Good catch, looks good to me.

BR,
Jani.

On Tue, 29 Nov 2011 10:58:31 -0700, Chris Gray <chrismgray@gmail.com> wrote:
> The process-lines function calls the notmuch binary.  The location of
> the binary may have been customized by the user, so it is better to
> use the customized location rather than allowing the process-lines
> function to search the user's PATH for the binary.
> ---
>  emacs/notmuch.el |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/emacs/notmuch.el b/emacs/notmuch.el
> index f15a75b..8936149 100644
> --- a/emacs/notmuch.el
> +++ b/emacs/notmuch.el
> @@ -885,7 +885,7 @@ PROMPT is the string to prompt with."
>  		      "subject:" "attachment:")
>  		(mapcar (lambda (tag)
>  			  (concat "tag:" tag))
> -			(process-lines "notmuch" "search" "--output=tags" "*")))))
> +			(process-lines notmuch-command "search" "--output=tags" "*")))))
>      (let ((keymap (copy-keymap minibuffer-local-map))
>  	  (minibuffer-completion-table
>  	   (completion-table-dynamic
> -- 
> 1.7.7.3
> 
> 
> _______________________________________________
> notmuch mailing list
> notmuch@notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch