GNU bug report logs - #26890
[PATCH] gnu: Add agrep.

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 26890 in the body.
You can then email your comments to 26890 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Ricardo Wurmus <rekado <at> elephly.net>
To: guix-patches <at> gnu.org
Cc: Ricardo Wurmus <rekado <at> elephly.net>
Subject: [PATCH] gnu: Add agrep.
Date: Thu, 11 May 2017 21:30:45 +0200

* gnu/packages/base.scm (agrep): New variable.
---
 gnu/packages/base.scm | 33 ++++++++++++++++++++++++++++++++-
 1 file changed, 32 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/base.scm b/gnu/packages/base.scm
index 59a9acd67..a07638c38 100644
--- a/gnu/packages/base.scm
+++ b/gnu/packages/base.scm
@@ -8,6 +8,7 @@
 ;;; Copyright © 2016 Efraim Flashner <efraim <at> flashner.co.il>
 ;;; Copyright © 2016 Jan Nieuwenhuizen <janneke <at> gnu.org>
 ;;; Copyright © 2017 Marius Bakke <mbakke <at> fastmail.com>
+;;; Copyright © 2017 Ricardo Wurmus <rekado <at> elephly.net>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -26,7 +27,7 @@
 
 (define-module (gnu packages base)
   #:use-module ((guix licenses)
-                #:select (gpl3+ lgpl2.0+ lgpl3+ public-domain))
+                #:select (gpl2+ gpl3+ lgpl2.0+ lgpl3+ public-domain))
   #:use-module (gnu packages)
   #:use-module (gnu packages acl)
   #:use-module (gnu packages bash)
@@ -116,6 +117,36 @@ including, for example, recursive directory searching.")
    (license gpl3+)
    (home-page "https://www.gnu.org/software/grep/")))
 
