From mboxrd@z Thu Jan 1 00:00:00 1970 From: Leo Famulari Subject: Re: [PATCH 0/1] Move cursynth to music.scm Date: Wed, 30 Dec 2015 21:31:14 -0500 Message-ID: <20151231023114.GB24128@jasmine> References: <20151228171715.5450cca0@openmailbox.org> <20151229043237.GB2878@jasmine> <87k2nx4tq6.fsf@elephly.net> <877fjwzv7e.fsf@gnu.org> <20151230011002.GA9185@jasmine> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:57419) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aET1J-0000ol-Vo for guix-devel@gnu.org; Wed, 30 Dec 2015 21:31:18 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aET1G-0004Ts-OX for guix-devel@gnu.org; Wed, 30 Dec 2015 21:31:17 -0500 Received: from out4-smtp.messagingengine.com ([66.111.4.28]:35968) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aET1G-0004Rv-LZ for guix-devel@gnu.org; Wed, 30 Dec 2015 21:31:14 -0500 Content-Disposition: inline In-Reply-To: <20151230011002.GA9185@jasmine> 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: Ludovic =?iso-8859-1?Q?Court=E8s?= Cc: guix-devel@gnu.org On Tue, Dec 29, 2015 at 08:10:02PM -0500, Leo Famulari wrote: > On Wed, Dec 30, 2015 at 12:33:25AM +0100, Ludovic Courtès wrote: > > Ricardo Wurmus skribis: > > > > > Leo Famulari writes: > > > > > >> On Mon, Dec 28, 2015 at 05:17:15PM -0600, Eric Bavier wrote: > > >>> On Mon, 28 Dec 2015 18:09:09 -0500 > > >>> Leo Famulari wrote: > > >>> > > >>> > I think it would be better for this software synthesizer to be in > > >>> > music.scm. > > >>> > > > >>> > Thoughts? > > >>> > > >>> IIRC, the original thought was that many GNU packages have their own > > >>> modules, so this was done for cursynth as well. > > >> > > >> Okay, sure. > > > > > > I think it would be nice to have cursynth in “music.scm”. > > > > +1 > > > > >> To be honest, I don't understand the reasoning behind grouping packages > > >> into modules. Is it just for humans or is there some technical reason > > >> for it? > > > > > > It’s mostly for humans AFAIU. Personally, I prefer try to avoid a > > > proliferation of one-off modules; maybe because I don’t like the > > > boilerplate (license header, module definition with imports, adding the > > > module to “gnu-system.am”). > > > > Same here. > > > > More modules also lead to more I/O for the various commands. > > That's a good reason. > > I'll apply the patch if there are no objections. Pushed as 658212ebf7. > > > > > Ludo’. >