diff mbox

gnu: Add python-wcwidth and python2-wcwidth.

Message ID 20160613205914.GA12302@jasmine
State New
Headers show

Commit Message

Leo Famulari June 13, 2016, 8:59 p.m. UTC
On Sun, Jun 12, 2016 at 05:40:06PM +0200, Danny Milosavljevic wrote:
> * gnu/packages/python.scm (python-wcwidth, python2-wcwidth): New variables.

Thanks for the patch!

What do you think of the attached changes to the synopsis and
description?

Comments

Danny Milosavljevic June 14, 2016, 12:13 p.m. UTC | #1
On Mon, 13 Jun 2016 16:59:14 -0400
Leo Famulari <leo@famulari.name> wrote:

> On Sun, Jun 12, 2016 at 05:40:06PM +0200, Danny Milosavljevic wrote:
> > * gnu/packages/python.scm (python-wcwidth, python2-wcwidth): New variables.  
> 
> Thanks for the patch!
> 
> What do you think of the attached changes to the synopsis and
> description?

Nice!

>@code{wcwidth} and @code{wcwidth}

Errrr
Leo Famulari June 14, 2016, 1:41 p.m. UTC | #2
On Tue, Jun 14, 2016 at 02:13:32PM +0200, Danny Milosavljevic wrote:
> On Mon, 13 Jun 2016 16:59:14 -0400
> Leo Famulari <leo@famulari.name> wrote:
> 
> > On Sun, Jun 12, 2016 at 05:40:06PM +0200, Danny Milosavljevic wrote:
> > > * gnu/packages/python.scm (python-wcwidth, python2-wcwidth): New variables.  
> > 
> > Thanks for the patch!
> > 
> > What do you think of the attached changes to the synopsis and
> > description?
> 
> Nice!
> 
> >@code{wcwidth} and @code{wcwidth}
> 
> Errrr

I'm not sure how to interpret that :) Is it a negative reaction?

It's texinfo markup that allows the marked up text to be displayed "as
code", however you want to interpret that. I think it is monospaced in
some formats. In `guix package --show` it is quoted in a particular way.
Danny Milosavljevic June 14, 2016, 11:15 p.m. UTC | #3
On Tue, 14 Jun 2016 09:41:01 -0400
Leo Famulari <leo@famulari.name> wrote:

> > >@code{wcwidth} and @code{wcwidth}  

Is that twice the same thing?
Leo Famulari June 15, 2016, 1:52 a.m. UTC | #4
On Wed, Jun 15, 2016 at 01:15:37AM +0200, Danny Milosavljevic wrote:
> On Tue, 14 Jun 2016 09:41:01 -0400
> Leo Famulari <leo@famulari.name> wrote:
> 
> > > >@code{wcwidth} and @code{wcwidth}  
> 
> Is that twice the same thing?

Yes! :p

The second one should be wcswidth.
Leo Famulari June 15, 2016, 2:45 p.m. UTC | #5
On Tue, Jun 14, 2016 at 09:52:01PM -0400, Leo Famulari wrote:
> On Wed, Jun 15, 2016 at 01:15:37AM +0200, Danny Milosavljevic wrote:
> > On Tue, 14 Jun 2016 09:41:01 -0400
> > Leo Famulari <leo@famulari.name> wrote:
> > 
> > > > >@code{wcwidth} and @code{wcwidth}  
> > 
> > Is that twice the same thing?
> 
> Yes! :p
> 
> The second one should be wcswidth.

Anyways, pushed as a44fd439dc! Thanks!
diff mbox

Patch

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index 3168de7..4c15771 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -9197,18 +9197,19 @@  are optionally backed by a C extension built on librdkafka.")
       (uri (string-append
              "https://pypi.python.org/packages/"
              "c2/d1/7689293086a8d5320025080cde0e3155b94ae0a7496fb89a3fbaa92c354a/"
-             "wcwidth-"
-             version
-             ".tar.gz"))
+             "wcwidth-" version ".tar.gz"))
       (sha256
         (base32
           "02wjrpf001gjdjsaxxbzcwfg19crlk2dbddayrfc2v06f53yrcyw"))))
   (build-system python-build-system)
   (home-page "https://github.com/jquast/wcwidth")
   (synopsis
-    "Measures number of Terminal column cells of wide-character codes")
-  (description
-    "Measures number of Terminal column cells of wide-character codes")
+    "Measures number of terminal column cells of wide-character codes.")
+  (description "Wcwidth measures the number of terminal column cells of
+wide-character codes.  It is useful for those implementing a terminal emulator,
+or programs that carefully produce output to be interpreted by one.  It is a
+Python implementation of the @code{wcwidth} and @code{wcwidth} C functions
+specified in POSIX.1-2001 and POSIX.1-2008.")
   (license license:expat)))
 
 (define-public python2-wcwidth