bug#74800: guix package lacks offloading capabilities (guile-ssh@0.18)

bug#74800: guix package lacks offloading capabilities (guile-ssh@0.18)

[Dariqq]

Hello,

I was trying the patch for https://issues.guix.gnu.org/74787 and was 
wondering why it didnt work for me.

The default guix package is currently built without the offloading/ssh 
support:

from the configure script:

checking whether Guile-SSH is available and recent enough... no


E.g. this recent build on ci.g.g.o: 
https://ci.guix.gnu.org/build/6971850/details

bug#74800: guix package lacks offloading capabilities (guile-ssh@0.18)

[Dariqq]

Running what the GUIX_CHECK_GUILE_SSH macro expands to with 
guile-ssh@0.18 yields


--8<---------------cut here---------------start------------->8---
scheme@(guile-user)> (and (@ (ssh channel) channel-send-eof)
             (@ (ssh popen) open-remote-pipe)
             (@ (ssh dist node) node-eval)
             (@ (ssh auth) userauth-gssapi!)
             ((@ (ssh session) make-session) #:nodelay #t))
ice-9/boot-9.scm:1685:16: In procedure raise-exception:
Throw to key `guile-ssh-error' with args `("'config' is specified, but 
'host' option is missed.")'.
--8<---------------cut here---------------end--------------->8---


On 11.12.24 21:38, Dariqq wrote:
> Hello,
> 
> I was trying the patch for https://issues.guix.gnu.org/74787 and was 
> wondering why it didnt work for me.
> 
> The default guix package is currently built without the offloading/ssh 
> support:
> 
> from the configure script:
> 
> checking whether Guile-SSH is available and recent enough... no
> 
> 
> E.g. this recent build on ci.g.g.o: https://ci.guix.gnu.org/ 
> build/6971850/details

[bug#74802] [PATCH] m4: guix: Fix detection of guile-ssh.

[Tomas Volf]

The check for make-session stopped working after update to 0.18.0.  This
commit adds the additional `#:config #f' to skip reading the ssh
configuration, arguably we did not want to do that anyway.

* m4/guix.m4 (GUIX_CHECK_GUILE_SSH): Add `#:config #f' to make-session call.

Change-Id: Id6ea3860292159ac2e6cf2a77df5f720f66aa071
---
 m4/guix.m4 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/m4/guix.m4 b/m4/guix.m4
index 8c2757a8b7..376f40943c 100644
--- a/m4/guix.m4
+++ b/m4/guix.m4
@@ -143,7 +143,7 @@ AC_DEFUN([GUIX_CHECK_GUILE_SSH], [
             (@ (ssh popen) open-remote-pipe)
             (@ (ssh dist node) node-eval)
             (@ (ssh auth) userauth-gssapi!)
-            ((@ (ssh session) make-session) #:nodelay #t))])
+            ((@ (ssh session) make-session) #:nodelay #t #:config #f))])
      if test "$retval" = 0; then
        guix_cv_have_recent_guile_ssh="yes"
      else
-- 
2.46.0

bug#74800: guix package lacks offloading capabilities (guile-ssh@0.18)

[Ludovic Courtès]

Tomas Volf <~@wolfsden.cz> skribis:

> The check for make-session stopped working after update to 0.18.0.  This
> commit adds the additional `#:config #f' to skip reading the ssh
> configuration, arguably we did not want to do that anyway.
>
> * m4/guix.m4 (GUIX_CHECK_GUILE_SSH): Add `#:config #f' to make-session call.
>
> Change-Id: Id6ea3860292159ac2e6cf2a77df5f720f66aa071

Woow, good catch.  Applied, thanks!

This Guile-SSH upgrade turned out to be trickier than expected.  :-)

Ludo’.