Re: Emacs cannot sort in *shell* buffer

Re: Emacs cannot sort in *shell* buffer

[Kenichi Handa]

In article <43E0CA5F-95E6-49CF-A7CE-1F80BBAD56A5@web.de>, Peter Dyballa <Peter_Dyballa@web.de> writes:

>> Can you reproduce it with CVS HEAD emacs?

> What do you mean with this technical term? Every few days I invoke  
> 'cvs up' and then re-compile GNU Emacs 22.0.50 or 23.0.0 -- are both  
> CVS HEAD Emacsen by this procedure?

Your first report about this bug contained this line:

In GNU Emacs 23.0.0.1 (powerpc-apple-darwin8.5.0, X toolkit, Xaw3d  
scroll bars)

So, I thought you faced with this bug while using Emacs
Ver.23 (i.e. emacs-unicode-2 branch).  But, as I can't
reproduce the bug, to clarify the bug is introduced in
emacs-unicode-2 or not, I asked you if you can face with the
same bug by Emacs Ver.22 (i.e. the main trank of CVS emacs
often referred as CVS HEAD version).

>>> Putting the block from *shell* buffer into *scratch* buffer or
>>> inserting it into a TeX file sorting works fine ...
>> 
>> ?? Then how and in which buffer (and mode) did you make the
>> list when sorting didn't work?

> The sorting fails in *shell* buffers.

I see.  That's the key infomation that your missed in the
report.  I confirmed that bug both with Ver.22 and Ver.23
and I think the attached patch should be installed.  But, as
this is the first time I read sort.el, I'm not 100%
confident.

Should I install it?

---
Kenichi Handa
handa@m17n.org

*** sort.el	23 Feb 2006 19:45:32 +0900	1.53
--- sort.el	09 Mar 2006 20:47:33 +0900	
***************
*** 121,126 ****
--- 121,127 ----
  (defun sort-build-lists (nextrecfun endrecfun startkeyfun endkeyfun)
    (let ((sort-lists ())
  	(start-rec nil)
+ 	(inhibit-field-text-motion t)
  	done key)
      ;; Loop over sort records.
      ;(goto-char (point-min)) -- it is the caller's responsibility to

Re: Emacs cannot sort in *shell* buffer

[Peter Dyballa]

Am 09.03.2006 um 12:57 schrieb Kenichi Handa:

>> The sorting fails in *shell* buffers.
>
> I see.  That's the key infomation that your missed in the
> report.  I confirmed that bug both with Ver.22 and Ver.23
> and I think the attached patch should be installed.  But, as
> this is the first time I read sort.el, I'm not 100%
> confident.
>
> Should I install it?

YES! It works. (I updated to the latest version of 22.0.50 and  
patched sort.el befoke making bootstrap.)

--
Greetings
   Pete

To be is to do.
                        -- I. Kant
To do is to be.
                        -- A. Sartre
Yabba-Dabba-Doo!
                        -- F. Flintstone

Re: Emacs cannot sort in *shell* buffer

[Kenichi Handa]

In article <D477B1E1-368E-4F65-BAF5-A3262D44A4A1@web.de>, Peter Dyballa <Peter_Dyballa@web.de> writes:

> Am 09.03.2006 um 12:57 schrieb Kenichi Handa:

>>> The sorting fails in *shell* buffers.
>> 
>> I see.  That's the key infomation that your missed in the
>> report.  I confirmed that bug both with Ver.22 and Ver.23
>> and I think the attached patch should be installed.  But, as
>> this is the first time I read sort.el, I'm not 100%
>> confident.
>> 
>> Should I install it?

> YES! It works. (I updated to the latest version of 22.0.50 and  
> patched sort.el befoke making bootstrap.)

As there's no objection, I've just committed this change.

2006-03-16  Kenichi Handa  <handa@m17n.org>

	* sort.el (sort-build-lists): Temporarily bind
	inhibit-field-text-motion to t.

---
Kenichi Handa
handa@m17n.org