GNU bug report logs -
#42192
[PATCH 0/3] Improve launchpad updater
Previous Next
Reported by: Arun Isaac <arunisaac <at> systemreboot.net>
Date: Sat, 4 Jul 2020 18:27:01 UTC
Severity: normal
Tags: patch
Done: Arun Isaac <arunisaac <at> systemreboot.net>
Bug is archived. No further changes may be made.
Full log
View this message in rfc822 format
[Message part 1 (text/plain, inline)]
Your bug report
#42192: [PATCH 0/3] Improve launchpad updater
which was filed against the guix-patches package, has been closed.
The explanation is attached below, along with your original report.
If you require more details, please reply to 42192 <at> debbugs.gnu.org.
--
42192: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=42192
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems
[Message part 2 (message/rfc822, inline)]
>> 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. :-)
[Message part 3 (message/rfc822, inline)]
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
This bug report was last modified 4 years and 259 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.