unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: "Alfred M. Szmidt" <ams@gnu.org>
To: Lars Ingebrigtsen <larsi@gnus.org>
Cc: rgm@gnu.org, 786707@bugs.debian.org, 27933@debbugs.gnu.org,
	steve@einval.com, 786707-forwarded@bugs.debian.org,
	rlb@defaultvalue.org
Subject: bug#27933: emacs25: default info dir initialization slow with remote filesystems
Date: Thu, 14 Apr 2022 02:51:56 -0400	[thread overview]
Message-ID: <E1netKe-0007CP-Pq@fencepost.gnu.org> (raw)
In-Reply-To: <875yndp086.fsf@gnus.org> (message from Lars Ingebrigtsen on Wed,  13 Apr 2022 15:43:53 +0200)

   >    > Standard info does the right thing, and shows both local (those listed
   >    > by INFOPATH) dir files, and the system ones.
   >    >
   >    > Reverting the change (5f7db63a93f0c670d6f88dd24f922462abce7d8c) makes
   >    > things work as expected.
   >
   >    "Alfred M. Szmidt" <ams@gnu.org> writes:
   >
   >    > Sorry -- too many distractions while writing, I missed to add one
   >    > additional (important) note.  Started with -Q and then evaluating:
   >    >
   >    >   (add-hook 'Info-mode-hook
   >    >   	  (lambda ()
   >    >       	    (setq Info-additional-directory-list Info-default-directory-list)))
   >    >
   >    > before entering info.
   >
   >    Do you need to have both INFOPATH set and this hook set to reproduce the
   >    problem?
   >
   > Yes, exactly.

   The code here is pretty convoluted, but reading it, I think we just
   don't support doing something like that any more (since
   Info-default-directory-list isn't populated any more at this point).
   Which is why the changes here are listed under "Incompatible changes in
   Emacs 29.1".

Yes, it is convulted. :-/

But I think the situation gets slightly more convulted with this
change.  Now, there is no easy way of adding back the default
directories where Emacs searches.

The docstring could need a touch up, mentioning how
Info-default-directory-list and Info--default-directory-list work
together or hint towards it. The text is slightly misleading, since
the following is not really the case:

  This variable is used as the default for initializing
  `Info-default-directory-list' when Info is started, unless the
  environment variable INFOPATH is set.


I've tried to get the same behaviour that I had, namley that of
INFOPATH + Info-default-directory-list (in that order).  The only
thing that comes to mind is explicitly calling
Info--default-directory-list to set it up; which doesn't seem very
nice.

  (add-hook 'Info-mode-hook
  	  (lambda ()
  	    (setq Info-additional-directory-list 
  		  (if (fboundp 'Info--default-directory-list)
  		      (Info--default-directory-list)
  		    Info-default-directory-list))))
	      
If one could re-do all of this, having Info-additional-directory-list
be initialized to INFOPATH would have made most sense.  There can be
very few reasons to delete the default Emacs info directory locations,
and having INFOPATH override those always seemed backwards, and not
what stand-alone Info does -- so while the Emacs documentation says
that one should customize INFOPATH for both, the customizations aren't
really the same :-(

PS.  Shouldn't bugs become unarchived when someone replies to an
already archived one?  The discussion here is now lost in the ether...





  parent reply	other threads:[~2022-04-14  6:51 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20150524155316.28851.58585.reportbug@tack.local>
2017-08-03 13:59 ` bug#27933: emacs25: default info dir initialization slow with remote filesystems Rob Browning
2017-08-03 15:29   ` Glenn Morris
2017-08-03 16:39     ` Steve McIntyre
2017-08-03 16:41       ` Glenn Morris
2017-08-03 16:49         ` Steve McIntyre
     [not found]         ` <20170803164913.wkuvwws3ry4l4ofy@tack.einval.com>
2017-08-03 17:25           ` Eli Zaretskii
2017-08-03 17:35             ` Glenn Morris
2017-08-03 18:31               ` Eli Zaretskii
2017-08-03 21:37                 ` Glenn Morris
2022-02-03 20:04               ` Lars Ingebrigtsen
2022-03-04 16:36                 ` Lars Ingebrigtsen
2022-03-14  0:57                   ` Steve McIntyre
     [not found]                   ` <E1neaLG-0000XL-2e@fencepost.gnu.org>
     [not found]                     ` <87h76xp50o.fsf@gnus.org>
     [not found]                       ` <E1neco7-0006BJ-8F@fencepost.gnu.org>
     [not found]                         ` <875yndp086.fsf@gnus.org>
2022-04-14  6:51                           ` Alfred M. Szmidt [this message]
2022-04-14 13:41                             ` Lars Ingebrigtsen
2022-04-14 14:58                               ` Drew Adams
2022-04-14 15:43                               ` Glenn Morris

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=E1netKe-0007CP-Pq@fencepost.gnu.org \
    --to=ams@gnu.org \
    --cc=27933@debbugs.gnu.org \
    --cc=786707-forwarded@bugs.debian.org \
    --cc=786707@bugs.debian.org \
    --cc=larsi@gnus.org \
    --cc=rgm@gnu.org \
    --cc=rlb@defaultvalue.org \
    --cc=steve@einval.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).