From mboxrd@z Thu Jan 1 00:00:00 1970 From: Leo Famulari Subject: Re: [PATCH] gnu: Add re2. Date: Thu, 18 Aug 2016 16:43:10 -0400 Message-ID: <20160818204310.GD2393@jasmine> References: <87eg5qgp2r.fsf@ike.i-did-not-set--mail-host-address--so-tickle-me> <20160816175910.GB1260@jasmine> <87r39nzntl.fsf@ike.i-did-not-set--mail-host-address--so-tickle-me> <87bn0rzanb.fsf@ike.i-did-not-set--mail-host-address--so-tickle-me> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:33963) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1baU9s-0006se-Sw for guix-devel@gnu.org; Thu, 18 Aug 2016 16:43:25 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1baU9m-0004gd-Ti for guix-devel@gnu.org; Thu, 18 Aug 2016 16:43:23 -0400 Received: from out2-smtp.messagingengine.com ([66.111.4.26]:33667) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1baU9l-0004e3-Mf for guix-devel@gnu.org; Thu, 18 Aug 2016 16:43:18 -0400 Content-Disposition: inline In-Reply-To: <87bn0rzanb.fsf@ike.i-did-not-set--mail-host-address--so-tickle-me> 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: Marius Bakke Cc: guix-devel@gnu.org On Wed, Aug 17, 2016 at 04:18:48PM +0100, Marius Bakke wrote: > Marius Bakke writes: > > > Leo Famulari writes: > > > >> On Mon, Aug 15, 2016 at 02:05:16PM +0100, Marius Bakke wrote: > >>> > >>> I wasn't sure where to put this, so went with its own file. It does not > >>> fully implement PCRE so pcre.scm seems inappropriate. Perhaps that could > >>> be renamed to regex.scm or similar. > >> > >> The patch looks good, but please put it in regex.scm! But, I don't want > >> to move all the regex packages into this new regex module. Perhaps tre, > >> since it appears to have no users in our tree. > > > > Moved to regex.scm. Thanks! > > ..and here is a patch that moves tre.scm over as well. The code is > unchanged apart from license: prefix. When moving packages around, all users of the package need to have their module imports updated. Copyright attribution must be carefully handled. And merging the various *-updates branches into master and vice versa becomes more complicated and prone to error. I *think* this patch does it right. And it will really grate on my sense of aesthetics to have both regex.scm and tre.scm. But, in general, do we want to make this change? What does everyone think?