unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#24467: 25.1; package
@ 2016-09-19  5:29 SK Kim
  2016-09-19 22:01 ` Noam Postavsky
  0 siblings, 1 reply; 9+ messages in thread
From: SK Kim @ 2016-09-19  5:29 UTC (permalink / raw)
  To: 24467

[-- Attachment #1: Type: text/plain, Size: 311 bytes --]

After i set gnu and mepla for pacakage-archives in my init.el as
below,

(package-initialize)
(setq package-archives
    '(("gnu" . "http://elpa.gnu.org/packages/")
     ("melpa" . "http://melpa.milkbox.net/packages/")))

whenever i start package-list-packages, status of mepla packages is
always new.

thanks.

[-- Attachment #2: Type: text/html, Size: 559 bytes --]

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

* bug#24467: 25.1; package
  2016-09-19  5:29 bug#24467: 25.1; package SK Kim
@ 2016-09-19 22:01 ` Noam Postavsky
  2016-09-24 22:28   ` Michael Heerdegen
  0 siblings, 1 reply; 9+ messages in thread
From: Noam Postavsky @ 2016-09-19 22:01 UTC (permalink / raw)
  To: SK Kim; +Cc: 24467, Michael Heerdegen

merge 24467 21703
quit

On Mon, Sep 19, 2016 at 1:29 AM, SK Kim <tttuuu888@gmail.com> wrote:
> After i set gnu and mepla for pacakage-archives in my init.el as
> below,
>
> (package-initialize)
> (setq package-archives
>     '(("gnu" . "http://elpa.gnu.org/packages/")
>      ("melpa" . "http://melpa.milkbox.net/packages/")))
>
> whenever i start package-list-packages, status of mepla packages is
> always new.

Seems to be the same as #21703 "25.0.50; package.el: Thousands of new
packages each day", but I'm not able to reproduce.

I put the above as my init.el in a temporary HOME dir, when I run
package-list-packages melpa packages are all "available" for me.

  0blayout           20160919.823  available  melpa      Layout
grouping with ease
  2048-game          20151026.1233 available  melpa      play 2048 in Emacs
  4clojure           20131014.1507 available  melpa      Open and
evaluate 4clojure.com questions
  @                  20140707.520  available  melpa
multiple-inheritance prototype-based objects DSL
...





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

* bug#24467: 25.1; package
  2016-09-19 22:01 ` Noam Postavsky
@ 2016-09-24 22:28   ` Michael Heerdegen
  2016-09-25  2:45     ` npostavs
  0 siblings, 1 reply; 9+ messages in thread
From: Michael Heerdegen @ 2016-09-24 22:28 UTC (permalink / raw)
  To: Noam Postavsky; +Cc: 24467, SK Kim

Noam Postavsky <npostavs@users.sourceforge.net> writes:

> merge 24467 21703
> quit
>
> On Mon, Sep 19, 2016 at 1:29 AM, SK Kim <tttuuu888@gmail.com> wrote:
> > After i set gnu and mepla for pacakage-archives in my init.el as
> > below,
> >
> > (package-initialize)
> > (setq package-archives
> >     '(("gnu" . "http://elpa.gnu.org/packages/")
> >      ("melpa" . "http://melpa.milkbox.net/packages/")))
> >
> > whenever i start package-list-packages, status of mepla packages is
> > always new.
>
> Seems to be the same as #21703 "25.0.50; package.el: Thousands of new
> packages each day", but I'm not able to reproduce.
>
> I put the above as my init.el in a temporary HOME dir, when I run
> package-list-packages melpa packages are all "available" for me.

Maybe it is different on the first run?

Hmm, anyway, I found out that with

 (package-initialize)
 (setq package-archives
     '(("gnu" . "http://elpa.gnu.org/packages/")
      ("melpa" . "http://melpa.milkbox.net/packages/")))

in my init file I see the issue.  But with

  (setq package-archives
        '(("gnu" . "http://elpa.gnu.org/packages/")
          ("melpa" . "http://melpa.milkbox.net/packages/")))
  (package-initialize)

it seems to be fixed (i.e. the same expressions just transposed).  With
other words, `package-archives' seemingly needs to be set before calling
`package-intialize' (@SK Kim, do you see the same?).

