[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 02/11] gnu: Add oslotest.
From: |
Cyril Roelandt |
Subject: |
Re: [PATCH 02/11] gnu: Add oslotest. |
Date: |
Thu, 23 Jul 2015 00:09:49 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Icedove/31.7.0 |
On 07/16/2015 01:10 PM, Mathieu Lirzin wrote:
> Hi,
>
> I've not test it but I have some stylistic remarks :). Most of them apply
> to other parts of your patch and to the other patches.
>
Oh, Indeed, I screwed up the indentation everywhere :/ I'll fix it when
resending the patches if it is needed.
> Cyril Roelandt <address@hidden> writes:
>
>> + (home-page "http://github.com/ActiveState/appdirs")
>> + (synopsis
>> + "A small Python module for determining appropriate \" +
>> \"platform-specific dirs, e.g. a \"user data dir\".")
>> + (description
>> + "A small Python module for determining appropriate \" +
>> \"platform-specific dirs, e.g. a \"user data dir\".")
>
> I guess this is something not intentional.
>
To be honest, I have no idea of what this package does, so I thought I'd
just use the synopsis as a description :)
Cyril.
- [PATCH 00/11] Add a few modules from the Oslo library., Cyril Roelandt, 2015/07/15
- [PATCH 01/11] gnu: Bump python-six to 1.9.0., Cyril Roelandt, 2015/07/15
- [PATCH 03/11] gnu: Add oslo.i18n., Cyril Roelandt, 2015/07/15
- [PATCH 02/11] gnu: Add oslotest., Cyril Roelandt, 2015/07/15
- Re: [PATCH 02/11] gnu: Add oslotest., Mathieu Lirzin, 2015/07/16
- Re: [PATCH 02/11] gnu: Add oslotest.,
Cyril Roelandt <=
- Re: [PATCH 02/11] gnu: Add oslotest., Thompson, David, 2015/07/20
- Re: [PATCH 02/11] gnu: Add oslotest., Cyril Roelandt, 2015/07/22
- Re: [PATCH 02/11] gnu: Add oslotest., Ian Denhardt, 2015/07/22
- Re: [PATCH 02/11] gnu: Add oslotest., Thompson, David, 2015/07/22
- Re: [PATCH 02/11] gnu: Add oslotest., Ludovic Courtès, 2015/07/23
- Re: [PATCH 02/11] gnu: Add oslotest., Cyril Roelandt, 2015/07/23
- Re: [PATCH 02/11] gnu: Add oslotest., Thompson, David, 2015/07/23
[PATCH 04/11] gnu: Add oslo.config, Cyril Roelandt, 2015/07/15
[PATCH 05/11] gnu: Add oslo.utils., Cyril Roelandt, 2015/07/15
[PATCH 06/11] gnu: Add oslo.context., Cyril Roelandt, 2015/07/15