unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* package-quickstart and info:
@ 2021-01-07 21:14 T.V Raman
  2021-01-07 23:16 ` Stefan Monnier
  0 siblings, 1 reply; 4+ messages in thread
From: T.V Raman @ 2021-01-07 21:14 UTC (permalink / raw)
  To: emacs-devel

While debugging something else with force-load-messages set to t, I
noticed that loading package-quickstart.el loads info -- is there a
good reason for this?

-- 

Thanks,

--Raman
♉ Id: kg:/m/0285kf1  🦮

-- 

Thanks,

--Raman
♉ Id: kg:/m/0285kf1  🦮



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: package-quickstart and info:
  2021-01-07 21:14 package-quickstart and info: T.V Raman
@ 2021-01-07 23:16 ` Stefan Monnier
  2021-01-07 23:40   ` T.V Raman
  0 siblings, 1 reply; 4+ messages in thread
From: Stefan Monnier @ 2021-01-07 23:16 UTC (permalink / raw)
  To: T.V Raman; +Cc: emacs-devel

> While debugging something else with force-load-messages set to t, I
> noticed that loading package-quickstart.el loads info -- is there a
> good reason for this?

IIRC when a package includes an Info manual, we load `info` before
modifying the `Info-directory-list`.  This could be improved to avoid
loading `info`.

The place where this happens in the non-quickstart case is
`package-activate-1` and in `package-quickstart-refresh` you can see
similar code being added to the quickstart file.

IIRC the problem is that depending on whether 
`info-initialize` has already been called or not, we need to add the
directory either to `Info-directory-list` or to
`Info-default-directory-list`.

Patches welcome (and if you need more info about the precise
relationship between those two Info*directory-list vars, I think Eli is
the one who knows all about it).


        Stefan




^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: package-quickstart and info:
  2021-01-07 23:16 ` Stefan Monnier
@ 2021-01-07 23:40   ` T.V Raman
  2021-01-08  2:12     ` Stefan Monnier
  0 siblings, 1 reply; 4+ messages in thread
From: T.V Raman @ 2021-01-07 23:40 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: emacs-devel

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset=gb18030, Size: 1146 bytes --]

Stefan Monnier <monnier@iro.umontreal.ca> writes:

Given that this is all happening from early-init.el, I suspect info
always gets loaded. 
>> While debugging something else with force-load-messages set to t, I
>> noticed that loading package-quickstart.el loads info -- is there a
>> good reason for this?
>
> IIRC when a package includes an Info manual, we load `info` before
> modifying the `Info-directory-list`.  This could be improved to avoid
> loading `info`.
>
> The place where this happens in the non-quickstart case is
> `package-activate-1` and in `package-quickstart-refresh` you can see
> similar code being added to the quickstart file.
>
> IIRC the problem is that depending on whether 
> `info-initialize` has already been called or not, we need to add the
> directory either to `Info-directory-list` or to
> `Info-default-directory-list`.
>
> Patches welcome (and if you need more info about the precise
> relationship between those two Info*directory-list vars, I think Eli is
> the one who knows all about it).
>
>
>         Stefan
>

-- 

Thanks,

--Raman
7©4 Id: kg:/m/0285kf1  •0Ü8



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: package-quickstart and info:
  2021-01-07 23:40   ` T.V Raman
@ 2021-01-08  2:12     ` Stefan Monnier
  0 siblings, 0 replies; 4+ messages in thread
From: Stefan Monnier @ 2021-01-08  2:12 UTC (permalink / raw)
  To: T.V Raman; +Cc: emacs-devel

> Given that this is all happening from early-init.el, I suspect info
> always gets loaded. 

Hmm... I don't understand how that relates to what I said,


        Stefan


>>> While debugging something else with force-load-messages set to t, I
>>> noticed that loading package-quickstart.el loads info -- is there a
>>> good reason for this?
>>
>> IIRC when a package includes an Info manual, we load `info` before
>> modifying the `Info-directory-list`.  This could be improved to avoid
>> loading `info`.
>>
>> The place where this happens in the non-quickstart case is
>> `package-activate-1` and in `package-quickstart-refresh` you can see
>> similar code being added to the quickstart file.
>>
>> IIRC the problem is that depending on whether 
>> `info-initialize` has already been called or not, we need to add the
>> directory either to `Info-directory-list` or to
>> `Info-default-directory-list`.
>>
>> Patches welcome (and if you need more info about the precise
>> relationship between those two Info*directory-list vars, I think Eli is
>> the one who knows all about it).
>>
>>
>>         Stefan
>>




^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2021-01-08  2:12 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-07 21:14 package-quickstart and info: T.V Raman
2021-01-07 23:16 ` Stefan Monnier
2021-01-07 23:40   ` T.V Raman
2021-01-08  2:12     ` Stefan Monnier

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).