all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [bug#58432] [PATCH] gnu: Add hiredis
@ 2022-10-11  9:19 Thomas Albers via Guix-patches via
  2022-10-11  9:50 ` ( via Guix-patches via
  2022-10-11 13:08 ` Thomas Albers via Guix-patches via
  0 siblings, 2 replies; 6+ messages in thread
From: Thomas Albers via Guix-patches via @ 2022-10-11  9:19 UTC (permalink / raw)
  To: 58432; +Cc: Thomas Albers

* gnu/packages/databases.scm (hiredis): New variable.
---
 gnu/packages/databases.scm | 26 +++++++++++++++++++++++++-
 1 file changed, 25 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/databases.scm b/gnu/packages/databases.scm
index cf07832df7..b372423a77 100644
--- a/gnu/packages/databases.scm
+++ b/gnu/packages/databases.scm
@@ -58,6 +58,7 @@
 ;;; Copyright © 2022 Zhu Zihao <all_but_last@163.com>
 ;;; Copyright © 2021 Brice Waegeneire <brice@waegenei.re>
 ;;; Copyright © 2022 muradm <mail@muradm.net>
+;;; Copyright © 2022 Thomas Albers Raviola <thomas@thomaslabs.org>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -2515,6 +2516,29 @@ (define-public ruby-redis
     (home-page "https://github.com/redis/redis-rb")
     (license license:expat)))

