a bug report on info.el

a bug report on info.el

[Hiroshi Okagawa]

Hi,

I've found a bug in info.el which comes with Emacs 22.0.50.1
and want to report it.

Its info-initialize is written as:
> (defun info-initialize ()
>   "Initialize `Info-directory-list', if that hasn't been done yet."
>   (unless Info-directory-list
>     (let ((path (getenv "INFOPATH")))
>       (setq Info-directory-list
> 	    (prune-directory-list
> 	     (if path
> 		 (if (string-match ":\\'" path)
> 		     (append (split-string (substring path 0 -1)
> 					   (regexp-quote path-separator))
> 			     (Info-default-dirs))
> 		   (split-string path (regexp-quote path-separator)))
> 	       (Info-default-dirs)))))))

However, I think last two lines should be written like this:

	  (append   (split-string path (regexp-quote path-separator)))
  (Info-default-dirs)))

Otherwise, the last (Info-default-dirs) won't be appended to
Info-directory-list.

I think this bug appeared in Revision 1.343 of info.el for the first time.

Best regards,
--
Hiroshi Okagawa
okagawa@gmail.com

Re: a bug report on info.el

[Richard M. Stallman]

    However, I think last two lines should be written like this:

	      (append   (split-string path (regexp-quote path-separator)))
      (Info-default-dirs)))

    Otherwise, the last (Info-default-dirs) won't be appended to
    Info-directory-list.

You are supposed to end INFOPATH with a colon
if you want the defaults added at the end.