unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
From: swedebugia <swedebugia@riseup.net>
To: "Ludovic Courtès" <ludo@gnu.org>
Cc: "33436@debbugs.gnu.org" <33436@debbugs.gnu.org>
Subject: [bug#33436] [PATCH] gnu: Add python-warcio.
Date: Sun, 24 Feb 2019 03:01:29 +0100	[thread overview]
Message-ID: <ae61574a-f769-a78f-71ac-48a57477ba0e@riseup.net> (raw)
In-Reply-To: <87zhs78un9.fsf@gnu.org>


[-- Attachment #1.1.1: Type: text/plain, Size: 1532 bytes --]

On 2019-01-11 09:29, Ludovic Courtès wrote:
> Ping!
> 
> ludo@gnu.org (Ludovic Courtès) skribis:
> 
>> Hello!
>>
>> swedebugia <swedebugia@riseup.net> skribis:
>>
>>>> >From 537b2b111a464956bdec640ea5f84c4598ea66f9 Mon Sep 17 00:00:00 2001
>>> From: swedebugia <swedebugia@riseup.net>
>>> Date: Mon, 19 Nov 2018 21:37:46 +0100
>>> Subject: [PATCH] gnu: Add python-warcio.
>>>
>>> * gnu/packages/python.scm: New variable.
>>                            ^
>> Nitpick: You forgot the variable name here.  :-)
>>
>>> +   (arguments
>>> +    ;; FIXME: Some tests require network access. 150 out of 1354 fail
>>> +    '(#:tests? #f))
>>
>> Could you investigate a bit further?  What do the test logs show?
>>
>> It would be good to see if these tests can be easily fixed, if they
>> should definitely be skipped (for instance because they rely on some
>> external service), or if it’s something else.
>>
>>> +   (home-page "https://github.com/webrecorder/warcio")
>>> +   (synopsis "Streaming web archival archive (WARC) library")
>>> +   (description
>>> +    "warcio is designed for fast, low-level access to web archival
>>                  ^^
>> What about: “warcio is a Python library to read and write the WARC format
>> commonly used in Web archives.  It is designed for…”?
>>
>> Thank you,
>> Ludo’.
Here comes the log from the build.

I see a lot of "OSError: [Errno 9] Bad file descriptor" and a few
"FileNotFoundError"

Should I report upstream?

-- 
Cheers Swedebugia

[-- Attachment #1.1.2: gwkz0zj2bqbw83nhh1ba1iq7lndryg-python-warcio-1.6.3.drv.bz2 --]
[-- Type: application/x-bzip, Size: 20899 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

  reply	other threads:[~2019-02-24  7:20 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-11-19 20:41 [bug#33436] [PATCH] gnu: Add python-warcio swedebugia
2018-11-21 10:21 ` Ludovic Courtès
2019-01-11  8:29   ` Ludovic Courtès
2019-02-24  2:01     ` swedebugia [this message]
2019-03-04 13:43       ` Ludovic Courtès
2021-07-13 15:23   ` bug#33436: " Maxim Cournoyer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ae61574a-f769-a78f-71ac-48a57477ba0e@riseup.net \
    --to=swedebugia@riseup.net \
    --cc=33436@debbugs.gnu.org \
    --cc=ludo@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).