unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* [chris@mightyreason.com: recent CVS has aspell error]
@ 2006-01-25  3:31 Richard M. Stallman
  2006-01-26 11:36 ` Agustin Martin
  0 siblings, 1 reply; 3+ messages in thread
From: Richard M. Stallman @ 2006-01-25  3:31 UTC (permalink / raw)


Would someone please DTRT about this, and ack?

------- Start of forwarded message -------
Date: Sun, 22 Jan 2006 23:04:11 +0000
From: Chris Kuklewicz <chris@mightyreason.com>
MIME-Version: 1.0
To: emacs-pretest-bug@gnu.org
Content-Type: text/plain; charset=ISO-8859-1
Subject: recent CVS has aspell error
Reply-To: chris@mightyreason.com
X-Spam-Status: No, hits=0.1 required=5.0 tests=RCVD_IN_SORBS autolearn=no 
	version=2.63

This bug report will be sent to the Free Software Foundation,
not to your local site managers!
Please write in English if possible, because the Emacs maintainers
usually do not have translators to read other languages for them.

Your bug report will be posted to the emacs-pretest-bug@gnu.org mailing
list.

Please describe exactly what actions triggered the bug
and the precise symptoms of the bug:

Short version : Emacs can no longer use spell checking

I have been using a CVS derived carbon build of emacs on OS X for some
time, and this new version has a bug running /usr/local/bin/aspell:

In GNU Emacs 22.0.50.1 (powerpc-apple-darwin8.4.0)
 of 2006-01-19 on Reason.local
X server distributor `Apple Computers', version 10.4.4
configured using `configure '--without-x' '--with-kerboros'
'--with-kerberos5' '--prefix=/usr/local''

I have been using this version of aspell for some time:

/usr/local/bin/aspell -v
@(#) International Ispell Version 3.1.20 (but really Aspell 0.60.2)

The error occurs when Emacs tries to start "ispell".  The *Messages*
buffer shows this when I call "ispell-buffer"

Loading ispell...done
Starting new Ispell process...
Spell checking *scratch* using aspell with default dictionary...
Spell-checking using aspell with default dictionary done
ispell-get-line: Wrong type argument: stringp, nil

The value for ispell-program-name is "/usr/local/bin/aspell", derived
from the default list expression:
(or
 (locate-file "aspell" exec-path exec-suffixes 'file-executable-p)
 "ispell")

This error prevents me from spell checking anything within Emacs.  This
is a show stopper for documents.

This error is independent of my ~/.emacs file (I moved it out of the way
and it had no effect). If my ~/.aspell.conf would be interesting, then I
could send it.  But I tried moving it out of the way and nothing improved.

I checked that this error is still present in CVS from now:
$ date
Sun Jan 22 23:00:31 GMT 2006

Looking as emacs/lisp/ChangeLog indicates that ispell and flyspell code
has been changed recently.  So I expect that this is mainly bad timing
on my part.

This error did not occur for my previous checkout(s) from CVS.  In
particular the next-most-recent which I have now reverted to is this
vintage:

In GNU Emacs 22.0.50.1 (powerpc-apple-darwin8.2.0)
 of 2005-07-30 on Reason.local
X server distributor `Apple Computers', version 10.4.4
configured using `configure '--without-x' '--prefix=/usr/local''

This older works flawlessly.  It finds /usr/local/bin/aspell and calls
it without a hitch.  I cannot see any difference in how Emacs is
configured between the old and new versions.

Cheers,
  Chris Kuklewicz




_______________________________________________
emacs-pretest-bug mailing list
emacs-pretest-bug@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-pretest-bug
------- End of forwarded message -------

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [chris@mightyreason.com: recent CVS has aspell error]
@ 2006-01-25 20:51 Kevin Rodgers
  0 siblings, 0 replies; 3+ messages in thread
From: Kevin Rodgers @ 2006-01-25 20:51 UTC (permalink / raw)


[-- Attachment #1: Type: text/plain, Size: 158 bytes --]

 
 

__________________________________________________
Do You Yahoo!?
Tired of spam?  Yahoo! Mail has the best spam protection around 
http://mail.yahoo.com 

[-- Attachment #2: Type: message/rfc822, Size: 2809 bytes --]

From: Chris Kuklewicz <chris@mightyreason.com>
To: Kevin Rodgers <ihs_4664@yahoo.com>
Subject: Re: [chris@mightyreason.com: recent CVS has aspell error]
Date: Wed, 25 Jan 2006 20:14:15 +0000

Kevin Rodgers wrote:
> Richard M. Stallman wrote:
>> Would someone please DTRT about this, and ack?
> 
> This same error was reported and discussed recently on help-gnu-emacs; see
> http://lists.gnu.org/archive/html/help-gnu-emacs/2005-12/msg00713.html
> http://lists.gnu.org/archive/html/help-gnu-emacs/2005-12/msg00759.html
> 
> So it seems likely that ispell-dictionary has been set incorrectly,
> perhaps in default.el or site-start.el.
> 
> What does `C-h v ispell-dictionary' display?  

