From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marius Bakke Subject: Re: [PATCH] gnu: Add re2. Date: Fri, 19 Aug 2016 12:37:55 +0100 Message-ID: <87y43txa3w.fsf@ike.i-did-not-set--mail-host-address--so-tickle-me> 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> <20160818204310.GD2393@jasmine> Mime-Version: 1.0 Content-Type: text/plain Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:36205) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bai7o-0002Bf-F9 for guix-devel@gnu.org; Fri, 19 Aug 2016 07:38:13 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bai7k-00072w-6i for guix-devel@gnu.org; Fri, 19 Aug 2016 07:38:11 -0400 Received: from out2-smtp.messagingengine.com ([66.111.4.26]:48055) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bai7h-00072I-Oe for guix-devel@gnu.org; Fri, 19 Aug 2016 07:38:08 -0400 In-Reply-To: <20160818204310.GD2393@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" To: Alex Kost , Leo Famulari Cc: guix-devel@gnu.org Leo Famulari writes: > 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? I agree that package moves should be kept at a minimum. Not only does it make merges difficult, but git log/blame information is also "lost", as in having to check out pre-move versions to trace the history. It can also break third-party repositories as I found after tcsh move. ...for this particular package (tre) however, I think a move is inevitable. It has no in-tree users and will forever stay alone in tre.scm. It doesn't mean we have to go out of our way to move everything else though. There are more important things to spend time on right now.