guix-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] gnu: Add re2.


From: Leo Famulari
Subject: Re: [PATCH] gnu: Add re2.
Date: Thu, 18 Aug 2016 16:43:10 -0400
User-agent: Mutt/1.6.2hg (2016-07-01)

On Wed, Aug 17, 2016 at 04:18:48PM +0100, Marius Bakke wrote:
> Marius Bakke <address@hidden> writes:
> 
> > Leo Famulari <address@hidden> 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?



reply via email to

[Prev in Thread] Current Thread [Next in Thread]