unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#33565] [PATCH] gnu: Add python-xmltodict.
@ 2018-12-01 20:18 Oleg Pykhalov
  2018-12-03 13:32 ` Ludovic Courtès
  0 siblings, 1 reply; 5+ messages in thread
From: Oleg Pykhalov @ 2018-12-01 20:18 UTC (permalink / raw)
  To: 33565

* gnu/packages/xml.scm (python-xmltodict, python2-xmltodict): New variables.
---
 gnu/packages/xml.scm | 25 +++++++++++++++++++++++++
 1 file changed, 25 insertions(+)

diff --git a/gnu/packages/xml.scm b/gnu/packages/xml.scm
index 347ac878a3..1bb6ba48aa 100644
--- a/gnu/packages/xml.scm
+++ b/gnu/packages/xml.scm
@@ -39,6 +39,7 @@
 (define-module (gnu packages xml)
   #:use-module (gnu packages)
   #:use-module (gnu packages autotools)
+  #:use-module (gnu packages check)
   #:use-module (gnu packages compression)
   #:use-module (gnu packages curl)
   #:use-module (gnu packages gnupg)
@@ -2133,3 +2134,27 @@ It converts the procedure call into an XML document, sends it to a remote
 server using HTTP, and gets back the response as XML.  This library provides a
 modular implementation of XML-RPC for C and C++.")
     (license (list license:psfl license:expat))))
+
+(define-public python-xmltodict
+  (package
+    (name "python-xmltodict")
+    (version "0.11.0")
+    (source
+      (origin
+        (method url-fetch)
+        (uri (pypi-uri "xmltodict" version))
+        (sha256
+          (base32
+            "1pxh4yjhvmxi1h6f92skv41g4kbsws3ams57150kzn18m907v3cg"))))
+    (build-system python-build-system)
+    (native-inputs
+     `(("python-coverage" ,python-coverage)
+       ("python-nose" ,python-nose)))
+    (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")
+    (license license:expat)))
+
+(define-public python2-xmltodict
+  (package-with-python2 python-xmltodict))
-- 
2.19.1

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [bug#33565] [PATCH] gnu: Add python-xmltodict.
  2018-12-01 20:18 [bug#33565] [PATCH] gnu: Add python-xmltodict Oleg Pykhalov
@ 2018-12-03 13:32 ` Ludovic Courtès
  2018-12-04  5:13   ` Oleg Pykhalov
  0 siblings, 1 reply; 5+ messages in thread
From: Ludovic Courtès @ 2018-12-03 13:32 UTC (permalink / raw)
  To: Oleg Pykhalov; +Cc: 33565

Oleg Pykhalov <go.wigust@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?

> +(define-public python2-xmltodict
> +  (package-with-python2 python-xmltodict))

Unless you really need it, you should omit the Python 2.x variant.

Otherwise LGTM, thanks!

Ludo’.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [bug#33565] [PATCH] gnu: Add python-xmltodict.
  2018-12-03 13:32 ` Ludovic Courtès
@ 2018-12-04  5:13   ` Oleg Pykhalov
  2018-12-04  8:47     ` Ludovic Courtès
  0 siblings, 1 reply; 5+ messages in thread
From: Oleg Pykhalov @ 2018-12-04  5:13 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: 33565

[-- Attachment #1: Type: text/plain, Size: 1134 bytes --]

Hi Ludovic.

Thank you for review.

ludo@gnu.org (Ludovic Courtès) writes:

> Oleg Pykhalov <go.wigust@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.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [bug#33565] [PATCH] gnu: Add python-xmltodict.
  2018-12-04  5:13   ` Oleg Pykhalov
@ 2018-12-04  8:47     ` Ludovic Courtès
  2018-12-05  0:45       ` Oleg Pykhalov
  0 siblings, 1 reply; 5+ messages in thread
From: Ludovic Courtès @ 2018-12-04  8:47 UTC (permalink / raw)
  To: Oleg Pykhalov; +Cc: 33565

Hi!

Oleg Pykhalov <go.wigust@gmail.com> skribis:

> ludo@gnu.org (Ludovic Courtès) writes:
>
>> Oleg Pykhalov <go.wigust@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.”,

@code{OrderedDict} even.  :-)

LGTM with this change, thank you!

Ludo’.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [bug#33565] [PATCH] gnu: Add python-xmltodict.
  2018-12-04  8:47     ` Ludovic Courtès
@ 2018-12-05  0:45       ` Oleg Pykhalov
  0 siblings, 0 replies; 5+ messages in thread
From: Oleg Pykhalov @ 2018-12-05  0:45 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: 33565-done, 33565

[-- Attachment #1: Type: text/plain, Size: 1187 bytes --]

ludo@gnu.org (Ludovic Courtès) writes:

> Oleg Pykhalov <go.wigust@gmail.com> skribis:
>
>> ludo@gnu.org (Ludovic Courtès) writes:
>>
>>> Oleg Pykhalov <go.wigust@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.”,
>
> @code{OrderedDict} even.  :-)
>
> LGTM with this change, thank you!

Pushed as 79a0eabd2a8765c294b3e2decf71a27cd1dd219d

Thanks,
Oleg.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2018-12-05  0:46 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-12-01 20:18 [bug#33565] [PATCH] gnu: Add python-xmltodict Oleg Pykhalov
2018-12-03 13:32 ` Ludovic Courtès
2018-12-04  5:13   ` Oleg Pykhalov
2018-12-04  8:47     ` Ludovic Courtès
2018-12-05  0:45       ` Oleg Pykhalov

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).