This variable is nil in the older working version of cvs-emacs

This variable does not exist in the newer broken version of cvs-emacs

> If you start emacs with
> the --no-init-file and --no-site-file command line options, does the
> problem still occur?

Yes, the problem is identical if I use those options.

I have read the email messages you pointed to and found a workaround:  If I run
the command ispell-change-dictionary and select one of the options (e.g.
american) then spell checking starts working.  This command creates
ispell-dictionary and sets it to nil and it creates the previous missing
ispell-local-dictionary and sets it to american.

This can be made permanent: Adding (setq ispell-dictionary "american") to my
~/.emacs causes ispell-dictionary to be set properly and it leaves
ispell-local-dictionary uncreated.  After performing ispell-buffer, I notice
ispell-local-dictionary has been created and set to nil.

Conclusion: It seems there is a problem in the latest cvs version where
ispell-dictionary is not being created by default (and set to nil) and this
prevents ispell from working for me.

[-- Attachment #3: Type: text/plain, Size: 142 bytes --]

_______________________________________________
Emacs-devel mailing list
Emacs-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-devel

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [chris@mightyreason.com: recent CVS has aspell error]
  2006-01-25  3:31 [chris@mightyreason.com: recent CVS has aspell error] Richard M. Stallman
@ 2006-01-26 11:36 ` Agustin Martin
  0 siblings, 0 replies; 3+ messages in thread
From: Agustin Martin @ 2006-01-26 11:36 UTC (permalink / raw)


[-- Attachment #1: Type: text/plain, Size: 1084 bytes --]

On Tue, Jan 24, 2006 at 10:31:27PM -0500, Richard M. Stallman wrote:
> Would someone please DTRT about this, and ack?
> 
> ------- Start of forwarded message -------
> From: Chris Kuklewicz <chris at mightyreason.com>
> 
> /usr/local/bin/aspell -v
> @(#) International Ispell Version 3.1.20 (but really Aspell 0.60.2)
> 
> The error occurs when Emacs tries to start "ispell".  The *Messages*
> buffer shows this when I call "ispell-buffer"
> 
> Loading ispell...done
> Starting new Ispell process...
> Spell checking *scratch* using aspell with default dictionary...
> Spell-checking using aspell with default dictionary done
> ispell-get-line: Wrong type argument: stringp, nil

This seems the same problem described by Stefan Monnier in the "Problems
with ispell/flyspell" thread when no english aspell dict is available,

http://lists.gnu.org/archive/html/emacs-devel/2005-12/msg01343.html

where he proposed making sure a sane default is available by using first
entry of ispell-dictionary-alist-1 when nothing else works.

Attached patch seems to take care of that,

-- 
Agustin

[-- Attachment #2: ispell.el.safe-aspell-default-dict.diff --]
[-- Type: text/plain, Size: 512 bytes --]

--- ispell.el.orig0	2005-12-20 15:06:29.000000000 +0100
+++ ispell.el	2006-01-12 19:40:32.000000000 +0100
@@ -913,7 +913,9 @@
     (ispell-aspell-add-aliases)
     ;; Add a default entry
     (let* ((english-dict (assoc "en" ispell-dictionary-alist))
-	   (default-dict (cons nil (cdr english-dict))))
+	   (default-dict
+	     (cons nil (or (cdr english-dict)
+			   (cdr (car ispell-dictionary-alist-1))))))
       (push default-dict ispell-dictionary-alist))
     (setq ispell-have-aspell-dictionaries t)))
 

[-- Attachment #3: Type: text/plain, Size: 142 bytes --]

_______________________________________________
Emacs-devel mailing list
Emacs-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-devel

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2006-01-26 11:36 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2006-01-25  3:31 [chris@mightyreason.com: recent CVS has aspell error] Richard M. Stallman
2006-01-26 11:36 ` Agustin Martin
  -- strict thread matches above, loose matches on Subject: below --
2006-01-25 20:51 Kevin Rodgers

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.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).