On Thu, Nov 08, 2007 at 07:17:06PM +0000, Barak A. Pearlmutter wrote:
> This shadowing of the native emacs22 flyspell causes a number of
> actual errors in my emacs. For instance:
>
> 1. Turn-on-flyspell is registered as an autoloading function, but
> fails to correctly autoload because the wrong flyspell.el is
> loaded, causing a trip to the debugger and an inability to visit a
> file when the extremely reasonable seeming
>
> (when (fboundp 'turn-on-flyspell)
> (add-hook 'text-mode-hook 'turn-on-flyspell)))
>
> is present in .emacs. Having a function that is listed as
> autoloading and but then doesn't is a rather nasty bug.
> (You can see this bug with "emacs22 -q & M-: (turn-on-flyspell)",
> so it is not a .emacs issue.)
The work to add current CVS (emacs 22 or later if fixes bugs) ispell.el and
flyspell.el is mostly done, but the problem you mention is trivial to fix
and should have deserved a bug report. Unless next upload already includes
the most recent ispell.el and flyspell.el I will fix it there for both
turn-{on,off}-flyspell. We are currently experiencing some hardware
problems, so cannot promise where that will be.
> 2. Because ispell-maybe-find-aspell-dictionaries is referred to in the
> emacs22 flyspell.el but not in the dictionaries-common flyspell.el
> and is defined in the emacs22 ispell.el but not in the
> dictionaries-common ispell.el, explicit loading of pre-compiled
> system library files can lead to failure of some very basic
> functionality.
If you want to explicitely load things, make sure you do that in the correct
ordering, first ispell, then flyspell (remember that flyspell requires
ispell).
>
> 3. With a suitable file, emacs22 and emacs-snapshot can crash
> (segmentation violation) when flyspell-mode is turned on. I have
> not yet gone to the trouble of tracking this down in detail; it is
> a hassle without debugging support compiled in.
If emacs-snapshot is failing, this problem has nothing to do with this bug
report. emacs-snapshot does not use *at all* the dictionaries-common stuff,
When starting emacs-snapshot or byte-compiling for it you should have read
something like
"Skipping dictionaries-common setup for emacs-snapshot"
"install/dictionaries-common: Skipping byte-compilation for emacs-snapshot"
Let me know if this is no longer true, so I contact emacs-snapshot
maintainer about this.
> It seems to me that the current situation results in (as mentioned in
> earlier messages on this report) extremely severe slowdowns, and also
> the loss of various bits of documented functionality.
I only noticed some slowdown, but not at all as severe as the original
reporter pointed out. However I did not check extensively. And none of us
could understand where that difference comes from. Not very important if
files are updated, but something about what I remain curious.
> For this
> reason, it would be a good idea to figure out a way of using the
> upstream flyspell.el and ispell.el, with some sort of minor tweaks to
> get Debian dictionaries incorporated. Right now, the situation is a
> serious problem, particularly since it requires substantial expertise
> to figure out what the problem is and figure out how to disable
> loading of the Debian versions that, by default, shadow the more
> modern ones from upstream.
/etc/emacs/site-start.d/50dictionaries-common.el
It is the only emacs related conffile, although I admit should have
documented this better in dictionaries-common README.emacs. I will change
this to be something like
(setq emacs-flavors-skip-list '(emacs19 emacs20 emacs-snapshot))
(if (member debian-emacs-flavor emacs-flavors-skip-list)
...
so is more clear.
> I would suggest reconsideration of whether this is a "wishlist" issue.
> Documented interfaces are being broken, and there is rather extreme
> performance degradation. I know some people that don't use the
> Debianized emacs packages because they hit these issues and that was
> the first workaround that they thought of.
The only remaining things are related to Debian files reorganization as well
as to problems currently present in upstream versions. I wanted to have them
fixed and tested here before submitting a bug report to the emacs-cvs mailing
list, so Debian diff patches are as shortened as possible.
You can look at the fsf-emacs branch at our CVS,
http://alioth.debian.org/plugins/scmcvs/cvsweb.php/?cvsroot=dict-common
It started at 0.81.2, so all further changes in main branch are not included.
--
Agustin
--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
