unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* bug#26890: [PATCH] gnu: Add agrep.
@ 2017-05-11 19:30 Ricardo Wurmus
  2017-05-11 21:04 ` Kei Kebreau
  0 siblings, 1 reply; 7+ messages in thread
From: Ricardo Wurmus @ 2017-05-11 19:30 UTC (permalink / raw)
  To: 26890; +Cc: Ricardo Wurmus

* 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@flashner.co.il>
 ;;; Copyright © 2016 Jan Nieuwenhuizen <janneke@gnu.org>
 ;;; Copyright © 2017 Marius Bakke <mbakke@fastmail.com>
+;;; Copyright © 2017 Ricardo Wurmus <rekado@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

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* bug#26890: [PATCH] gnu: Add agrep.
  2017-05-11 19:30 bug#26890: [PATCH] gnu: Add agrep Ricardo Wurmus
@ 2017-05-11 21:04 ` Kei Kebreau
  2017-05-11 21:14   ` Ricardo Wurmus
  2017-05-11 21:24   ` Leo Famulari
  0 siblings, 2 replies; 7+ messages in thread
From: Kei Kebreau @ 2017-05-11 21:04 UTC (permalink / raw)
  To: Ricardo Wurmus; +Cc: 26890

[-- Attachment #1: Type: text/plain, Size: 2744 bytes --]

Ricardo Wurmus <rekado@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@flashner.co.il>
>  ;;; Copyright © 2016 Jan Nieuwenhuizen <janneke@gnu.org>
>  ;;; Copyright © 2017 Marius Bakke <mbakke@fastmail.com>
> +;;; Copyright © 2017 Ricardo Wurmus <rekado@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?

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]

^ permalink raw reply	[flat|nested] 7+ messages in thread

* bug#26890: [PATCH] gnu: Add agrep.
  2017-05-11 21:04 ` Kei Kebreau
@ 2017-05-11 21:14   ` Ricardo Wurmus
  2017-05-11 22:03     ` Marius Bakke
  2017-05-11 21:24   ` Leo Famulari
  1 sibling, 1 reply; 7+ messages in thread
From: Ricardo Wurmus @ 2017-05-11 21:14 UTC (permalink / raw)
  To: Kei Kebreau; +Cc: 26890


Kei Kebreau <kei@openmailbox.org> writes:

> Ricardo Wurmus <rekado@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@flashner.co.il>
>>  ;;; Copyright © 2016 Jan Nieuwenhuizen <janneke@gnu.org>
>>  ;;; Copyright © 2017 Marius Bakke <mbakke@fastmail.com>
>> +;;; Copyright © 2017 Ricardo Wurmus <rekado@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

^ permalink raw reply	[flat|nested] 7+ messages in thread

* bug#26890: [PATCH] gnu: Add agrep.
  2017-05-11 21:04 ` Kei Kebreau
  2017-05-11 21:14   ` Ricardo Wurmus
@ 2017-05-11 21:24   ` Leo Famulari
  2017-05-11 21:36     ` Ricardo Wurmus
  1 sibling, 1 reply; 7+ messages in thread
From: Leo Famulari @ 2017-05-11 21:24 UTC (permalink / raw)
  To: Kei Kebreau; +Cc: Ricardo Wurmus, 26890

[-- Attachment #1: Type: text/plain, Size: 365 bytes --]

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

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 7+ messages in thread

* bug#26890: [PATCH] gnu: Add agrep.
  2017-05-11 21:24   ` Leo Famulari
@ 2017-05-11 21:36     ` Ricardo Wurmus
  2017-06-22 20:10       ` Ludovic Courtès
  0 siblings, 1 reply; 7+ messages in thread
From: Ricardo Wurmus @ 2017-05-11 21:36 UTC (permalink / raw)
  To: Leo Famulari; +Cc: 26890, Kei Kebreau


Leo Famulari <leo@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

^ permalink raw reply	[flat|nested] 7+ messages in thread

* bug#26890: [PATCH] gnu: Add agrep.
  2017-05-11 21:14   ` Ricardo Wurmus
@ 2017-05-11 22:03     ` Marius Bakke
  0 siblings, 0 replies; 7+ messages in thread
From: Marius Bakke @ 2017-05-11 22:03 UTC (permalink / raw)
  To: Ricardo Wurmus, Kei Kebreau; +Cc: 26890

[-- Attachment #1: Type: text/plain, Size: 256 bytes --]

Ricardo Wurmus <rekado@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.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 487 bytes --]

^ permalink raw reply	[flat|nested] 7+ messages in thread

* bug#26890: [PATCH] gnu: Add agrep.
  2017-05-11 21:36     ` Ricardo Wurmus
@ 2017-06-22 20:10       ` Ludovic Courtès
  0 siblings, 0 replies; 7+ messages in thread
From: Ludovic Courtès @ 2017-06-22 20:10 UTC (permalink / raw)
  To: Ricardo Wurmus; +Cc: 26890-done, Kei Kebreau

Ricardo Wurmus <rekado@elephly.net> skribis:

> Leo Famulari <leo@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’.

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2017-06-22 20:11 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-05-11 19:30 bug#26890: [PATCH] gnu: Add agrep Ricardo Wurmus
2017-05-11 21:04 ` Kei Kebreau
2017-05-11 21:14   ` Ricardo Wurmus
2017-05-11 22:03     ` Marius Bakke
2017-05-11 21:24   ` Leo Famulari
2017-05-11 21:36     ` Ricardo Wurmus
2017-06-22 20:10       ` Ludovic Courtès

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).