bug#63757: 29.0.91 order of package paths changed: random old versions of packages in load-path

["Евгений Бойков via Bug reports for GNU Emacs, the Swiss army knife of text editors" <bug-gnu-emacs@gnu.org>]

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

By the way, is "Prefer builtin packages" related to (package-disabled-p p1
v1)? I think the comment and the code mean different things.


On Sat, Jun 3, 2023 at 10:53 PM Евгений Бойков <artscan@list.ru> wrote:

> The idea is to split the list to 3 parts and to sort each of them by
> version-list-< separately (but I am not sure which combination of
> preference is correct) and to flatten at the end
>
> ((VC packages) (builtin packages) (the rest))
>
> I use (package-vc-p p1) and (assq p1 package--builtins) predicates for
> splitting (not sure for builtin). The code below needs refactoring.
>
> (require 'cl)
> (defun package--get-activatable-pkg (pkg-name)
>   ;; Is "activatable" a word?
>   (let ((pkg-descs
>          (flatten-list
>           (mapcar (lambda (x)
>                     (sort x
>                           (lambda (p1 p2)
>                             (let ((v1 (package-desc-version p1))
>                                   (v2 (package-desc-version p2)))
>                               (version-list-< v2 v1)))))
>                   (let ((y (reduce
>                             (lambda (p1 p2)
>                               (if (package-vc-p p1)
>                                   (push p1 (first p2))
>                                 (push p1 (second p2)))
>                               p2)
>                             (cdr (assq pkg-name package-alist))
>                             :initial-value (list nil nil)
>                             :from-end t)))
>                     (append (list (car y))
>                             (reduce
>                              (lambda (p1 p2)
>                                (if (assq p1 package--builtins)
>                                    (push p1 (first p2))
>                                  (push p1 (second p2))
>                                  p2))
>                              (cadr y)
>                              :initial-value (list nil nil)
>                              :from-end t)))))))
>     ;; Check if PACKAGE is available in `package-alist'.
>     (while
>         (when pkg-descs
>           (let ((available-version (package-desc-version (car pkg-descs))))
>             (package-disabled-p pkg-name available-version)))
>       (setq pkg-descs (cdr pkg-descs)))
>     (car pkg-descs)))
>
>
>
> On Sat, Jun 3, 2023 at 9:50 PM Евгений Бойков <artscan@list.ru> wrote:
>
>> The fix above will not work.
>>
>> I can do it for "Prefer VC packages" only:
>>
>> (require 'cl)
>> (defun package--get-activatable-pkg (pkg-name)
>>   ;; Is "activatable" a word?
>>   (let ((pkg-descs
>>          (flatten-list
>>           (mapcar (lambda (x)
>>                     (sort x
>>                           (lambda (p1 p2)
>>                             (let ((v1 (package-desc-version p1))
>>                                   (v2 (package-desc-version p2)))
>>                               (version-list-< v2 v1)))))
>>                   (reduce
>>                    (lambda (p1 p2)
>>                      (if (package-vc-p p1)
>>                          (push p1 (first p2))
>>                        (push p1 (second p2)))
>>                      p2)
>>                    (cdr (assq pkg-name package-alist))
>>                    :initial-value (list nil nil)
>>                    :from-end t)))))
>>     ;; Check if PACKAGE is available in `package-alist'.
>>     (while
>>         (when pkg-descs
>>           (let ((available-version (package-desc-version (car
>> pkg-descs))))
>>             (package-disabled-p pkg-name available-version)))
>>       (setq pkg-descs (cdr pkg-descs)))
>>     (car pkg-descs)))
>>
>>
>> Still can't for "Prefer builtin packages".
>>
>>
>> On Sat, Jun 3, 2023 at 8:37 PM Eli Zaretskii <eliz@gnu.org> wrote:
>>
>>> > From: Philip Kaludercic <philipk@posteo.net>
>>> > Cc: Евгений Бойков <artscan@list.ru>,
>>> >   63757@debbugs.gnu.org,  Stefan
>>> >  Monnier <monnier@iro.umontreal.ca>
>>> > Date: Sat, 03 Jun 2023 10:19:24 +0000
>>> >
>>> > > Won't this work:
>>> > >
>>> > > diff --git a/lisp/emacs-lisp/package.el b/lisp/emacs-lisp/package.el
>>> > > index 01826da273d..7eb185e7042 100644
>>> > > --- a/lisp/emacs-lisp/package.el
>>> > > +++ b/lisp/emacs-lisp/package.el
>>> > > @@ -927,7 +927,9 @@ package--get-activatable-pkg
>>> > >                                (package-vc-p p2)
>>> > >                                ;; Prefer builtin packages.
>>> > >                                (package-disabled-p p1 v1)
>>> > > -                              (not (package-disabled-p p2 v2))))))))
>>> > > +                              (not (package-disabled-p p2 v2))
>>> > > +                              ;; Prever newer packages
>>> > > +                              (version-list-< v2 v1)))))))
>>> > >      ;; Check if PACKAGE is available in `package-alist'.
>>> > >      (while
>>> > >          (when pkg-descs
>>> >
>>> > Ping?
>>>
>>> If there's no response, and you think this is the right fix, we can
>>> simply install it, I guess.
>>>
>>
>>
>> --
>> __________________________
>>
>> С уважением,
>> Бойков Евгений Алексеевич
>> сот. 8-924-202-25-65
>> e-mail: artscan@list.ru
>>
>
>
> --
> __________________________
>
> С уважением,
> Бойков Евгений Алексеевич
> сот. 8-924-202-25-65
> e-mail: artscan@list.ru
>


-- 
__________________________

С уважением,
Бойков Евгений Алексеевич
сот. 8-924-202-25-65
e-mail: artscan@list.ru

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