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

2016-10-18 Thread Marius Bakke 
Leo Famulari  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.

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

2016-10-17 Thread Leo Famulari 
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.

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

2016-10-17 Thread Marius Bakke 
* 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(-)

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
-- 
2.10.1