On 01/30/2014 11:39 PM, Andreas Enge wrote:
> On Thu, Jan 30, 2014 at 05:36:03PM +0100, Sree Harsha Totakura wrote:
>> > Here is the much awaited recipe for GNUnet. The patches follow this email.
> excellent! I also worked on gnunet and think my handling of phases is
> a bit simpler and clearer:
>
> (arguments
> `(#:phases
> ;; swap check and install phases and set paths to installed binaries
> (alist-cons-before
> 'check 'set-path-for-check
> (lambda* (#:key outputs #:allow-other-keys)
> (let ((out (assoc-ref outputs "out")))
> (setenv "GNUNET_PREFIX" out)
> (setenv "PATH" (string-append (getenv "PATH") ":" out "/bin"))))
> (alist-cons-after
> 'install 'check
> (assoc-ref %standard-phases 'check)
> (alist-delete
> 'check
> %standard-phases)))))
>
Yes, this looks better, perhaps Ludo can replace the relevant part with
this.
> (where the 'set-path-for-check phase will be made obsolete by the recent
> gnunet modification I suggested in its bug tracker).
Yes, this will be made obsolete.
>
> We have slightly different inputs; what is opus needed for? Apart from that,
We now have a new voice-chat service called conversation. Opus is the
audio codec it requires, hence the new dependency.
> I have additional inputs:
> ("libmicrohttpd" ,libmicrohttpd)
> ("openssl" ,openssl) ; needed for some tests creating TLS certificates
>
> Maybe openssl is a remainder from an earlier gnunet version and not needed/
> used anymore?
The README mentions openssl as a requirement; I forgot to add it.
Perhaps the build didn't build relevant parts or testcases.
I shall try adding the openssl dependency and see how it goes.
Sree
