reassign 399483 xemacs21-basesupport 2006.05.10-1
retitle 399483 xemacs21-basesupport: dired-use-ls-dired switch doesn't work 
properly
tags 399483 patch

severity 403702 normal
reassign 403702 xemacs21-basesupport 2006.05.10-1
retitle 403702 xemacs21-basesupport: garbage on screen in dired mode

merge 399483 403702
thanks


* Vladimir G. Troitsky <[EMAIL PROTECTED]> [2006-12-18 22:01]:

> Package: xemacs21
> Version: 21.4.19-2
> Severity: minor
> 
> There is garbage on the screen in dired mode. Occasionally I get one or
> several copies of the following message in the beginning or the middle
> of a dired fuffer:
> 
> //DIRED-OPTIONS// --quoting-style=literal
> 
> Also, at the end of the buffer I get a long line like this:
> //DIRED// 62 63 112 114 163 172 221 234 283 289 338 349 398 406 455...

This bug has been already reported as #399483 (hence the merge command
above).  Actually, this problem occurs in the xemacs21-basesupport package,
not in xemacs21 (hence the reassign and retitle commands above).

The changes proposed by Peter Chubb in his report seem to work correctly.
Therefore, I took the liberty of producing an appropriate dpatch (attached
below) for the the xemacs21-packages source package (hence the "patch" tag).
Now, the maintainer has no excuse to delaying the fix for the bug ;-)

In waiting for the fixed package, I have added the following code to my
~/.emacs:

(add-hook 'dired-load-hook
  (lambda ()
    (set-variable 'dired-use-ls-dired
      (and (string-match "gnu" system-configuration)
           ;; Only supported for XEmacs >= 21.5 and GNU Emacs >= 21.4 (I think)
           (if (featurep 'xemacs)
               (and
                (fboundp 'emacs-version>=)
                (emacs-version>= 21 5))
             (and (boundp 'emacs-major-version)
                  (boundp 'emacs-minor-version)
                  (or (> emacs-major-version 21)
                      (and (= emacs-major-version 21)
                           (>= emacs-minor-version 4)))))))))

-- 
Rafael
#! /bin/sh /usr/share/dpatch/dpatch-run
## 02_dired-use-ls-dired.dpatch by Rafael Laboissiere <[EMAIL PROTECTED]>
##
## DP: Fixed logic for the default value of dired-use-ls-dired, which is
## DP: failing in xemacs21.  This should fix Bug#399483 and Bug#403702

@DPATCH@

--- xemacs21-packages-2006.05.10.orig/xemacs-packages/dired/dired.el
+++ xemacs21-packages-2006.05.10/xemacs-packages/dired/dired.el
@@ -210,14 +210,15 @@
 (defcustom dired-use-ls-dired
   (and (string-match "gnu" system-configuration)
        ;; Only supported for XEmacs >= 21.5 and GNU Emacs >= 21.4 (I think)
-       (or (and (featurep 'xemacs)
-               (fboundp 'emacs-version>=)
-               (emacs-version>= 21 5))
-          (and (boundp 'emacs-major-version)
-               (boundp 'emacs-minor-version)
-               (or (> emacs-major-version 21)
-                   (and (= emacs-major-version 21)
-                        (>= emacs-minor-version 4))))))
+       (if (featurep 'xemacs)
+           (and
+            (fboundp 'emacs-version>=)
+            (emacs-version>= 21 5))
+         (and (boundp 'emacs-major-version)
+              (boundp 'emacs-minor-version)
+              (or (> emacs-major-version 21)
+                  (and (= emacs-major-version 21)
+                       (>= emacs-minor-version 4))))))
   "Non-nil means Dired should use `ls --dired'."
   :type 'boolean
   :group 'dired-programs)

Reply via email to