+(define-public agrep
+  (package
+    (name "agrep")
+    (version "3.41")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append "https://www.tgries.de/agrep/agrep-"
+                                  version ".tgz"))
+              (sha256
+               (base32
+                "0ki3gbvc7lh8alks9zm48f216l2gp8r6vssmr5yccpvjz7xfl205"))))
+    (build-system gnu-build-system)
+    (arguments
+     `(#:tests? #f ; no "check" target
+       #:phases
+       (modify-phases %standard-phases
+         (delete 'configure)
+         (replace 'install
+           (lambda* (#:key outputs #:allow-other-keys)
+             (let ((bin (string-append (assoc-ref outputs "out") "/bin")))
+               (mkdir-p bin)
+               (install-file "agrep" bin)
+               #t))))))
+    (home-page "https://www.tgries.de/agrep/")
+    (synopsis "Approximate grep for fast fuzzy string searching")
+    (description "Agrep is a tool for fast searching of files for a string or
+regular expression, with approximate matching capabilities and user-definable
+records.")
+    (license gpl2+)))
+
 (define-public sed
   (package
    (name "sed")
-- 
2.12.2

Message #8 received at 26890 <at> debbugs.gnu.org (full text, mbox):

From: Kei Kebreau <kei <at> openmailbox.org>
To: Ricardo Wurmus <rekado <at> elephly.net>
Cc: 26890 <at> debbugs.gnu.org
Subject: Re: bug#26890: [PATCH] gnu: Add agrep.
Date: Thu, 11 May 2017 17:04:08 -0400

[Message part 1 (text/plain, inline)]

Ricardo Wurmus <rekado <at> elephly.net> writes:

> * gnu/packages/base.scm (agrep): New variable.
> ---
>  gnu/packages/base.scm | 33 ++++++++++++++++++++++++++++++++-
>  1 file changed, 32 insertions(+), 1 deletion(-)
>
> diff --git a/gnu/packages/base.scm b/gnu/packages/base.scm
> index 59a9acd67..a07638c38 100644
> --- a/gnu/packages/base.scm
> +++ b/gnu/packages/base.scm
> @@ -8,6 +8,7 @@
>  ;;; Copyright © 2016 Efraim Flashner <efraim <at> flashner.co.il>
>  ;;; Copyright © 2016 Jan Nieuwenhuizen <janneke <at> gnu.org>
>  ;;; Copyright © 2017 Marius Bakke <mbakke <at> fastmail.com>
> +;;; Copyright © 2017 Ricardo Wurmus <rekado <at> elephly.net>
>  ;;;
>  ;;; This file is part of GNU Guix.
>  ;;;
> @@ -26,7 +27,7 @@
>  
>  (define-module (gnu packages base)
>    #:use-module ((guix licenses)
> -                #:select (gpl3+ lgpl2.0+ lgpl3+ public-domain))
> +                #:select (gpl2+ gpl3+ lgpl2.0+ lgpl3+ public-domain))
>    #:use-module (gnu packages)
>    #:use-module (gnu packages acl)
>    #:use-module (gnu packages bash)
> @@ -116,6 +117,36 @@ including, for example, recursive directory searching.")
>     (license gpl3+)
>     (home-page "https://www.gnu.org/software/grep/")))
>  
> +(define-public agrep
> +  (package
> +    (name "agrep")
> +    (version "3.41")
> +    (source (origin
> +              (method url-fetch)
> +              (uri (string-append "https://www.tgries.de/agrep/agrep-"
> +                                  version ".tgz"))
> +              (sha256
> +               (base32
> +                "0ki3gbvc7lh8alks9zm48f216l2gp8r6vssmr5yccpvjz7xfl205"))))
> +    (build-system gnu-build-system)
> +    (arguments
> +     `(#:tests? #f ; no "check" target
> +       #:phases
> +       (modify-phases %standard-phases
> +         (delete 'configure)
> +         (replace 'install
> +           (lambda* (#:key outputs #:allow-other-keys)
> +             (let ((bin (string-append (assoc-ref outputs "out") "/bin")))
> +               (mkdir-p bin)
> +               (install-file "agrep" bin)
> +               #t))))))
> +    (home-page "https://www.tgries.de/agrep/")
> +    (synopsis "Approximate grep for fast fuzzy string searching")
> +    (description "Agrep is a tool for fast searching of files for a string or
> +regular expression, with approximate matching capabilities and user-definable
> +records.")
> +    (license gpl2+)))
> +
>  (define-public sed
>    (package
>     (name "sed")

Quote from copying.agrep in the source directory:
  4. Redistribution for profit requires the express, written permission
     of the authors.

There is a similar clause on the website. Does the GPL or this clause
take precedence?

[signature.asc (application/pgp-signature, inline)]

Message #11 received at 26890 <at> debbugs.gnu.org (full text, mbox):

From: Ricardo Wurmus <rekado <at> elephly.net>
To: Kei Kebreau <kei <at> openmailbox.org>
Cc: 26890 <at> debbugs.gnu.org
Subject: Re: bug#26890: [PATCH] gnu: Add agrep.
Date: Thu, 11 May 2017 23:14:40 +0200

Kei Kebreau <kei <at> openmailbox.org> writes:

> Ricardo Wurmus <rekado <at> elephly.net> writes:
>
>> * gnu/packages/base.scm (agrep): New variable.
>> ---
>>  gnu/packages/base.scm | 33 ++++++++++++++++++++++++++++++++-
>>  1 file changed, 32 insertions(+), 1 deletion(-)
>>
>> diff --git a/gnu/packages/base.scm b/gnu/packages/base.scm
>> index 59a9acd67..a07638c38 100644
>> --- a/gnu/packages/base.scm
>> +++ b/gnu/packages/base.scm
>> @@ -8,6 +8,7 @@
>>  ;;; Copyright © 2016 Efraim Flashner <efraim <at> flashner.co.il>
>>  ;;; Copyright © 2016 Jan Nieuwenhuizen <janneke <at> gnu.org>
>>  ;;; Copyright © 2017 Marius Bakke <mbakke <at> fastmail.com>
>> +;;; Copyright © 2017 Ricardo Wurmus <rekado <at> elephly.net>
>>  ;;;
>>  ;;; This file is part of GNU Guix.
>>  ;;;
>> @@ -26,7 +27,7 @@
>>  
>>  (define-module (gnu packages base)
>>    #:use-module ((guix licenses)
>> -                #:select (gpl3+ lgpl2.0+ lgpl3+ public-domain))
>> +                #:select (gpl2+ gpl3+ lgpl2.0+ lgpl3+ public-domain))
>>    #:use-module (gnu packages)
>>    #:use-module (gnu packages acl)
>>    #:use-module (gnu packages bash)
>> @@ -116,6 +117,36 @@ including, for example, recursive directory searching.")
>>     (license gpl3+)
>>     (home-page "https://www.gnu.org/software/grep/")))
>>  
>> +(define-public agrep
>> +  (package
>> +    (name "agrep")
>> +    (version "3.41")
>> +    (source (origin
>> +              (method url-fetch)
>> +              (uri (string-append "https://www.tgries.de/agrep/agrep-"
>> +                                  version ".tgz"))
>> +              (sha256
>> +               (base32
>> +                "0ki3gbvc7lh8alks9zm48f216l2gp8r6vssmr5yccpvjz7xfl205"))))
>> +    (build-system gnu-build-system)
>> +    (arguments
>> +     `(#:tests? #f ; no "check" target
>> +       #:phases
>> +       (modify-phases %standard-phases
>> +         (delete 'configure)
>> +         (replace 'install
>> +           (lambda* (#:key outputs #:allow-other-keys)
>> +             (let ((bin (string-append (assoc-ref outputs "out") "/bin")))
>> +               (mkdir-p bin)
>> +               (install-file "agrep" bin)
>> +               #t))))))
>> +    (home-page "https://www.tgries.de/agrep/")
>> +    (synopsis "Approximate grep for fast fuzzy string searching")
>> +    (description "Agrep is a tool for fast searching of files for a string or
>> +regular expression, with approximate matching capabilities and user-definable
>> +records.")
>> +    (license gpl2+)))
>> +
>>  (define-public sed
>>    (package
>>     (name "sed")
>
> Quote from copying.agrep in the source directory:
>   4. Redistribution for profit requires the express, written permission
>      of the authors.
>
> There is a similar clause on the website. Does the GPL or this clause
> take precedence?

Hmm, this is confusing.  The COPYRIGHT and README files in the git
repository say that the license is ICS since Sept 18, 2014, but that’s
not part of the tarball.

:~/ ?

I’m not so happy to put this in base, by the way.  I just added it
there because that’s where grep is.  Any other ideas?

-- 
Ricardo

GPG: BCA6 89B6 3655 3801 C3C6  2150 197A 5888 235F ACAC
https://elephly.net

Message #14 received at 26890 <at> debbugs.gnu.org (full text, mbox):

From: Leo Famulari <leo <at> famulari.name>
To: Kei Kebreau <kei <at> openmailbox.org>
Cc: Ricardo Wurmus <rekado <at> elephly.net>, 26890 <at> debbugs.gnu.org
Subject: Re: bug#26890: [PATCH] gnu: Add agrep.
Date: Thu, 11 May 2017 17:24:41 -0400

[Message part 1 (text/plain, inline)]

On Thu, May 11, 2017 at 05:04:08PM -0400, Kei Kebreau wrote:
> Quote from copying.agrep in the source directory:
>   4. Redistribution for profit requires the express, written permission
>      of the authors.

Unfortunately, this clause makes agrep not free software. Indeed, Debian
distributes it from their non-free repos:

https://packages.debian.org/sid/agrep

[signature.asc (application/pgp-signature, inline)]

Message #17 received at 26890 <at> debbugs.gnu.org (full text, mbox):

From: Ricardo Wurmus <rekado <at> elephly.net>
To: Leo Famulari <leo <at> famulari.name>
Cc: 26890 <at> debbugs.gnu.org, Kei Kebreau <kei <at> openmailbox.org>
Subject: Re: bug#26890: [PATCH] gnu: Add agrep.
Date: Thu, 11 May 2017 23:36:35 +0200

Leo Famulari <leo <at> famulari.name> writes:

> On Thu, May 11, 2017 at 05:04:08PM -0400, Kei Kebreau wrote:
>> Quote from copying.agrep in the source directory:
>>   4. Redistribution for profit requires the express, written permission
>>      of the authors.
>
> Unfortunately, this clause makes agrep not free software. Indeed, Debian
> distributes it from their non-free repos:
>
> https://packages.debian.org/sid/agrep

This is confusing.  Debian has version 4.17-9 but I don’t even know
where they got it from.  The home page that’s linked there only has 2.x
tarballs.

And this clashes with the declaration of the ICS version in the git
repo.

Weird!  I really don’t mind dropping this patch (I just packaged it
while demonstrating Guix), but it’s sad that the license situation here
is so contradictory.

--
Ricardo

GPG: BCA6 89B6 3655 3801 C3C6  2150 197A 5888 235F ACAC
https://elephly.net

Message #20 received at 26890 <at> debbugs.gnu.org (full text, mbox):

From: Marius Bakke <mbakke <at> fastmail.com>
To: Ricardo Wurmus <rekado <at> elephly.net>, Kei Kebreau <kei <at> openmailbox.org>
Cc: 26890 <at> debbugs.gnu.org
Subject: Re: bug#26890: [PATCH] gnu: Add agrep.
Date: Fri, 12 May 2017 00:03:57 +0200

[Message part 1 (text/plain, inline)]

Ricardo Wurmus <rekado <at> elephly.net> writes:

> I’m not so happy to put this in base, by the way.  I just added it
> there because that’s where grep is.  Any other ideas?

I'd use 'textutils.scm', but there is also 'code.scm' with similar
tools.

[signature.asc (application/pgp-signature, inline)]

Message #25 received at 26890-done <at> debbugs.gnu.org (full text, mbox):

From: ludo <at> gnu.org (Ludovic Courtès)
To: Ricardo Wurmus <rekado <at> elephly.net>
Cc: 26890-done <at> debbugs.gnu.org, Kei Kebreau <kei <at> openmailbox.org>,
 Leo Famulari <leo <at> famulari.name>
Subject: Re: bug#26890: [PATCH] gnu: Add agrep.
Date: Thu, 22 Jun 2017 22:10:04 +0200

Ricardo Wurmus <rekado <at> elephly.net> skribis:

> Leo Famulari <leo <at> famulari.name> writes:
>
>> On Thu, May 11, 2017 at 05:04:08PM -0400, Kei Kebreau wrote:
>>> Quote from copying.agrep in the source directory:
>>>   4. Redistribution for profit requires the express, written permission
>>>      of the authors.
>>
>> Unfortunately, this clause makes agrep not free software. Indeed, Debian
>> distributes it from their non-free repos:
>>
>> https://packages.debian.org/sid/agrep
>
> This is confusing.  Debian has version 4.17-9 but I don’t even know
> where they got it from.  The home page that’s linked there only has 2.x
> tarballs.
>
> And this clashes with the declaration of the ICS version in the git
> repo.
>
> Weird!  I really don’t mind dropping this patch (I just packaged it
> while demonstrating Guix), but it’s sad that the license situation here
> is so contradictory.

So I’m closing this entry if you don’t mind.  :-)

Ludo’.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.