Patchwork [06/15] gnu: Add chez-irregex.

login
register
mail settings
Submitter Federico Beffa
Date Oct. 16, 2016, 3:31 p.m.
Message ID <CAKrPhPOq5LjN-5oMLp04HoKUBE=fPmNpBmw3VgL6Md77QFbXEg@mail.gmail.com>
Download mbox | patch
Permalink /patch/16557/
State New
Headers show

Comments

Federico Beffa - Oct. 16, 2016, 3:31 p.m.

Ludovic Courtès - Oct. 17, 2016, 2:40 p.m.
Federico Beffa <beffa@ieee.org> skribis:

> From 4293c55d921d82c0542907024a1b51abea32b6dc Mon Sep 17 00:00:00 2001
> From: Federico Beffa <beffa@fbengineering.ch>
> Date: Sun, 16 Oct 2016 16:10:31 +0200
> Subject: [PATCH 06/15] gnu: Add chez-irregex.
>
> * gnu/packages/chez.scm (chez-irregex): New variable.

[...]

> +       (uri (string-append
> +             "https://github.com/fedeinthemix/chez-irregex/archive"
> +             "/v" version ".tar.gz"))

Wouldn’t it work to use the upstream Irregex as done in the
‘guile-irregex’ package?

Otherwise LGTM.

Ludo’.
Federico Beffa - Oct. 17, 2016, 4:19 p.m.
On Mon, Oct 17, 2016 at 4:40 PM, Ludovic Courtès <ludo@gnu.org> wrote:
> Federico Beffa <beffa@ieee.org> skribis:
>
>> From 4293c55d921d82c0542907024a1b51abea32b6dc Mon Sep 17 00:00:00 2001
>> From: Federico Beffa <beffa@fbengineering.ch>
>> Date: Sun, 16 Oct 2016 16:10:31 +0200
>> Subject: [PATCH 06/15] gnu: Add chez-irregex.
>>
>> * gnu/packages/chez.scm (chez-irregex): New variable.
>
> [...]
>
>> +       (uri (string-append
>> +             "https://github.com/fedeinthemix/chez-irregex/archive"
>> +             "/v" version ".tar.gz"))
>
> Wouldn’t it work to use the upstream Irregex as done in the
> ‘guile-irregex’ package?

The original source includes a module definition for Guile, but not
one for R6RS Scheme. This package wraps the original code in an R6RS
library definition and adds a file to make it easy to run the test
suite.

>
> Otherwise LGTM.
>
> Ludo’.
Ludovic Courtès - Oct. 17, 2016, 8:09 p.m.
Federico Beffa <beffa@ieee.org> skribis:

> On Mon, Oct 17, 2016 at 4:40 PM, Ludovic Courtès <ludo@gnu.org> wrote:
>> Federico Beffa <beffa@ieee.org> skribis:
>>
>>> From 4293c55d921d82c0542907024a1b51abea32b6dc Mon Sep 17 00:00:00 2001
>>> From: Federico Beffa <beffa@fbengineering.ch>
>>> Date: Sun, 16 Oct 2016 16:10:31 +0200
>>> Subject: [PATCH 06/15] gnu: Add chez-irregex.
>>>
>>> * gnu/packages/chez.scm (chez-irregex): New variable.
>>
>> [...]
>>
>>> +       (uri (string-append
>>> +             "https://github.com/fedeinthemix/chez-irregex/archive"
>>> +             "/v" version ".tar.gz"))
>>
>> Wouldn’t it work to use the upstream Irregex as done in the
>> ‘guile-irregex’ package?
>
> The original source includes a module definition for Guile, but not
> one for R6RS Scheme. This package wraps the original code in an R6RS
> library definition and adds a file to make it easy to run the test
> suite.

OK, makes sense.  Thanks for explaining!

Ludo’.

Patch

From 4293c55d921d82c0542907024a1b51abea32b6dc Mon Sep 17 00:00:00 2001
From: Federico Beffa <beffa@fbengineering.ch>
Date: Sun, 16 Oct 2016 16:10:31 +0200
Subject: [PATCH 06/15] gnu: Add chez-irregex.

* gnu/packages/chez.scm (chez-irregex): New variable.
---
 gnu/packages/chez.scm | 33 +++++++++++++++++++++++++++++++++
 1 file changed, 33 insertions(+)

diff --git a/gnu/packages/chez.scm b/gnu/packages/chez.scm
index bc763df..34fd225 100644
--- a/gnu/packages/chez.scm
+++ b/gnu/packages/chez.scm
@@ -397,3 +397,36 @@  Chez Scheme.")
 Scheme 'match' package by Andrew Wright, written in fully portable
 'syntax-rules' and thus preserving hygiene.")
     (license public-domain)))
+
+(define-public chez-irregex
+  (package
+    (name "chez-irregex")
+    (version "0.9.4")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append
+             "https://github.com/fedeinthemix/chez-irregex/archive"
+             "/v" version ".tar.gz"))
+       (sha256
+        (base32 "0ywy5syaw549a58viz68dmgnv756ic705rcnlqxgjq27lnaim53b"))
+       (file-name (string-append name "-" version ".tar.gz"))))
+    (build-system gnu-build-system)
+    (inputs
+     `(("chez-matchable" ,chez-matchable))) ; for tests
+    (propagated-inputs
+     `(("chez-srfi" ,chez-srfi))) ; for irregex-utils
+    (native-inputs
+     `(("chez-scheme" ,chez-scheme)))
+    (arguments
+     `(#:make-flags ,(chez-make-flags name version)
+       #:test-target "test"
+       #:phases (modify-phases %standard-phases
+                  (replace 'configure ,chez-configure))))
+    (home-page "https://github.com/fedeinthemix/chez-irregex")
+    (synopsis "Portable regular expression library for Scheme")
+    (description "This package provides a portable and efficient
+R[4567]RS implementation of regular expressions, supporting both POSIX
+syntax with various (irregular) PCRE extensions, as well as SCSH's SRE
+syntax, with various aliases for commonly used patterns.")
+    (license bsd-3)))
-- 
2.7.4