From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Magnus Henoch Newsgroups: gmane.emacs.devel Subject: Re: flyspell bug Date: Wed, 27 Apr 2005 16:04:45 +0200 Message-ID: <87br80tiwy.fsf@zemdatav.stor.no-ip.org> References: <17005.64537.459392.453517@notch.amtp.cam.ac.uk> <17007.22441.167975.862337@notch.amtp.cam.ac.uk> NNTP-Posting-Host: main.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Trace: sea.gmane.org 1114610695 7418 80.91.229.2 (27 Apr 2005 14:04:55 GMT) X-Complaints-To: usenet@sea.gmane.org NNTP-Posting-Date: Wed, 27 Apr 2005 14:04:55 +0000 (UTC) Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Wed Apr 27 16:04:51 2005 Return-path: Original-Received: from lists.gnu.org ([199.232.76.165]) by ciao.gmane.org with esmtp (Exim 4.43) id 1DQn8Z-0002Tn-Nb for ged-emacs-devel@m.gmane.org; Wed, 27 Apr 2005 16:03:36 +0200 Original-Received: from localhost ([127.0.0.1] helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1DQnEV-0005yo-Dk for ged-emacs-devel@m.gmane.org; Wed, 27 Apr 2005 10:09:43 -0400 Original-Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1DQnEH-0005yM-Ph for emacs-devel@gnu.org; Wed, 27 Apr 2005 10:09:29 -0400 Original-Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1DQnEE-0005wU-AS for emacs-devel@gnu.org; Wed, 27 Apr 2005 10:09:29 -0400 Original-Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1DQnEE-0005Ij-4Q for emacs-devel@gnu.org; Wed, 27 Apr 2005 10:09:26 -0400 Original-Received: from [80.91.229.2] (helo=ciao.gmane.org) by monty-python.gnu.org with esmtp (TLS-1.0:RSA_AES_128_CBC_SHA:16) (Exim 4.34) id 1DQnGh-0001Yo-QM for emacs-devel@gnu.org; Wed, 27 Apr 2005 10:12:00 -0400 Original-Received: from list by ciao.gmane.org with local (Exim 4.43) id 1DQn5O-0001ys-6z for emacs-devel@gnu.org; Wed, 27 Apr 2005 16:00:18 +0200 Original-Received: from h170n1fls23o1074.bredband.comhem.se ([213.67.239.170]) by main.gmane.org with esmtp (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Wed, 27 Apr 2005 16:00:18 +0200 Original-Received: from mange by h170n1fls23o1074.bredband.comhem.se with local (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Wed, 27 Apr 2005 16:00:18 +0200 X-Injected-Via-Gmane: http://gmane.org/ Original-To: emacs-devel@gnu.org Original-Lines: 43 Original-X-Complaints-To: usenet@sea.gmane.org X-Gmane-NNTP-Posting-Host: h170n1fls23o1074.bredband.comhem.se User-Agent: Gnus/5.110003 (No Gnus v0.3) Emacs/22.0.50 (berkeley-unix) Cancel-Lock: sha1:jPeKgEkzr6zpzFNVJVmMZY3LkEk= X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:36451 X-Report-Spam: http://spam.gmane.org/gmane.emacs.devel:36451 Stephen Eglen writes: > Having looked at ispell.el, I don't think they are inconsistent; I > think the logic of ispell.el is that if ispell-program-name is > "ispell", it just checks the output of ispell -v: > > $ ispell -v > @(#) International Ispell Version 3.1.20 (but really Aspell 0.50.5) > > and that parenthetic comment triggers ispell-really-aspell to t. > > I suggest we just fix flyspell to always use the -l flag in this case, > rather than "list". I disagree. I was the one who asked for the change from -l to list in the first place, because it wouldn't work for me. The situation seems to be that your version of aspell installs a compatibility "ispell" command which supports the -l option meaning "list", while my version of aspell (0.60.2) doesn't install any such alias and only understands "list" when invoked as aspell. (-l specifies the language to use) So it seems that the variable ispell-really-aspell is moderately useful, as there are three situations: ispell, aspell in ispell compatibility mode, and "real" aspell. The following patch eliminates the middle case. The documentation of aspell 0.50.5 claims that it supports the "list" command, but I haven't tested it. --- orig/lisp/textmodes/ispell.el +++ mod/lisp/textmodes/ispell.el @@ -301,7 +301,8 @@ :type 'integer :group 'ispell) -(defcustom ispell-program-name "ispell" +(defcustom ispell-program-name (or (executable-find "aspell") + "ispell") "Program invoked by \\[ispell-word] and \\[ispell-region] commands." :type 'string :group 'ispell) Magnus