With the first version, after Emacs startup, `package-archive-contents'
only includes the "gnu" packages.  With the second version, it seems to
include "everything" (the list is too long to be printed in reasonable
time).  That variable is used to decide what packages are new.  That's
were the problem seems to come from.

I think the behavior is at least "surprising", a pitfall.


Regards,

Michael.





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

* bug#24467: 25.1; package
  2016-09-24 22:28   ` Michael Heerdegen
@ 2016-09-25  2:45     ` npostavs
  2016-09-25 13:38       ` Michael Heerdegen
  0 siblings, 1 reply; 9+ messages in thread
From: npostavs @ 2016-09-25  2:45 UTC (permalink / raw)
  To: Michael Heerdegen; +Cc: 24467, SK Kim

tags 24467 confirmed
severity 24467 minor
quit

Michael Heerdegen <michael_heerdegen@web.de> writes:

> Noam Postavsky <npostavs@users.sourceforge.net> writes:
>>
>> I put the above as my init.el in a temporary HOME dir, when I run
>> package-list-packages melpa packages are all "available" for me.
>
> Maybe it is different on the first run?

Yes, if I close and reopen Emacs, then I see the "new" packages too.

>
> Hmm, anyway, I found out that with
>
>  (package-initialize)
>  (setq package-archives
>      '(("gnu" . "http://elpa.gnu.org/packages/")
>       ("melpa" . "http://melpa.milkbox.net/packages/")))
>
> in my init file I see the issue.  But with
>
>   (setq package-archives
>         '(("gnu" . "http://elpa.gnu.org/packages/")
>           ("melpa" . "http://melpa.milkbox.net/packages/")))
>   (package-initialize)
>
> it seems to be fixed (i.e. the same expressions just transposed).  With
> other words, `package-archives' seemingly needs to be set before calling
> `package-intialize' (@SK Kim, do you see the same?).

Ah, so the packages are "new" in the sense that they come from archives
added after package-initialize was called.

>
> With the first version, after Emacs startup, `package-archive-contents'
> only includes the "gnu" packages.  With the second version, it seems to
> include "everything" (the list is too long to be printed in reasonable
> time).  That variable is used to decide what packages are new.  That's
> were the problem seems to come from.
>
> I think the behavior is at least "surprising", a pitfall.

Would be nice to avoid it, indeed.






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

* bug#24467: 25.1; package
  2016-09-25  2:45     ` npostavs
@ 2016-09-25 13:38       ` Michael Heerdegen
  2018-11-03 15:07         ` Stefan Monnier
  0 siblings, 1 reply; 9+ messages in thread
From: Michael Heerdegen @ 2016-09-25 13:38 UTC (permalink / raw)
  To: npostavs; +Cc: 24467, SK Kim

npostavs@users.sourceforge.net writes:

> Ah, so the packages are "new" in the sense that they come from
> archives added after package-initialize was called.

I'm not sure whether this is by design and if we would break something
if we changed it.  I think the fix would be quite trivial: simply
speaking, we would change `package-read-all-archive-contents' to iterate
over "$package-user-dir/archives/*" instead of `package-archives'.


Michael.





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

* bug#24467: 25.1; package
  2016-09-25 13:38       ` Michael Heerdegen
@ 2018-11-03 15:07         ` Stefan Monnier
  2019-01-22 22:56           ` Stefan Monnier
  0 siblings, 1 reply; 9+ messages in thread
From: Stefan Monnier @ 2018-11-03 15:07 UTC (permalink / raw)
  To: Michael Heerdegen; +Cc: 24467, SK Kim, npostavs

>> Ah, so the packages are "new" in the sense that they come from
>> archives added after package-initialize was called.
>
> I'm not sure whether this is by design and if we would break something
> if we changed it.  I think the fix would be quite trivial: simply
> speaking, we would change `package-read-all-archive-contents' to iterate
> over "$package-user-dir/archives/*" instead of `package-archives'.

It's clearly a bug, and it's made worse in Emacs-27 where
package-initialize is called automatically before reading ~/.emacs.

IOW, I think "this" needs to be fixed before Emacs-27.1.

The fix you propose might work (tho with unintended side effects), but
I don't see any reason why we need to package-read-all-archive-contents
so early on anyway.  More specifically, package-activate-all should be
changed so it doesn't end up running package-read-all-archive-contents:
it's already the case when a package-quickstart-file is found, but if
there's no such file we call package-initialize which ends up calling
package-read-all-archive-contents even though we're only interested in
the package-load-all-descriptors part, really.


        Stefan





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

* bug#24467: 25.1; package
  2018-11-03 15:07         ` Stefan Monnier
