unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Re: master cf672c6: * lisp/emacs-lisp/package.el (package-activate-all): Another tweak
       [not found] ` <20210106022610.4BD5720CB7@vcs0.savannah.gnu.org>
@ 2021-01-06  8:57   ` Lars Ingebrigtsen
  2021-01-06 15:40     ` Stefan Monnier
  0 siblings, 1 reply; 6+ messages in thread
From: Lars Ingebrigtsen @ 2021-01-06  8:57 UTC (permalink / raw)
  To: emacs-devel; +Cc: Stefan Monnier

monnier@iro.umontreal.ca (Stefan Monnier) writes:

>     * lisp/emacs-lisp/package.el (package-activate-all): Another tweak
>
>     `package-quickstart.el` files presume `package-activated-list`
>     is a bound variable, so make sure this is the case even when `package.el` is
>     not yet loaded.

I'm still getting

In command-line:
startup.el:1233:28: Warning: reference to free variable `package-user-dir'
startup.el:1233:45: Warning: reference to free variable
    `package-directory-list'

when building Emacs...

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no



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

* Re: master cf672c6: * lisp/emacs-lisp/package.el (package-activate-all): Another tweak
  2021-01-06  8:57   ` master cf672c6: * lisp/emacs-lisp/package.el (package-activate-all): Another tweak Lars Ingebrigtsen
@ 2021-01-06 15:40     ` Stefan Monnier
  2021-01-07 11:36       ` Lars Ingebrigtsen
  2021-01-17 14:33       ` Phillip Lord
  0 siblings, 2 replies; 6+ messages in thread
From: Stefan Monnier @ 2021-01-06 15:40 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: emacs-devel

>>     * lisp/emacs-lisp/package.el (package-activate-all): Another tweak
>>
>>     `package-quickstart.el` files presume `package-activated-list`
>>     is a bound variable, so make sure this is the case even when `package.el` is
>>     not yet loaded.
>
> I'm still getting
>
> In command-line:
> startup.el:1233:28: Warning: reference to free variable `package-user-dir'
> startup.el:1233:45: Warning: reference to free variable
>     `package-directory-list'
>
> when building Emacs...

Even after

    (cd lisp; make autoloads)

??


        Stefan




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

* Re: master cf672c6: * lisp/emacs-lisp/package.el (package-activate-all): Another tweak
  2021-01-06 15:40     ` Stefan Monnier
@ 2021-01-07 11:36       ` Lars Ingebrigtsen
  2021-01-08  3:29         ` Stefan Monnier
  2021-01-17 14:33       ` Phillip Lord
  1 sibling, 1 reply; 6+ messages in thread
From: Lars Ingebrigtsen @ 2021-01-07 11:36 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: emacs-devel

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

>> when building Emacs...
>
> Even after
>
>     (cd lisp; make autoloads)

Hm...  yup.  But ... perhaps the bootstrap wipes out the autoloads
again?  I'm still a bit vague on how and when they're actually uploaded,
other than that Glenn does something magical every week.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no



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

* Re: master cf672c6: * lisp/emacs-lisp/package.el (package-activate-all): Another tweak
  2021-01-07 11:36       ` Lars Ingebrigtsen
@ 2021-01-08  3:29         ` Stefan Monnier
  0 siblings, 0 replies; 6+ messages in thread
From: Stefan Monnier @ 2021-01-08  3:29 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: emacs-devel

>>> when building Emacs...
>>
>> Even after
>>
>>     (cd lisp; make autoloads)
>
> Hm...  yup.  But ... perhaps the bootstrap wipes out the autoloads
> again?

Indeed it does.  I believe this warning is harmless there: it does mean
that the bootstrap build is not fully functional (it will fail to start
in non-batch mode), but that bootstrap build will successfully build the
lisp/loaddefs.el (and all the .elc files etc...) and the subsequent
"real build" will then be fully functional.

> I'm still a bit vague on how and when they're actually uploaded,
> other than that Glenn does something magical every week.

Right, the warning should disappear next time Glenn updates the
`ldefs-boot.el` file.


        Stefan




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

* Re: master cf672c6: * lisp/emacs-lisp/package.el (package-activate-all): Another tweak
  2021-01-06 15:40     ` Stefan Monnier
  2021-01-07 11:36       ` Lars Ingebrigtsen
@ 2021-01-17 14:33       ` Phillip Lord
  2021-01-17 15:23         ` Stefan Monnier
  1 sibling, 1 reply; 6+ messages in thread
From: Phillip Lord @ 2021-01-17 14:33 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: Lars Ingebrigtsen, emacs-devel



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

>>>     * lisp/emacs-lisp/package.el (package-activate-all): Another tweak
>>>
>>>     `package-quickstart.el` files presume `package-activated-list`
>>>     is a bound variable, so make sure this is the case even when `package.el` is
>>>     not yet loaded.
>>
>> I'm still getting
>>
>> In command-line:
>> startup.el:1233:28: Warning: reference to free variable `package-user-dir'
>> startup.el:1233:45: Warning: reference to free variable
>>     `package-directory-list'
>>
>> when building Emacs...
>
> Even after
>
>     (cd lisp; make autoloads)
>
> ??


I normally reset package-user-dir in my early-init.el. This seems to
have broken recently on master -- by the time init.el is run any changes
in early-init.el have been reverted.

Could these changes be related?

Phil



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

* Re: master cf672c6: * lisp/emacs-lisp/package.el (package-activate-all): Another tweak
  2021-01-17 14:33       ` Phillip Lord
@ 2021-01-17 15:23         ` Stefan Monnier
  0 siblings, 0 replies; 6+ messages in thread
From: Stefan Monnier @ 2021-01-17 15:23 UTC (permalink / raw)
  To: Phillip Lord; +Cc: Lars Ingebrigtsen, emacs-devel

> I normally reset package-user-dir in my early-init.el. This seems to
> have broken recently on master -- by the time init.el is run any changes
> in early-init.el have been reverted.

Yes, see bug#45857,


        Stefan




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

end of thread, other threads:[~2021-01-17 15:23 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <20210106022608.17208.61029@vcs0.savannah.gnu.org>
     [not found] ` <20210106022610.4BD5720CB7@vcs0.savannah.gnu.org>
2021-01-06  8:57   ` master cf672c6: * lisp/emacs-lisp/package.el (package-activate-all): Another tweak Lars Ingebrigtsen
2021-01-06 15:40     ` Stefan Monnier
2021-01-07 11:36       ` Lars Ingebrigtsen
2021-01-08  3:29         ` Stefan Monnier
2021-01-17 14:33       ` Phillip Lord
2021-01-17 15:23         ` 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).