Re: [PATCH 2/2] gnu: python: Add i3-quickswitch.

2016-09-22 Thread Hartmut Goebel
Am 19.09.2016 um 23:42 schrieb Leo Famulari: > Our packaging guidelines say that we should use the "project name chosen > upstream" [0], which in this case appears to be quickswitch-for-i3 [1]. The project itself is not consistent about this. While the title of the readme says "quickswitch-for-i3"

Re: [PATCH 2/2] gnu: python: Add i3-quickswitch.

2016-09-20 Thread Ivan Vilata i Balaguer
Leo Famulari (2016-09-19 17:39:37 -0400) wrote: > On Mon, Sep 19, 2016 at 11:24:02AM +0200, Ivan Vilata i Balaguer wrote: > > Thanks Leo and Brendan for your comments, I'm writing new versions > > of the packages following your suggestions, unfortunately I found > > out that ``quickswitch.py``

Re: [PATCH 2/2] gnu: python: Add i3-quickswitch.

2016-09-19 Thread Leo Famulari
On Mon, Sep 19, 2016 at 08:20:26PM +1000, Brendan Tildesley wrote: > I'm not a fan of "quickswitch-for-i3" and "python-i3-py". Perhaps a > better solution is to ensure "i3-py" in the description or synopsis so > that `guix package -s' finds it. As for quickswitch, isn't the upstream > name also

Re: [PATCH 2/2] gnu: python: Add i3-quickswitch.

2016-09-19 Thread Leo Famulari
On Mon, Sep 19, 2016 at 11:28:09AM +0200, Ivan Vilata i Balaguer wrote: > Ivan Vilata i Balaguer (2016-09-19 11:24:02 +0200) wrote: > > > Thanks Leo and Brendan for your comments, I'm writing new versions of > > the packages following your suggestions, unfortunately I found out > > that

Re: [PATCH 2/2] gnu: python: Add i3-quickswitch.

2016-09-19 Thread Leo Famulari
On Mon, Sep 19, 2016 at 11:24:02AM +0200, Ivan Vilata i Balaguer wrote: > Thanks Leo and Brendan for your comments, I'm writing new versions of > the packages following your suggestions, unfortunately I found out that > ``quickswitch.py`` makes use of the Debian-specific ``which`` tool, so > I'm

Re: [PATCH 2/2] gnu: python: Add i3-quickswitch.

2016-09-19 Thread Brendan Tildesley
Leo Famulari 於 2016-09-16 15:46 寫道: > On Wed, Sep 14, 2016 at 11:17:24AM +0200, Ivan Vilata i Balaguer wrote: >> Subject: [PATCH 2/2] gnu: python: Add i3-quickswitch. >> >> * gnu/packages/python.scm (i3-quickswitch): New variable. > Cool, this works :) > >> +(define-public i3-quickswitch >> +

Re: [PATCH 2/2] gnu: python: Add i3-quickswitch.

2016-09-19 Thread Ivan Vilata i Balaguer
Ivan Vilata i Balaguer (2016-09-19 11:24:02 +0200) wrote: > Thanks Leo and Brendan for your comments, I'm writing new versions of > the packages following your suggestions, unfortunately I found out > that ``quickswitch.py`` makes use of the Debian-specific ``which`` > tool, so I'm also working

Re: [PATCH 2/2] gnu: python: Add i3-quickswitch.

2016-09-19 Thread Ivan Vilata i Balaguer
Leo Famulari (2016-09-16 01:46:37 -0400) wrote: > On Wed, Sep 14, 2016 at 11:17:24AM +0200, Ivan Vilata i Balaguer wrote: > > Subject: [PATCH 2/2] gnu: python: Add i3-quickswitch. > > > > * gnu/packages/python.scm (i3-quickswitch): New variable. > > Cool, this works :) > > > +(define-public

Re: [PATCH 2/2] gnu: python: Add i3-quickswitch.

2016-09-15 Thread Leo Famulari
On Wed, Sep 14, 2016 at 11:17:24AM +0200, Ivan Vilata i Balaguer wrote: > Subject: [PATCH 2/2] gnu: python: Add i3-quickswitch. > > * gnu/packages/python.scm (i3-quickswitch): New variable. Cool, this works :) > +(define-public i3-quickswitch > + (package > +(name "i3-quickswitch") Again,

Re: [PATCH 2/2] gnu: python: Add i3-quickswitch.

2016-09-14 Thread Brendan Tildesley
On 2016-09-14 19:17, Ivan Vilata i Balaguer wrote: Welcome to Guix. I tested this out in i3 and it works just fine. I may end up using it. the latest version seems to be 2.2, but I see you commented on that on the github, as it is not in pypi yet. Do we prefer to use pypi or directly source