[PATCH] gnu: pcre: Update to 8.36 and enable JIT support

[PATCH] gnu: pcre: Update to 8.36 and enable JIT support

[Mark H Weaver]

[-- Attachment #1: [PATCH] gnu: pcre: Update to 8.36 and enable JIT support --]
[-- Type: text/x-patch, Size: 1695 bytes --]

From 5796cf1178fbcf41ef1c3d6406adb992d7e71dd0 Mon Sep 17 00:00:00 2001
From: Mark H Weaver <mhw@netris.org>
Date: Wed, 10 Dec 2014 15:56:00 -0500
Subject: [PATCH] gnu: pcre: Update to 8.36 and enable JIT support.

* gnu/packages/pcre.scm (pcre): Update to 8.36.  Enable JIT support.
---
 gnu/packages/pcre.scm | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/pcre.scm b/gnu/packages/pcre.scm
index 54a3e22..e6c9ab8 100644
--- a/gnu/packages/pcre.scm
+++ b/gnu/packages/pcre.scm
@@ -1,5 +1,6 @@
 ;;; GNU Guix --- Functional package management for GNU
 ;;; Copyright © 2013 Andreas Enge <andreas@enge.fr>
+;;; Copyright © 2014 Mark H Weaver <mhw@netris.org>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -27,20 +28,21 @@
 (define-public pcre
   (package
    (name "pcre")
-   (version "8.32")
+   (version "8.36")
    (source (origin
             (method url-fetch)
             (uri (string-append "mirror://sourceforge/pcre/pcre/"
                                 version "/pcre-" version ".tar.bz2"))
             (sha256
              (base32
-              "0m8gvrf1q0iwll4csirvvj98xygw4cy7r14i5l53ivsqs2dzn4x9"))))
+              "1fs5p1z67m9f4xnyil3s4lhgyld78f7m4d1yawpyhh0cvrbk90zg"))))
    (build-system gnu-build-system)
    (inputs `(("bzip2" ,bzip2)
              ("readline" ,readline)
              ("zlib" ,zlib)))
    (arguments
     `(#:configure-flags '("--enable-utf"
+                          "--enable-jit"
                           "--enable-pcregrep-libz"
                           "--enable-pcregrep-libbz2"
                           "--enable-pcretest-libreadline")))
-- 
2.1.2

Re: [PATCH] gnu: pcre: Update to 8.36 and enable JIT support

[Ludovic Courtès]

Mark H Weaver <mhw@netris.org> skribis:

> From 5796cf1178fbcf41ef1c3d6406adb992d7e71dd0 Mon Sep 17 00:00:00 2001
> From: Mark H Weaver <mhw@netris.org>
> Date: Wed, 10 Dec 2014 15:56:00 -0500
> Subject: [PATCH] gnu: pcre: Update to 8.36 and enable JIT support.
>
> * gnu/packages/pcre.scm (pcre): Update to 8.36.  Enable JIT support.

OK!

Is JIT supported everywhere, or is it automatically turned off on
platforms that don’t support it?

Ludo’.

Re: [PATCH] gnu: pcre: Update to 8.36 and enable JIT support

[Mark H Weaver]

ludo@gnu.org (Ludovic Courtès) writes:

> Mark H Weaver <mhw@netris.org> skribis:
>
>> From 5796cf1178fbcf41ef1c3d6406adb992d7e71dd0 Mon Sep 17 00:00:00 2001
>> From: Mark H Weaver <mhw@netris.org>
>> Date: Wed, 10 Dec 2014 15:56:00 -0500
>> Subject: [PATCH] gnu: pcre: Update to 8.36 and enable JIT support.
>>
>> * gnu/packages/pcre.scm (pcre): Update to 8.36.  Enable JIT support.
>
> OK!
>
> Is JIT supported everywhere, or is it automatically turned off on
> platforms that don’t support it?

According to the docs, passing "--enable-jit" will cause the compilation
to fail on platforms not supported by their JIT, so we might have to
make that flag conditional in the future.  However, the build and test
suite passed on MIPS, and they support ARM as well, so I think it'll be
fine for the foreseeable future.

      Mark

Re: [PATCH] gnu: pcre: Update to 8.36 and enable JIT support

[Ludovic Courtès]

Mark H Weaver <mhw@netris.org> skribis:

> ludo@gnu.org (Ludovic Courtès) writes:
>
>> Mark H Weaver <mhw@netris.org> skribis:
>>
>>> From 5796cf1178fbcf41ef1c3d6406adb992d7e71dd0 Mon Sep 17 00:00:00 2001
>>> From: Mark H Weaver <mhw@netris.org>
>>> Date: Wed, 10 Dec 2014 15:56:00 -0500
>>> Subject: [PATCH] gnu: pcre: Update to 8.36 and enable JIT support.
>>>
>>> * gnu/packages/pcre.scm (pcre): Update to 8.36.  Enable JIT support.
>>
>> OK!
>>
>> Is JIT supported everywhere, or is it automatically turned off on
>> platforms that don’t support it?
>
> According to the docs, passing "--enable-jit" will cause the compilation
> to fail on platforms not supported by their JIT, so we might have to
> make that flag conditional in the future.  However, the build and test
> suite passed on MIPS, and they support ARM as well, so I think it'll be
> fine for the foreseeable future.

OK, perfect.

Ludo’.