From mboxrd@z Thu Jan 1 00:00:00 1970 From: ludo@gnu.org (Ludovic =?utf-8?Q?Court=C3=A8s?=) Subject: Re: [PATCH 3/3] gnu: Add newlib-nano-arm-none-eabi. Date: Wed, 21 Sep 2016 17:40:33 +0900 Message-ID: <87oa3hlk72.fsf@gnu.org> References: <20160918063513.11027-1-rekado@elephly.net> <20160918063513.11027-4-rekado@elephly.net> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:59242) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bmisP-0000Mj-Gf for guix-devel@gnu.org; Wed, 21 Sep 2016 10:52:03 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bmisJ-0005sv-TZ for guix-devel@gnu.org; Wed, 21 Sep 2016 10:51:57 -0400 In-Reply-To: <20160918063513.11027-4-rekado@elephly.net> (Ricardo Wurmus's message of "Sun, 18 Sep 2016 08:35:13 +0200") 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" To: Ricardo Wurmus Cc: guix-devel@gnu.org Ricardo Wurmus skribis: > * gnu/packages/embedded.scm (newlib-nano-arm-none-eabi): New variable. [...] > +(define-public newlib-nano-arm-none-eabi > + (package (inherit newlib-arm-none-eabi) > + (name "newlib-nano") I think it should be =E2=80=9Cnewlib-minimal=E2=80=9D, no? Or does =E2=80= =9Cnano=E2=80=9D have a special meaning? Also, make sure to add a synopsis that mentions its minimalness, and maybe a comment stating what the intended use case is, or how the feature set was chosen. Otherwise LGTM, thanks! Ludo=E2=80=99.