emacs-bug-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#42192: closed ([PATCH 0/3] Improve launchpad updater)


From: GNU bug Tracking System
Subject: bug#42192: closed ([PATCH 0/3] Improve launchpad updater)
Date: Tue, 01 Sep 2020 17:21:01 +0000

Your message dated Tue, 01 Sep 2020 22:50:46 +0530
with message-id <87zh691ki9.fsf@systemreboot.net>
and subject line Re: [bug#42192] [PATCH 0/3] Improve launchpad updater
has caused the debbugs.gnu.org bug report #42192,
regarding [PATCH 0/3] Improve launchpad updater
to be marked as done.

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


-- 
42192: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=42192
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems
--- Begin Message --- Subject: [PATCH 0/3] Improve launchpad updater Date: Sat, 4 Jul 2020 23:55:48 +0530
This patchset fixes a bug in the launchpad importer, and improves it to
recognize more URLs and extensions. `guix refresh --list-updaters` reports an
improvement in coverage from 0.1% to 0.2%.

On a side note, all our updaters try to detect the URL format through a series
of guesses. But, all our packages already encode information about how to
construct the source URL from the version. If we could somehow tap into that
information, our updaters would be much simpler. Unfortunately, all the uri
fields of packages are strings. We would need them to be functions that take
the version as an argument. This is a major conversion. Is it feasible? Any
better ideas?

Arun Isaac (3):
  import: launchpad: Handle list of source URLs correctly.
  import: launchpad: Recognize the .orig.tar.gz extension.
  import: launchpad: Recognize more URLs.

 guix/import/launchpad.scm | 38 +++++++++++++++++++++++++++++---------
 1 file changed, 29 insertions(+), 9 deletions(-)

-- 
2.26.2




--- End Message ---
--- Begin Message --- Subject: Re: [bug#42192] [PATCH 0/3] Improve launchpad updater Date: Tue, 01 Sep 2020 22:50:46 +0530
>> This patchset fixes a bug in the launchpad importer, and improves it to
>> recognize more URLs and extensions. `guix refresh --list-updaters` reports an
>> improvement in coverage from 0.1% to 0.2%.
>
> All three patches LGTM, thank you!

Pushed to master, thanks for the review!

> I sympathize with the idea, but I think it’s a bit too ambitious.  All
> in all, what we currently have seems like a reasonable tradeoff.

Fair enough. :-)


--- End Message ---

reply via email to

[Prev in Thread] Current Thread [Next in Thread]