[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#62720: 29.0.60; Not easy at all to upgrade :core packages like Eglot
From: |
Dmitry Gutov |
Subject: |
bug#62720: 29.0.60; Not easy at all to upgrade :core packages like Eglot |
Date: |
Sun, 23 Apr 2023 02:46:05 +0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.10.0 |
On 22/04/2023 15:00, Eli Zaretskii wrote:
I think I explained in the previous email why reusing
package-install-upgrade-built-in doesn't seem like a good idea.
And I thought I've explained why I didn't see a need for another
option.
I also don't see the point of using an option here.
We must not change past behavior unconditionally and by default, not
this close to the release.
We're still allowed bugfixing, I believe.
and only then
update built-in packages.
I asked what plausible scenario you think might be broken by having
package-update upgrade builtin package by default.
That's obvious: this is how package-update behaved until now.
That's not an answer to the question.
It is for me (and I'm quite surprised that it is not for you).
Not sure why you're surprised, you know my approach toward backward
compatibility. Never a fan of enshrining problems in amber.
But in this case it's also a function that's never been in a released
Emacs, so the formal conditions are lacking as well.
And it's okay to use the time since the code was added as a rough proxy
for stability, but when it's pretty clear (just from the comments in
this very thread) that most people never noticed or forgot that it's
there. So it's obviously not very well tested.
Just from reading it code and testing, I see another bug: it removes the
updated package from package-selected-packages because it doesn't pass
NOSAVE to package-delete. Meaning, 'M-x package-autoremove' at any time
later will delete it.
Alternatively, we could add an optional argument to package-install
which would mean "install the latest version anyway".
There is already such an option, added as part of fixing this bug.
Ok, since you insist. See attached.
Just to be clear, we are talking about the 4 lines at the end, right?
Yes, and also the (somewhat mysterious) additions of tests for
pkg-desc.
pkg-desc is nil for builtin packages in this case (they are not in
package-alist, so (assq package package-alist) returns nil).
This should either be commented, or a variable with a telltale name
added to reflect this subtlety.
Not a problem.
package-update-fix.diff
Description: Text Data
- bug#62720: 29.0.60; Not easy at all to upgrade :core packages like Eglot, (continued)
- bug#62720: 29.0.60; Not easy at all to upgrade :core packages like Eglot, Dmitry Gutov, 2023/04/21
- bug#62720: 29.0.60; Not easy at all to upgrade :core packages like Eglot, Eli Zaretskii, 2023/04/22
- bug#62720: 29.0.60; Not easy at all to upgrade :core packages like Eglot, Dmitry Gutov, 2023/04/22
- bug#62720: 29.0.60; Not easy at all to upgrade :core packages like Eglot, Eli Zaretskii, 2023/04/22
- bug#62720: 29.0.60; Not easy at all to upgrade :core packages like Eglot, Dmitry Gutov, 2023/04/22
- bug#62720: 29.0.60; Not easy at all to upgrade :core packages like Eglot, Dmitry Gutov, 2023/04/22
- bug#62720: 29.0.60; Not easy at all to upgrade :core packages like Eglot, Eli Zaretskii, 2023/04/22
- bug#62720: 29.0.60; Not easy at all to upgrade :core packages like Eglot, Eli Zaretskii, 2023/04/22
- bug#62720: 29.0.60; Not easy at all to upgrade :core packages like Eglot, Dmitry Gutov, 2023/04/22
- bug#62720: 29.0.60; Not easy at all to upgrade :core packages like Eglot, Eli Zaretskii, 2023/04/22
- bug#62720: 29.0.60; Not easy at all to upgrade :core packages like Eglot,
Dmitry Gutov <=
- bug#62720: 29.0.60; Not easy at all to upgrade :core packages like Eglot, Eli Zaretskii, 2023/04/23
- bug#62720: 29.0.60; Not easy at all to upgrade :core packages like Eglot, Dmitry Gutov, 2023/04/23
- bug#62720: 29.0.60; Not easy at all to upgrade :core packages like Eglot, Eli Zaretskii, 2023/04/23
- bug#62720: 29.0.60; Not easy at all to upgrade :core packages like Eglot, Dmitry Gutov, 2023/04/23
- bug#62720: 29.0.60; Not easy at all to upgrade :core packages like Eglot, Eli Zaretskii, 2023/04/23
- bug#62720: 29.0.60; Not easy at all to upgrade :core packages like Eglot, Dmitry Gutov, 2023/04/23
- bug#62720: 29.0.60; Not easy at all to upgrade :core packages like Eglot, Eli Zaretskii, 2023/04/24
- bug#62720: 29.0.60; Not easy at all to upgrade :core packages like Eglot, Dmitry Gutov, 2023/04/24
- bug#62720: 29.0.60; Not easy at all to upgrade :core packages like Eglot, Eli Zaretskii, 2023/04/25
- bug#62720: 29.0.60; Not easy at all to upgrade :core packages like Eglot, Dmitry Gutov, 2023/04/25