[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#29966] [PATCH] Update guile-wiredtiger to 0.6.3
From: |
Kei Kebreau |
Subject: |
[bug#29966] [PATCH] Update guile-wiredtiger to 0.6.3 |
Date: |
Mon, 08 Jan 2018 12:47:58 -0500 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.3 (gnu/linux) |
Amirouche <address@hidden> writes:
> The latest release was meant to make it possible to run the tests in
> guix. So I also changed a bit the package.
>
>>From 68a3d58775a40956a17880a7e2eafad08b352d42 Mon Sep 17 00:00:00 2001
> From: amirouche <address@hidden>
> Date: Wed, 3 Jan 2018 20:43:52 +0100
> Subject: [PATCH] gnu: packages: guile-wiredtiger: Update to latest release
>
> * gnu/package/databases.scm (guile-wiredtiger): bump to new release,
> enable tests, disable parallel tests and htmlprag was removed from
> upstream release so there not need to delete it anymore. Improve
> the synopsis and description.
I've cleaned this commit message up a bit, like so:
gnu: guile-wiredtiger: Update to 0.6.3.
* gnu/package/databases.scm (guile-wiredtiger): Update to 0.6.3.
[arguments]: Enable tests; disable parallel tests; remove
'remove-bundled-dependencies' phase.
[synopsis, description]: Improve them.
> ---
> gnu/packages/databases.scm | 25 ++++++++++---------------
> 1 file changed, 10 insertions(+), 15 deletions(-)
>
> diff --git a/gnu/packages/databases.scm b/gnu/packages/databases.scm
> index 6306846a8..ef9c1b4ce 100644
> --- a/gnu/packages/databases.scm
> +++ b/gnu/packages/databases.scm
> @@ -30,6 +30,7 @@
> ;;; Copyright © 2017 Pierre Langlois <address@hidden>
> ;;; Copyright © 2015, 2017 Ricardo Wurmus <address@hidden>
> ;;; Copyright © 2017 Kristofer Buffington <address@hidden>
> +;;; Copyright © 2018 Amirouche Boubekki <address@hidden>
> ;;;
> ;;; This file is part of GNU Guix.
> ;;;
> @@ -1549,19 +1550,19 @@ trees (LSM), for sustained throughput under random
> insert workloads.")
> (define-public guile-wiredtiger
> (package
> (name "guile-wiredtiger")
> - (version "20171113.6cbc51da")
> + (version "0.6.3")
> (source (origin
> (method git-fetch)
> (uri (git-reference
> (url
> "https://framagit.org/a-guile-mind/guile-wiredtiger.git")
> - (commit "6cbc51dab95d28fe31ae025fbdd88f3ecbf2111b")))
> + (commit "070ed68139d99c279f058a6c293f00292d35dbd7")))
> (file-name (string-append name "-" version "-checkout"))
> (sha256
> (base32
> - "0x3qwpgch5pg0k21kc792h4y6b36a8xd1zkfq8ar2l2mqmpzkzyd"))))
> + "14rna97wsylajzxfif95wnblq85csgcfc666gh5dl0ssgd7x8llh"))))
> (build-system gnu-build-system)
> (arguments
> - '(#:tests? #f
> + '(#:parallel-tests? #f ;; tests can't be run in parallel, yet.
> #:configure-flags
> (list (string-append "--with-libwiredtiger-prefix="
> (assoc-ref %build-inputs "wiredtiger")))
> @@ -1570,15 +1571,9 @@ trees (LSM), for sustained throughput under random
> insert workloads.")
> (modify-phases %standard-phases
> (add-after 'unpack 'bootstrap
> (lambda _
> - (zero? (system* "sh" "bootstrap"))))
> - (add-before 'bootstrap 'remove-bundled-dependencies
> - (lambda _
> - ;; TODO: Remove microkanren.scm when we have a separate package
> - ;; for it.
> - (delete-file "htmlprag.scm")
> - (substitute* "Makefile.am"
> - (("htmlprag\\.scm") ""))
> - #t)))))
> + (zero? (system* "sh" "bootstrap")))))))
I've changed the (zero? (system* ...)) call to (invoke ...), a macro
available in (guix build utils) that does the same thing. AFAIK, our
codebase will be moving towards using 'invoke' for this purpose in the
future.
> + ;; TODO: Remove microkanren.scm when we have a separate package
> + ;; for it.
> (native-inputs
> `(("autoconf" ,autoconf)
> ("automake" ,automake)
> @@ -1588,10 +1583,10 @@ trees (LSM), for sustained throughput under random
> insert workloads.")
> ("guile" ,guile-2.2)))
> (propagated-inputs
> `(("guile-lib" ,guile-lib))) ;for (htmlprag)
> - (synopsis "Wired Tiger bindings for GNU Guile")
> + (synopsis "WiredTiger bindings for GNU Guile")
> (description
> "This package provides Guile bindings to the WiredTiger ``NoSQL''
> -database.")
> +database. Various higher level database abstractions.")
> (home-page "https://framagit.org/a-guile-mind/guile-wiredtiger")
> (license license:gpl3+)))
Otherwise, this LGTM! Pushed to master with the above changes. Thanks
for updating this!
signature.asc
Description: PGP signature