On Mon, Oct 17, 2016 at 05:55:05PM +0100, Marius Bakke wrote:
> * gnu/packages/search.scm (xapian)[arguments]: Use 'modify-phases'.
> + `(#:phases
> + (modify-phases %standard-phases
> + (add-after 'unpack 'patch-remotetcp-harness
> + (lambda _
> + (substitute* "tests/harness/backendmanager_remotetcp.cc"
> + (("/bin/sh") (which "bash"))))))))
Please change it to use (which "sh") instead. Bash changes its behavior
based on whether it is called as `bash` or `sh` (see Invocation in
bash(1)).
Otherwise LGTM.