+(define-public hiredis
+  (package
+   (name "hiredis")
+   (version "1.0.2")
+   (source (origin
+            (method git-fetch)
+            (uri (git-reference
+                  (url "https://github.com/redis/hiredis")
+                  (commit (string-append "v" version))))
+            (file-name (git-file-name name version))
+            (sha256
+             (base32 "0a55zk3qrw9yl27i87h3brg2hskmmzbfda77dhq9a4if7y70xnfb"))))
+   (build-system cmake-build-system)
+   (native-inputs
+    ;; needed for testing
+    (list redis))
+   (synopsis "Minimalistic C client library for the Redis database")
+   (description "Library for sending commands and receiving replies to and
+from a Redis server.  It comes with a synchronous API, asynchronous API and
+reply parsing API.  Only the binary-safe Redis protocol is supported.")
+   (home-page "https://github.com/redis/hiredis")
+   (license license:bsd-3)))
+
 (define-public go-github-com-cupcake-rdb
   (package
     (name "go-github-com-cupcake-rdb")
--
2.37.3




^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [bug#58432] [PATCH] gnu: Add hiredis
  2022-10-11  9:19 [bug#58432] [PATCH] gnu: Add hiredis Thomas Albers via Guix-patches via
@ 2022-10-11  9:50 ` ( via Guix-patches via
  2022-10-11 13:10   ` Thomas Albers Raviola via Guix-patches via
  2022-10-11 13:08 ` Thomas Albers via Guix-patches via
  1 sibling, 1 reply; 6+ messages in thread
From: ( via Guix-patches via @ 2022-10-11  9:50 UTC (permalink / raw)
  To: Thomas Albers, 58432

Heya Thomas,

Looks mostly good, just a few nitpicks:

On Tue Oct 11, 2022 at 10:19 AM BST, Thomas Albers via Guix-patches via wrote:
> +(define-public hiredis
> +  (package
> +   (name "hiredis")
     ^
We usually have two spaces of indentation here.

> +   (source (origin
> +            (method git-fetch)
              ^
Here too.

> +            (sha256
> +             (base32 "0a55zk3qrw9yl27i87h3brg2hskmmzbfda77dhq9a4if7y70xnfb"))))

Put the hash string on a new line.

> +   (description "Library for sending commands and receiving replies to and
> +from a Redis server.  It comes with a synchronous API, asynchronous API and
> +reply parsing API.  Only the binary-safe Redis protocol is supported.")

"This package provides a library for ..." is probably better.

    -- (




^ permalink raw reply	[flat|nested] 6+ messages in thread

* [bug#58432] [PATCH] gnu: Add hiredis
  2022-10-11  9:19 [bug#58432] [PATCH] gnu: Add hiredis Thomas Albers via Guix-patches via
  2022-10-11  9:50 ` ( via Guix-patches via
@ 2022-10-11 13:08 ` Thomas Albers via Guix-patches via
  2022-10-11 13:34   ` ( via Guix-patches via
  2022-10-13 10:10   ` bug#58432: " Christopher Baines
  1 sibling, 2 replies; 6+ messages in thread
From: Thomas Albers via Guix-patches via @ 2022-10-11 13:08 UTC (permalink / raw)
  To: 58432; +Cc: Thomas Albers

* gnu/packages/databases.scm (hiredis): New variable.
---
 gnu/packages/databases.scm | 28 +++++++++++++++++++++++++++-
 1 file changed, 27 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/databases.scm b/gnu/packages/databases.scm
index cf07832df7..8523c084e4 100644
--- a/gnu/packages/databases.scm
+++ b/gnu/packages/databases.scm
@@ -58,6 +58,7 @@
 ;;; Copyright © 2022 Zhu Zihao <all_but_last@163.com>
 ;;; Copyright © 2021 Brice Waegeneire <brice@waegenei.re>
 ;;; Copyright © 2022 muradm <mail@muradm.net>
+;;; Copyright © 2022 Thomas Albers Raviola <thomas@thomaslabs.org>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -2494,6 +2495,31 @@ (define-public redis
     (home-page "https://redis.io/")
     (license license:bsd-3)))

+(define-public hiredis
+  (package
+    (name "hiredis")
+    (version "1.0.2")
+    (source
+     (origin
+       (method git-fetch)
+       (uri (git-reference
+             (url "https://github.com/redis/hiredis")
+             (commit (string-append "v" version))))
+       (file-name (git-file-name name version))
+       (sha256
+        (base32 "0a55zk3qrw9yl27i87h3brg2hskmmzbfda77dhq9a4if7y70xnfb"))))
+    (build-system cmake-build-system)
+    (native-inputs
+     ;; needed for testing
+     (list redis))
+    (synopsis "Minimalistic C client library for the Redis database")
+    (description "This package provides a library for sending commands and
+receiving replies to and from a Redis server.  It comes with a synchronous
+API, asynchronous API and reply parsing API.  Only the binary-safe Redis
+protocol is supported.")
+    (home-page "https://github.com/redis/hiredis")
+    (license license:bsd-3)))
+
 (define-public ruby-redis
   (package
     (name "ruby-redis")
--
2.37.3




^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [bug#58432] [PATCH] gnu: Add hiredis
  2022-10-11  9:50 ` ( via Guix-patches via
@ 2022-10-11 13:10   ` Thomas Albers Raviola via Guix-patches via
  0 siblings, 0 replies; 6+ messages in thread
From: Thomas Albers Raviola via Guix-patches via @ 2022-10-11 13:10 UTC (permalink / raw)
  To: (; +Cc: 58432

Thanks ( for the observations. I have sent a new patch with the
corrections.




^ permalink raw reply	[flat|nested] 6+ messages in thread

* [bug#58432] [PATCH] gnu: Add hiredis
  2022-10-11 13:08 ` Thomas Albers via Guix-patches via
@ 2022-10-11 13:34   ` ( via Guix-patches via
  2022-10-13 10:10   ` bug#58432: " Christopher Baines
  1 sibling, 0 replies; 6+ messages in thread
From: ( via Guix-patches via @ 2022-10-11 13:34 UTC (permalink / raw)
  To: Thomas Albers, 58432

LGTM :)

    -- (




^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#58432: [PATCH] gnu: Add hiredis
  2022-10-11 13:08 ` Thomas Albers via Guix-patches via
  2022-10-11 13:34   ` ( via Guix-patches via
@ 2022-10-13 10:10   ` Christopher Baines
  1 sibling, 0 replies; 6+ messages in thread
From: Christopher Baines @ 2022-10-13 10:10 UTC (permalink / raw)
  To: Thomas Albers; +Cc: 58432-done

[-- Attachment #1: Type: text/plain, Size: 338 bytes --]


Thomas Albers via Guix-patches via <guix-patches@gnu.org> writes:

> * gnu/packages/databases.scm (hiredis): New variable.
> ---
>  gnu/packages/databases.scm | 28 +++++++++++++++++++++++++++-
>  1 file changed, 27 insertions(+), 1 deletion(-)

Thanks both, I've pushed this to master as
cc58a0c62739d93c1d22a9930f11abdb0bd7ac04.

Chris

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 987 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2022-10-13 10:52 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-10-11  9:19 [bug#58432] [PATCH] gnu: Add hiredis Thomas Albers via Guix-patches via
2022-10-11  9:50 ` ( via Guix-patches via
2022-10-11 13:10   ` Thomas Albers Raviola via Guix-patches via
2022-10-11 13:08 ` Thomas Albers via Guix-patches via
2022-10-11 13:34   ` ( via Guix-patches via
2022-10-13 10:10   ` bug#58432: " Christopher Baines

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.