unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#45977] [PATCH 0/1] gnu: guile-redis: Update to 2.0.0.
@ 2021-01-18 22:39 BonfaceKilz
  2021-01-18 22:39 ` [bug#45976] [PATCH 1/1] " BonfaceKilz
                   ` (2 more replies)
  0 siblings, 3 replies; 9+ messages in thread
From: BonfaceKilz @ 2021-01-18 22:39 UTC (permalink / raw)
  To: 45977; +Cc: BonfaceKilz

Hi! Find attached an update to guile-redis that adds proper support for Redis
Pub/Sub.

BonfaceKilz (1):
  gnu: guile-redis: Update to 2.0.0.

 gnu/packages/guile-xyz.scm | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

-- 
2.29.2





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

* [bug#45976] [PATCH 1/1] gnu: guile-redis: Update to 2.0.0.
  2021-01-18 22:39 [bug#45977] [PATCH 0/1] gnu: guile-redis: Update to 2.0.0 BonfaceKilz
@ 2021-01-18 22:39 ` BonfaceKilz
  2021-01-19 19:26 ` [bug#45977] [PATCH 0/1] " Efraim Flashner
  2021-01-26 10:42 ` [bug#45977] [PATCH v2 0/2] " BonfaceKilz
  2 siblings, 0 replies; 9+ messages in thread
From: BonfaceKilz @ 2021-01-18 22:39 UTC (permalink / raw)
  To: 45976; +Cc: BonfaceKilz

* gnu/packages/guile-xyz.scm (guile-redis): Update to 2.0.0.
---
 gnu/packages/guile-xyz.scm | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/guile-xyz.scm b/gnu/packages/guile-xyz.scm
index 9775cc33f0..50fee4cdd0 100644
--- a/gnu/packages/guile-xyz.scm
+++ b/gnu/packages/guile-xyz.scm
@@ -33,6 +33,7 @@
 ;;; Copyright © 2020 Mike Rosset <mike.rosset@gmail.com>
 ;;; Copyright © 2020 Leo Prikler <leo.prikler@student.tugraz.at>
 ;;; Copyright © 2020 pukkamustard <pukkamustard@posteo.net>
+;;; Copyright © 2021 Bonface Munyoki Kilyungi <me@bonfacemunyoki.com>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -2428,7 +2429,7 @@ interface for reading articles in any format.")
 (define-public guile-redis
   (package
     (name "guile-redis")
-    (version "1.3.0")
+    (version "2.0.0")
     (home-page "https://github.com/aconchillo/guile-redis")
     (source (origin
               (method git-fetch)
@@ -2438,7 +2439,7 @@ interface for reading articles in any format.")
               (file-name (git-file-name name version))
               (sha256
                (base32
-                "14izs8daxh7pb7vwpxi5g427qa31137jkaxrb1cy5rpjkwchy723"))))
+                "1zk2x37lw6ygf7rwy71svnsian8lj51axpxmm66ah7dazn69swlm"))))
     (build-system gnu-build-system)
     (native-inputs
      `(("autoconf" ,autoconf)
-- 
2.29.2





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

* [bug#45977] [PATCH 0/1] gnu: guile-redis: Update to 2.0.0.
  2021-01-18 22:39 [bug#45977] [PATCH 0/1] gnu: guile-redis: Update to 2.0.0 BonfaceKilz
  2021-01-18 22:39 ` [bug#45976] [PATCH 1/1] " BonfaceKilz
@ 2021-01-19 19:26 ` Efraim Flashner
  2021-01-20  6:47   ` Bonface Munyoki K.
  2021-01-26 10:42 ` [bug#45977] [PATCH v2 0/2] " BonfaceKilz
  2 siblings, 1 reply; 9+ messages in thread
From: Efraim Flashner @ 2021-01-19 19:26 UTC (permalink / raw)
  To: BonfaceKilz; +Cc: 45977

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

On Tue, Jan 19, 2021 at 01:39:25AM +0300, BonfaceKilz wrote:
> Hi! Find attached an update to guile-redis that adds proper support for Redis
> Pub/Sub.
> 
> BonfaceKilz (1):
>   gnu: guile-redis: Update to 2.0.0.
> 
>  gnu/packages/guile-xyz.scm | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 

This update breaks guile2.0-redis. The configure script still checks for
2.0 so I'm not sure if that's intentional. Can you look into it?

Alternatively, I'm not seeing guile2.0-redis used by any packages in
guix, but I'm not sure about just dropping it.

-- 
Efraim Flashner   <efraim@flashner.co.il>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

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

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

* [bug#45977] [PATCH 0/1] gnu: guile-redis: Update to 2.0.0.
  2021-01-19 19:26 ` [bug#45977] [PATCH 0/1] " Efraim Flashner
@ 2021-01-20  6:47   ` Bonface Munyoki K.
  0 siblings, 0 replies; 9+ messages in thread
From: Bonface Munyoki K. @ 2021-01-20  6:47 UTC (permalink / raw)
  To: Efraim Flashner; +Cc: 45977

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

Efraim Flashner <efraim@flashner.co.il> writes:

> On Tue, Jan 19, 2021 at 01:39:25AM +0300, BonfaceKilz wrote:
>> Hi! Find attached an update to guile-redis that adds proper support for Redis
>> Pub/Sub.
>> 
>> BonfaceKilz (1):
>>   gnu: guile-redis: Update to 2.0.0.
>> 
>>  gnu/packages/guile-xyz.scm | 5 +++--
>>  1 file changed, 3 insertions(+), 2 deletions(-)
>> 
>
> This update breaks guile2.0-redis. The configure script still checks for
> 2.0 so I'm not sure if that's intentional. Can you look into it?
>

Yep. Let me dig into it.

> Alternatively, I'm not seeing guile2.0-redis used by any packages in
> guix, but I'm not sure about just dropping it.

Let's not drop it just yet...
-- 
Bonface M. K. D4F09EB110177E03C28E2FE1F5BBAE1E0392253F
Humble GNU Emacs User / Bearer of scheme-y parens
Curator: <https://upbookclub.com> / Twitter: @BonfaceKilz

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

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

* [bug#45977] [PATCH v2 0/2] gnu: guile-redis: Update to 2.0.0.
  2021-01-18 22:39 [bug#45977] [PATCH 0/1] gnu: guile-redis: Update to 2.0.0 BonfaceKilz
  2021-01-18 22:39 ` [bug#45976] [PATCH 1/1] " BonfaceKilz
  2021-01-19 19:26 ` [bug#45977] [PATCH 0/1] " Efraim Flashner
@ 2021-01-26 10:42 ` BonfaceKilz
  2021-01-26 10:42   ` [bug#45977] [PATCH v2 1/2] " BonfaceKilz
                     ` (2 more replies)
  2 siblings, 3 replies; 9+ messages in thread
From: BonfaceKilz @ 2021-01-26 10:42 UTC (permalink / raw)
  To: 45977; +Cc: BonfaceKilz

Hi! For this patch, I've downgraded guile2.0-redis back to 1.3.0. The problem
with with trying to build v2.0.0 on guile2.0 is that libs like (ice-9
textual-ports) were not yet present; but are used in v2.0 of
guile-redis. Here's what I have when I get into a guile2.0 repl:

#+begin_src 
➜  guix git:(master) ✗ guix environment -C --ad-hoc guile@2.0.14 -- guile
The following derivation will be built:
   /gnu/store/f6pxq82f7ifzxmlaba72x0mn9rqfdf8b-profile.drv
The following profile hooks will be built:
   /gnu/store/3ghqadnzhddgcf8nljfzjrkyzvizn61m-manual-database.drv
   /gnu/store/4p7wnbk6bh9g6q1wlb48zmf09rlnwdxl-info-dir.drv
   /gnu/store/4q08ispks919baxrvp5g7wyipn1n98lc-fonts-dir.drv
   /gnu/store/f15hk17knzla3wzhmfyx61lwih1g5mjq-ca-certificate-bundle.drv
building CA certificate bundle...
building fonts directory...
building directory of Info manuals...
building database for manual pages...
building profile with 1 package...
GNU Guile 2.0.14
Copyright (C) 1995-2016 Free Software Foundation, Inc.

Guile comes with ABSOLUTELY NO WARRANTY; for details type `,show w'.
This program is free software, and you are welcome to redistribute it
under certain conditions; type `,show c' for details.

Enter `,help' for help.
scheme@(guile-user)> ,use (ice-9 textual-ports)
While executing meta-command:
ERROR: no code for module (ice-9 textual-ports)
scheme@(guile-user)> 
#+end_src

In this patch set, I've downgraded the guile2.0-redis version to 1.3.0.

BonfaceKilz (2):
  gnu: guile-redis: Update to 2.0.0.
  gnu: guile2.0-redis: Revert to 1.3.0.

 gnu/packages/guile-xyz.scm | 15 +++++++++++++--
 1 file changed, 13 insertions(+), 2 deletions(-)

-- 
2.29.2





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

* [bug#45977] [PATCH v2 1/2] gnu: guile-redis: Update to 2.0.0.
  2021-01-26 10:42 ` [bug#45977] [PATCH v2 0/2] " BonfaceKilz
@ 2021-01-26 10:42   ` BonfaceKilz
  2021-01-26 10:42   ` [bug#45977] [PATCH v2 2/2] gnu: guile2.0-redis: Revert to 1.3.0 BonfaceKilz
  2021-01-27  9:21   ` bug#45977: [PATCH v2 0/2] gnu: guile-redis: Update to 2.0.0 Efraim Flashner
  2 siblings, 0 replies; 9+ messages in thread
From: BonfaceKilz @ 2021-01-26 10:42 UTC (permalink / raw)
  To: 45977; +Cc: BonfaceKilz

* gnu/packages/guile-xyz.scm (guile-redis): Update to 2.0.0.
---
 gnu/packages/guile-xyz.scm | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/guile-xyz.scm b/gnu/packages/guile-xyz.scm
index c2d853f006..ac260c804b 100644
--- a/gnu/packages/guile-xyz.scm
+++ b/gnu/packages/guile-xyz.scm
@@ -33,6 +33,7 @@
 ;;; Copyright © 2020 Mike Rosset <mike.rosset@gmail.com>
 ;;; Copyright © 2020 Leo Prikler <leo.prikler@student.tugraz.at>
 ;;; Copyright © 2020 pukkamustard <pukkamustard@posteo.net>
+;;; Copyright © 2021 Bonface Munyoki Kilyungi <me@bonfacemunyoki.com>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -2430,7 +2431,7 @@ interface for reading articles in any format.")
 (define-public guile-redis
   (package
     (name "guile-redis")
-    (version "1.3.0")
+    (version "2.0.0")
     (home-page "https://github.com/aconchillo/guile-redis")
     (source (origin
               (method git-fetch)
@@ -2440,7 +2441,7 @@ interface for reading articles in any format.")
               (file-name (git-file-name name version))
               (sha256
                (base32
-                "14izs8daxh7pb7vwpxi5g427qa31137jkaxrb1cy5rpjkwchy723"))))
+                "1zk2x37lw6ygf7rwy71svnsian8lj51axpxmm66ah7dazn69swlm"))))
     (build-system gnu-build-system)
     (native-inputs
      `(("autoconf" ,autoconf)
-- 
2.29.2





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

* [bug#45977] [PATCH v2 2/2] gnu: guile2.0-redis: Revert to 1.3.0.
  2021-01-26 10:42 ` [bug#45977] [PATCH v2 0/2] " BonfaceKilz
  2021-01-26 10:42   ` [bug#45977] [PATCH v2 1/2] " BonfaceKilz
@ 2021-01-26 10:42   ` BonfaceKilz
  2021-01-27  9:21   ` bug#45977: [PATCH v2 0/2] gnu: guile-redis: Update to 2.0.0 Efraim Flashner
  2 siblings, 0 replies; 9+ messages in thread
From: BonfaceKilz @ 2021-01-26 10:42 UTC (permalink / raw)
  To: 45977; +Cc: BonfaceKilz

`(ice-9 textual-ports)` is not present in guile 2.0.x thereby breaking the
build.
---
 gnu/packages/guile-xyz.scm | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/gnu/packages/guile-xyz.scm b/gnu/packages/guile-xyz.scm
index ac260c804b..55551d3b15 100644
--- a/gnu/packages/guile-xyz.scm
+++ b/gnu/packages/guile-xyz.scm
@@ -2464,7 +2464,17 @@ key-value cache and store.")
 (define-public guile2.0-redis
   (package
     (inherit guile-redis)
+    (version "1.3.0")
     (name "guile2.0-redis")
+    (source (origin
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://github.com/aconchillo/guile-redis")
+                    (commit version)))
+              (file-name (git-file-name name version))
+              (sha256
+               (base32
+                "14izs8daxh7pb7vwpxi5g427qa31137jkaxrb1cy5rpjkwchy723"))))
     (native-inputs `(("guile" ,guile-2.0)
                      ,@(alist-delete "guile"
                                      (package-native-inputs guile-redis))))))
-- 
2.29.2





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

* bug#45977: [PATCH v2 0/2] gnu: guile-redis: Update to 2.0.0.
  2021-01-26 10:42 ` [bug#45977] [PATCH v2 0/2] " BonfaceKilz
  2021-01-26 10:42   ` [bug#45977] [PATCH v2 1/2] " BonfaceKilz
  2021-01-26 10:42   ` [bug#45977] [PATCH v2 2/2] gnu: guile2.0-redis: Revert to 1.3.0 BonfaceKilz
@ 2021-01-27  9:21   ` Efraim Flashner
  2021-01-27  9:40     ` [bug#45977] " Bonface Munyoki K.
  2 siblings, 1 reply; 9+ messages in thread
From: Efraim Flashner @ 2021-01-27  9:21 UTC (permalink / raw)
  To: BonfaceKilz; +Cc: 45977-done

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

On Tue, Jan 26, 2021 at 01:42:31PM +0300, BonfaceKilz wrote:
> Hi! For this patch, I've downgraded guile2.0-redis back to 1.3.0. The problem
> with with trying to build v2.0.0 on guile2.0 is that libs like (ice-9
> textual-ports) were not yet present; but are used in v2.0 of
> guile-redis. Here's what I have when I get into a guile2.0 repl:

I dug into it a bit more and it looks like (redis utils) imports (ice-9
textual-imports) only for the put-string function. If I remove the
import then guile2.0-redis compiles just fine, but I wasn't sure if
there was a good test for it.

(ins)efraim@3900XT ~$ guix environment --ad-hoc guile@2.0 -- guile
;;; WARNING: loading compiled file /home/efraim/.guix-profile/lib/guile/3.0/site-ccache/ice-9/readline.go failed:
;;; ERROR: In procedure make_objcode_from_file: bad header on object file: "\x7fELF\x02\x01\x01�\x00\x00\x00\x00\x00\x00\x00\x00"
;;; WARNING: loading compiled file /run/current-system/profile/lib/guile/3.0/site-ccache/ice-9/readline.go failed:
;;; ERROR: In procedure make_objcode_from_file: bad header on object file: "\x7fELF\x02\x01\x01�\x00\x00\x00\x00\x00\x00\x00\x00"
Consider installing the 'guile-readline' package for
convenient interactive line editing and input history.

;;; WARNING: loading compiled file /home/efraim/.guix-profile/lib/guile/3.0/site-ccache/ice-9/colorized.go failed:
;;; ERROR: In procedure make_objcode_from_file: bad header on object file: "\x7fELF\x02\x01\x01�\x00\x00\x00\x00\x00\x00\x00\x00"
;;; WARNING: loading compiled file /run/current-system/profile/lib/guile/3.0/site-ccache/ice-9/colorized.go failed:
;;; ERROR: In procedure make_objcode_from_file: bad header on object file: "\x7fELF\x02\x01\x01�\x00\x00\x00\x00\x00\x00\x00\x00"
GNU Guile 2.0.14
Copyright (C) 1995-2016 Free Software Foundation, Inc.

Guile comes with ABSOLUTELY NO WARRANTY; for details type `,show w'.
This program is free software, and you are welcome to redistribute it
under certain conditions; type `,show c' for details.

Enter `,help' for help.
scheme@(guile-user)> (put-string "abcde" %current-output-port)
;;; <stdin>:1:0: warning: possibly unbound variable `put-string'
;;; <stdin>:1:0: warning: possibly unbound variable `%current-output-port'
<unnamed port>:1:0: In procedure #<procedure 7ff4222f48e0 at <current input>:1:0 ()>:
<unnamed port>:1:0: In procedure module-lookup: Unbound variable: put-string

Entering a new prompt.  Type `,bt' for a backtrace or `,q' to continue.
scheme@(guile-user) [1]> ,q
scheme@(guile-user)> (use-modules (rnrs io ports))
scheme@(guile-user)> (put-string "abcde" %current-output-port)
;;; <stdin>:4:0: warning: possibly unbound variable `%current-output-port'
<unnamed port>:4:12: In procedure #<procedure 7ff422266160 at <current input>:4:0 ()>:
<unnamed port>:4:12: In procedure module-lookup: Unbound variable: %current-output-port

Entering a new prompt.  Type `,bt' for a backtrace or `,q' to continue.

Ignoring the errors about %current-output-port, in guile2.0 put-string
is in (rnrs io ports), so it looks like it's safe to substitute (ice-9
textual-ports) for (rnrs io ports) in guile2.0-redis. So I made that
change and dropped your second patch. If someone mentions that
guile2.0-redis@2.0.0 is broken we can revisit :)

> #+begin_src 
> ➜  guix git:(master) ✗ guix environment -C --ad-hoc guile@2.0.14 -- guile
> The following derivation will be built:
>    /gnu/store/f6pxq82f7ifzxmlaba72x0mn9rqfdf8b-profile.drv
> The following profile hooks will be built:
>    /gnu/store/3ghqadnzhddgcf8nljfzjrkyzvizn61m-manual-database.drv
>    /gnu/store/4p7wnbk6bh9g6q1wlb48zmf09rlnwdxl-info-dir.drv
>    /gnu/store/4q08ispks919baxrvp5g7wyipn1n98lc-fonts-dir.drv
>    /gnu/store/f15hk17knzla3wzhmfyx61lwih1g5mjq-ca-certificate-bundle.drv
> building CA certificate bundle...
> building fonts directory...
> building directory of Info manuals...
> building database for manual pages...
> building profile with 1 package...
> GNU Guile 2.0.14
> Copyright (C) 1995-2016 Free Software Foundation, Inc.
> 
> Guile comes with ABSOLUTELY NO WARRANTY; for details type `,show w'.
> This program is free software, and you are welcome to redistribute it
> under certain conditions; type `,show c' for details.
> 
> Enter `,help' for help.
> scheme@(guile-user)> ,use (ice-9 textual-ports)
> While executing meta-command:
> ERROR: no code for module (ice-9 textual-ports)
> scheme@(guile-user)> 
> #+end_src
> 
> In this patch set, I've downgraded the guile2.0-redis version to 1.3.0.
> 
> BonfaceKilz (2):
>   gnu: guile-redis: Update to 2.0.0.
>   gnu: guile2.0-redis: Revert to 1.3.0.
> 
>  gnu/packages/guile-xyz.scm | 15 +++++++++++++--
>  1 file changed, 13 insertions(+), 2 deletions(-)
> 
> -- 
> 2.29.2
> 

-- 
Efraim Flashner   <efraim@flashner.co.il>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

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

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

* [bug#45977] [PATCH v2 0/2] gnu: guile-redis: Update to 2.0.0.
  2021-01-27  9:21   ` bug#45977: [PATCH v2 0/2] gnu: guile-redis: Update to 2.0.0 Efraim Flashner
@ 2021-01-27  9:40     ` Bonface Munyoki K.
  0 siblings, 0 replies; 9+ messages in thread
From: Bonface Munyoki K. @ 2021-01-27  9:40 UTC (permalink / raw)
  To: Efraim Flashner; +Cc: 45977-done

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

Efraim Flashner <efraim@flashner.co.il> writes:

> On Tue, Jan 26, 2021 at 01:42:31PM +0300, BonfaceKilz wrote:
>> Hi! For this patch, I've downgraded
>> guile2.0-redis back to 1.3.0. The problem
>> with with trying to build v2.0.0 on guile2.0 is
>> that libs like (ice-9
>> textual-ports) were not yet present; but are used in v2.0 of
>> guile-redis. Here's what I have when I get into a guile2.0 repl:
>
> I dug into it a bit more and it looks like (redis
> utils) imports (ice-9
> textual-imports) only for the put-string function. If I remove the
> import then guile2.0-redis compiles just fine, but I wasn't sure if
> there was a good test for it.

Thanks!

[...]

-- 
Bonface M. K. D4F09EB110177E03C28E2FE1F5BBAE1E0392253F
Humble GNU Emacs User / Bearer of scheme-y parens
Curator: <https://upbookclub.com> / Twitter: @BonfaceKilz

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

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

end of thread, other threads:[~2021-01-27  9:41 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-18 22:39 [bug#45977] [PATCH 0/1] gnu: guile-redis: Update to 2.0.0 BonfaceKilz
2021-01-18 22:39 ` [bug#45976] [PATCH 1/1] " BonfaceKilz
2021-01-19 19:26 ` [bug#45977] [PATCH 0/1] " Efraim Flashner
2021-01-20  6:47   ` Bonface Munyoki K.
2021-01-26 10:42 ` [bug#45977] [PATCH v2 0/2] " BonfaceKilz
2021-01-26 10:42   ` [bug#45977] [PATCH v2 1/2] " BonfaceKilz
2021-01-26 10:42   ` [bug#45977] [PATCH v2 2/2] gnu: guile2.0-redis: Revert to 1.3.0 BonfaceKilz
2021-01-27  9:21   ` bug#45977: [PATCH v2 0/2] gnu: guile-redis: Update to 2.0.0 Efraim Flashner
2021-01-27  9:40     ` [bug#45977] " Bonface Munyoki K.

Code repositories for project(s) associated with this public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).