01/01: gnu: Add guile-curl.

[Mark H Weaver]

Hi Roel,

roelj pushed a commit to branch master
in repository guix.
commit 5e3010a2ac651397e0cb69239a7d7aa3c0a5703e
Date: Wed Apr 18 23:00:41 2018 +0200
gnu: Add guile-curl.
* gnu/packages/curl.scm (guile-curl): New variable.

[...]

+ (modify-phases %standard-phases
+ (add-after 'install 'patch-extension-path
+ (lambda* (#:key outputs #:allow-other-keys)
+ (let* ((out (assoc-ref outputs "out"))
+ (curl.scm (string-append
+ out "/share/guile/site/2.2/curl.scm"))
+ (curl.go (string-append
+ out "/lib/guile/2.2/site-ccache/curl.go"))
+ (ext (string-append out "/lib/guile/2.2/"
+ "extensions/libguile-curl")))
+ (substitute* curl.scm (("libguile-curl") ext))
+ ;; The build system does not actually compile the Scheme module.
+ ;; So we can compile it and put it in the right place in one go.
+ (system* "guild" "compile" curl.scm "-o" curl.go))
+ #t)))))

Please use 'invoke' instead of 'system*' from now on, so that errors in
the subprocess will be detected and reported using exceptions. As you
have it now, compile failures will be ignored.

Would you like to push a fix?

Mark

[Roel Janssen]

Post by Mark H Weaver
Hi Roel,

roelj pushed a commit to branch master
in repository guix.
commit 5e3010a2ac651397e0cb69239a7d7aa3c0a5703e
Date: Wed Apr 18 23:00:41 2018 +0200
gnu: Add guile-curl.
* gnu/packages/curl.scm (guile-curl): New variable.

[...]

+ (modify-phases %standard-phases
+ (add-after 'install 'patch-extension-path
+ (lambda* (#:key outputs #:allow-other-keys)
+ (let* ((out (assoc-ref outputs "out"))
+ (curl.scm (string-append
+ out "/share/guile/site/2.2/curl.scm"))
+ (curl.go (string-append
+ out "/lib/guile/2.2/site-ccache/curl.go"))
+ (ext (string-append out "/lib/guile/2.2/"
+ "extensions/libguile-curl")))
+ (substitute* curl.scm (("libguile-curl") ext))
+ ;; The build system does not actually compile the Scheme module.
+ ;; So we can compile it and put it in the right place in one go.
+ (system* "guild" "compile" curl.scm "-o" curl.go))
+ #t)))))

Please use 'invoke' instead of 'system*' from now on, so that errors in
the subprocess will be detected and reported using exceptions. As you
have it now, compile failures will be ignored.

Whoops. I will try to clear my brain's internal cache.
Thanks for letting me know.

Post by Mark H Weaver
Would you like to push a fix?

I pushed a fix in d28e5ad23.

Kind regards,
Roel Janssen