GNU bug report logs - #73781
[PATCH] Upgrade out-of-date dependencies with package-vc

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Philip Kaludercic <philipk <at> posteo.net>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#73781: closed ([PATCH] Upgrade out-of-date dependencies with
 package-vc)
Date: Sun, 23 Feb 2025 11:23:01 +0000

[Message part 1 (text/plain, inline)]

Your message dated Sun, 23 Feb 2025 11:22:02 +0000
with message-id <87cyf8zz1h.fsf <at> posteo.net>
and subject line Re: bug#73781: [PATCH] Upgrade out-of-date dependencies with package-vc
has caused the debbugs.gnu.org bug report #73781,
regarding [PATCH] Upgrade out-of-date dependencies with package-vc
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
73781: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=73781
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Joseph Turner <joseph <at> breatheoutbreathe.in>
To: bug-gnu-emacs <at> gnu.org
Subject: [PATCH] Upgrade out-of-date dependencies with package-vc
Date: Sun, 13 Oct 2024 01:23:38 +0200

[Message part 3 (text/plain, inline)]

Tags: patch

I noticed this bug when working on hyperdrive.el, which I had installed
with package-vc-install-from-checkout.  I updated the version of its
transient.el dependency in the Package-Requires header and attempted to
reinstall, but it did not attempt to update the dependency.

I think the attached patch may not be the whole solution.  I applied it,
attempted package-vc-install-from-checkout again, and got this error:

package--with-response-buffer-1: https://melpa.org/packages/transient-20241004.1739.tar: Not found

Do we need to call package-refresh-contents at some point inside of
package-vc-install-dependencies?

Thank you!

Joseph

[0001-Upgrade-out-of-date-dependencies.patch (text/patch, attachment)]

[Message part 5 (message/rfc822, inline)]

From: Philip Kaludercic <philipk <at> posteo.net>
To: Joseph Turner <joseph <at> breatheoutbreathe.in>
Cc: 73781-done <at> debbugs.gnu.org
Subject: Re: bug#73781: [PATCH] Upgrade out-of-date dependencies with
 package-vc
Date: Sun, 23 Feb 2025 11:22:02 +0000

Joseph Turner <joseph <at> breatheoutbreathe.in> writes:

> Tags: patch
>
> I noticed this bug when working on hyperdrive.el, which I had installed
> with package-vc-install-from-checkout.  I updated the version of its
> transient.el dependency in the Package-Requires header and attempted to
> reinstall, but it did not attempt to update the dependency.
>
> I think the attached patch may not be the whole solution.  I applied it,
> attempted package-vc-install-from-checkout again, and got this error:
>
> package--with-response-buffer-1: https://melpa.org/packages/transient-20241004.1739.tar: Not found
>
> Do we need to call package-refresh-contents at some point inside of
> package-vc-install-dependencies?
>
> Thank you!
>
> Joseph
>
>>From fd05beab4903e3ba9f2a6866a00e924f8bcd5637 Mon Sep 17 00:00:00 2001
> From: Joseph Turner <joseph <at> breatheoutbreathe.in>
> Date: Sun, 13 Oct 2024 01:10:02 +0200
> Subject: [PATCH] Upgrade out-of-date dependencies
>
> * lisp/emacs-lisp/package-vc.el (package-vc-install-dependencies): Pass
> the specified package version when checking if a package is installed.
> ---
>  lisp/emacs-lisp/package-vc.el | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/lisp/emacs-lisp/package-vc.el b/lisp/emacs-lisp/package-vc.el
> index e168096e153..58916637008 100644
> --- a/lisp/emacs-lisp/package-vc.el
> +++ b/lisp/emacs-lisp/package-vc.el
> @@ -465,7 +465,7 @@ package-vc-install-dependencies
>                    "Attempt to find all dependencies for PKG."
>                    (cond
>                     ((assq (car pkg) to-install)) ;inhibit cycles
> -                   ((package-installed-p (car pkg)))
> +                   ((package-installed-p (car pkg) (cadr pkg)))
>                     ((let* ((pac package-archive-contents)
>                             (desc (cadr (assoc (car pkg) pac))))
>                        (if desc

Stefan Kangas <stefankangas <at> gmail.com> writes:

> Stefan Kangas <stefankangas <at> gmail.com> writes:
>
>> Joseph Turner <joseph <at> breatheoutbreathe.in> writes:
>>
>>> Tags: patch
>>>
>>> I noticed this bug when working on hyperdrive.el, which I had installed
>>> with package-vc-install-from-checkout.  I updated the version of its
>>> transient.el dependency in the Package-Requires header and attempted to
>>> reinstall, but it did not attempt to update the dependency.
>>>
>>> I think the attached patch may not be the whole solution.  I applied it,
>>> attempted package-vc-install-from-checkout again, and got this error:
>>>
>>> package--with-response-buffer-1: https://melpa.org/packages/transient-20241004.1739.tar: Not found
>>>
>>> Do we need to call package-refresh-contents at some point inside of
>>> package-vc-install-dependencies?
>>>
>>> Thank you!
>>>
>>> Joseph
>>
>> Philip, any comments?
>
> Ping.  Philip, WDYT?

My apologies for missing the message!  Yes, the change makes sense and I
can push it to master.  If you think it is worthwhile, please
cherry-pick it onto the emacs-30 branch.

Thanks!

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.