@ 2019-01-22 22:56           ` Stefan Monnier
  2021-08-16 12:26             ` bug#21703: adding package archives after calling package-initialize lists all their packages as "new" Lars Ingebrigtsen
  0 siblings, 1 reply; 9+ messages in thread
From: Stefan Monnier @ 2019-01-22 22:56 UTC (permalink / raw)
  To: Michael Heerdegen; +Cc: 24467, SK Kim, npostavs

> It's clearly a bug, and it's made worse in Emacs-27 where
> package-initialize is called automatically before reading ~/.emacs.
> IOW, I think "this" needs to be fixed before Emacs-27.1.

I just installed a patch which should reduce the impact of this problem
in Emacs-27.1: the problem still occurs if you setq package-archives
after calling package-initialize, but at least you can now setq
package-archives in your ~/.emacs rather than having to do it in your
early-init.el.


        Stefan





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

* bug#21703: adding package archives after calling package-initialize lists all their packages as "new"
  2019-01-22 22:56           ` Stefan Monnier
@ 2021-08-16 12:26             ` Lars Ingebrigtsen
  2021-08-17 12:39               ` bug#24467: " Michael Heerdegen
  0 siblings, 1 reply; 9+ messages in thread
From: Lars Ingebrigtsen @ 2021-08-16 12:26 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: Michael Heerdegen, 24467, SK Kim, 21703, npostavs

Stefan Monnier <monnier@IRO.UMontreal.CA> writes:

>> It's clearly a bug, and it's made worse in Emacs-27 where
>> package-initialize is called automatically before reading ~/.emacs.
>> IOW, I think "this" needs to be fixed before Emacs-27.1.
>
> I just installed a patch which should reduce the impact of this problem
> in Emacs-27.1: the problem still occurs if you setq package-archives
> after calling package-initialize, but at least you can now setq
> package-archives in your ~/.emacs rather than having to do it in your
> early-init.el.

Is there more to be done here, or is this a sufficient solution to the
issue?

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





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

* bug#24467: bug#21703: adding package archives after calling package-initialize lists all their packages as "new"
  2021-08-16 12:26             ` bug#21703: adding package archives after calling package-initialize lists all their packages as "new" Lars Ingebrigtsen
@ 2021-08-17 12:39               ` Michael Heerdegen
  0 siblings, 0 replies; 9+ messages in thread
From: Michael Heerdegen @ 2021-08-17 12:39 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: 24467, SK Kim, Stefan Monnier, 21703, npostavs

Lars Ingebrigtsen <larsi@gnus.org> writes:

> Stefan Monnier <monnier@IRO.UMontreal.CA> writes:
>
> >> It's clearly a bug, and it's made worse in Emacs-27 where
> >> package-initialize is called automatically before reading ~/.emacs.
> >> IOW, I think "this" needs to be fixed before Emacs-27.1.
> >
> > I just installed a patch which should reduce the impact of this problem
> > in Emacs-27.1: the problem still occurs if you setq package-archives
> > after calling package-initialize, but at least you can now setq
> > package-archives in your ~/.emacs rather than having to do it in your
> > early-init.el.
>
> Is there more to be done here, or is this a sufficient solution to the
> issue?

If nothing more was changed, then the originally reported problem is not
fixed.  That behavior might still annoy or confuse people then.

Michael.





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

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

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-09-19  5:29 bug#24467: 25.1; package SK Kim
2016-09-19 22:01 ` Noam Postavsky
2016-09-24 22:28   ` Michael Heerdegen
2016-09-25  2:45     ` npostavs
2016-09-25 13:38       ` Michael Heerdegen
2018-11-03 15:07         ` Stefan Monnier
2019-01-22 22:56           ` Stefan Monnier
2021-08-16 12:26             ` bug#21703: adding package archives after calling package-initialize lists all their packages as "new" Lars Ingebrigtsen
2021-08-17 12:39               ` bug#24467: " Michael Heerdegen

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