From mboxrd@z Thu Jan 1 00:00:00 1970 From: Cyril Roelandt Subject: Re: [PATCH 02/11] gnu: Add oslotest. Date: Thu, 23 Jul 2015 00:09:49 +0200 Message-ID: <55B014AD.6070806@gmail.com> References: <1437003151-23838-1-git-send-email-tipecaml@gmail.com> <1437003151-23838-3-git-send-email-tipecaml@gmail.com> <87380ofjd9.fsf@openmailbox.org> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:39906) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZI2D7-0000ej-6p for guix-devel@gnu.org; Wed, 22 Jul 2015 18:09:58 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZI2D4-0004Dy-1M for guix-devel@gnu.org; Wed, 22 Jul 2015 18:09:57 -0400 Received: from mail-wi0-x234.google.com ([2a00:1450:400c:c05::234]:36751) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZI2D3-0004Df-Qa for guix-devel@gnu.org; Wed, 22 Jul 2015 18:09:53 -0400 Received: by wicgb10 with SMTP id gb10so117455579wic.1 for ; Wed, 22 Jul 2015 15:09:52 -0700 (PDT) In-Reply-To: <87380ofjd9.fsf@openmailbox.org> List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org To: Mathieu Lirzin Cc: guix-devel@gnu.org 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 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.