diff mbox

[4/4] gnu: xapian: Update to 1.4.0.

Message ID 20161017165506.17750-5-mbakke@fastmail.com
State New
Headers show

Commit Message

Marius Bakke Oct. 17, 2016, 4:55 p.m. UTC
* gnu/packages/search.scm (xapian): Update to 1.4.0.
[source]: Use https URL.
[home-page]: Use https URL.
[arguments]: Remove 'patch-remotetcp-harness' phase. Replace 'check'
phase with a list of custom test targets.
---
 gnu/packages/search.scm | 22 +++++++++++++++-------
 1 file changed, 15 insertions(+), 7 deletions(-)

Comments

Leo Famulari Oct. 17, 2016, 7:49 p.m. UTC | #1
On Mon, Oct 17, 2016 at 05:55:06PM +0100, Marius Bakke wrote:
> * gnu/packages/search.scm (xapian): Update to 1.4.0.
> [source]: Use https URL.
> [home-page]: Use https URL.
> [arguments]: Remove 'patch-remotetcp-harness' phase. Replace 'check'
> phase with a list of custom test targets.

Works for me with mu.
Marius Bakke Oct. 18, 2016, 9:55 p.m. UTC | #2
Leo Famulari <leo@famulari.name> writes:

> On Mon, Oct 17, 2016 at 05:55:06PM +0100, Marius Bakke wrote:
>> * gnu/packages/search.scm (xapian): Update to 1.4.0.
>> [source]: Use https URL.
>> [home-page]: Use https URL.
>> [arguments]: Remove 'patch-remotetcp-harness' phase. Replace 'check'
>> phase with a list of custom test targets.
>
> Works for me with mu.

Thanks a lot for reviewing, Leo. I'll go ahead and push these tomorrow
with the mentioned changes unless there are other comments. Not happy
about excluding those Xapian tests, but don't know what to do about it.
diff mbox

Patch

diff --git a/gnu/packages/search.scm b/gnu/packages/search.scm
index a43abde..5d7def5 100644
--- a/gnu/packages/search.scm
+++ b/gnu/packages/search.scm
@@ -35,30 +35,38 @@ 
 (define-public xapian
   (package
     (name "xapian")
-    (version "1.2.21")
+    (version "1.4.0")
     (source (origin
               (method url-fetch)
-              (uri (string-append "http://oligarchy.co.uk/xapian/" version
+              (uri (string-append "https://oligarchy.co.uk/xapian/" version
                                   "/xapian-core-" version ".tar.xz"))
               (sha256
-               (base32 "0grd2s6gf8yzqwdaa50g57j9d81mxkrrpkyldm2shgyizdc8gx33"))))
+               (base32 "0xv4da5rmqqzkkkzx2v3jwh5hz5zxhd2b7m8x30fk99a25blyn0h"))))
     (build-system gnu-build-system)
     (inputs `(("zlib" ,zlib)
               ("util-linux" ,util-linux)))
     (arguments
      `(#:phases
        (modify-phases %standard-phases
-         (add-after 'unpack 'patch-remotetcp-harness
+         (replace 'check
+           ;; As of Xapian 1.3.3, the TCP server implementation uses
+           ;; getaddrinfo(). This does not work in the build environment,
+           ;; so exclude those tests. See HACKING for the list of targets.
            (lambda _
-             (substitute* "tests/harness/backendmanager_remotetcp.cc"
-               (("/bin/sh") (which "bash"))))))))
+             (zero? (system* "make"
+                             "check-inmemory"
+                             "check-remoteprog"
+                             ;"check-remotetcp"
+                             "check-multi"
+                             "check-glass"
+                             "check-chert")))))))
     (synopsis "Search Engine Library")
     (description
      "Xapian is a highly adaptable toolkit which allows developers to easily
 add advanced indexing and search facilities to their own applications.  It
 supports the Probabilistic Information Retrieval model and also supports a
 rich set of boolean query operators.")
-    (home-page "http://xapian.org/")
+    (home-page "https://xapian.org/")
     (license (list gpl2+ bsd-3 x11))))
 
 (define-public libtocc