GNU bug report logs - #25723
[PATCH] gnu: Add python-cycler.

Previous Next

Full log

Message #13 received at 25723-done <at> debbugs.gnu.org (full text, mbox):

From: Marius Bakke <mbakke <at> fastmail.com>
To: Ricardo Wurmus <rekado <at> elephly.net>
Cc: 25723-done <at> debbugs.gnu.org
Subject: Re: bug#25723: [PATCH] gnu: Add python-cycler.
Date: Sun, 19 Feb 2017 22:49:47 +0100

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

Hi Ricardo,

Thanks a lot for the feedback! 

Ricardo Wurmus <rekado <at> elephly.net> writes:

>> +    (propagated-inputs
>> +     `(("python-six" ,python-six)))
>
> Is python-six really required at runtime?  Since you offer a variant for
> Python 2, should python-six only be used with one of these variants?

python-six is imported unconditionally by cycler.py. I believe this is
common practice, and the "six" class offers compatibility "both ways",
not just e.g. 2->3.

>> +    (home-page "http://matplotlib.org/cycler/")
>> +    (synopsis "Composable kwarg iterator")
>
> I assume that “kwarg” is a generally known term in Python, but maybe it
> is not.  Could this be replaced with “keyword argument”?

Yes, that is better, thanks!

>
>> +    (description
>> +     "When using @code{matplotlib} and plotting more than one line, it is
>> +common to want to be able to cycle over one or more artist styles; but the
>> +plotting logic can quickly become involved.
>> +
>> +To address this and enable easy cycling over arbitrary @code{kwargs}, the
>> +@code{Cycler} class was developed.")
>
> I would remove the empty line.
>
>> +    (license license:bsd-3)))
>> +
>> +(define-public python2-cycler
>> +  (package-with-python2 python-cycler))
>> +
>
> See question above about python-six.
> Other than that it looks good to me.  Thank you!

Committed with these changes, thank you!

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