GNU bug report logs - #33565
[PATCH] gnu: Add python-xmltodict.

Previous Next

Full log

Message #11 received at 33565 <at> debbugs.gnu.org (full text, mbox):

From: Oleg Pykhalov <go.wigust <at> gmail.com>
To: ludo <at> gnu.org (Ludovic Courtès)
Cc: 33565 <at> debbugs.gnu.org
Subject: Re: [bug#33565] [PATCH] gnu: Add python-xmltodict.
Date: Tue, 04 Dec 2018 08:13:44 +0300

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

Hi Ludovic.

Thank you for review.

ludo <at> gnu.org (Ludovic Courtès) writes:

> Oleg Pykhalov <go.wigust <at> gmail.com> skribis:
>
>> * gnu/packages/xml.scm (python-xmltodict, python2-xmltodict): New variables.
>
> [...]
>
>> +    (home-page "https://github.com/martinblech/xmltodict")
>> +    (synopsis "Work with XML like you are working with JSON")
>> +    (description "This package provides a Python library to work with XML like
>> +you are working with JSON")
>
> Could you expound a bit on what this means (I’m curious!) and add a
> period?

It's a quote from the home page. :-) But in reality this Python library
converts input string which contains XML to an OrderedDict object which
looks like a JSON.  Queries for data inside OrderedDict feels (I guess)
like you're working with JSON.  Maybe we should change a description to
“This package provides a Python library to convert XML to OrderedDict.”,
WDYT?

>> +(define-public python2-xmltodict
>> +  (package-with-python2 python-xmltodict))
>
> Unless you really need it, you should omit the Python 2.x variant.

Sure, I'll omit.

Oleg.

[signature.asc (application/pgp-signature, inline)]

Previous Next

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