[bug#41735] [PATCH 0/2] Remove X libraries from bare-bones system closure.

[bug#41735] [PATCH 0/2] Remove X libraries from bare-bones system closure.

[Mathieu Othacehe]

Hello,

Here's a small serie to remove X libraries from bare-bones system closure.

Thanks,

Mathieu

Mathieu Othacehe (2):
  gnu: Add openssh-no-x.
  system: bare-bones: Remove dependency to X libraries.

 gnu/packages/ssh.scm                | 8 ++++++++
 gnu/system/examples/bare-bones.tmpl | 3 ++-
 2 files changed, 10 insertions(+), 1 deletion(-)

-- 
2.26.2

[bug#41735] [PATCH 1/2] gnu: Add openssh-no-x.

[Mathieu Othacehe]

* gnu/packages/ssh.scm (openssh-no-x): New variable.
---
 gnu/packages/ssh.scm | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/gnu/packages/ssh.scm b/gnu/packages/ssh.scm
index 73841db6b5..13cf305695 100644
--- a/gnu/packages/ssh.scm
+++ b/gnu/packages/ssh.scm
@@ -239,6 +239,14 @@ Additionally, various channel-specific options can be negotiated.")
                                "See LICENSE in the distribution."))
    (home-page "https://www.openssh.com/")))
 
+;; OpenSSH without X support. This allows to use OpenSSH without dragging X
+;; libraries to the closure.
+(define-public openssh-no-x
+  (package
+    (inherit openssh)
+    (name "openssh-no-x")
+    (inputs (alist-delete "xauth" (package-inputs openssh)))))
+
 (define-public guile-ssh
   (package
     (name "guile-ssh")
-- 
2.26.2

[bug#41735] [PATCH 2/2] system: bare-bones: Remove dependency to X libraries.

[Mathieu Othacehe]

Use "openssh-no-x" to prevent the inclusion of X libraries to the bare-bones
system closure.

* gnu/system/examples/bare-bones.tmpl (operating-system)[services]: Use
openssh-no-x instead of openssh.
---
 gnu/system/examples/bare-bones.tmpl | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/gnu/system/examples/bare-bones.tmpl b/gnu/system/examples/bare-bones.tmpl
index 4f30a5b756..deed3c6c7a 100644
--- a/gnu/system/examples/bare-bones.tmpl
+++ b/gnu/system/examples/bare-bones.tmpl
@@ -3,7 +3,7 @@
 
 (use-modules (gnu))
 (use-service-modules networking ssh)
-(use-package-modules screen)
+(use-package-modules screen ssh)
 
 (operating-system
   (host-name "komputilo")
@@ -46,5 +46,6 @@
   (services (append (list (service dhcp-client-service-type)
                           (service openssh-service-type
                                    (openssh-configuration
+                                    (openssh openssh-no-x)
                                     (port-number 2222))))
                     %base-services)))
-- 
2.26.2

[bug#41735] [PATCH 1/2] gnu: Add openssh-no-x.

[Jan Nieuwenhuizen]

Mathieu Othacehe writes:

Hello!

> * gnu/packages/ssh.scm (openssh-no-x): New variable.
[..]
> diff --git a/gnu/packages/ssh.scm b/gnu/packages/ssh.scm
[..]
> +;; OpenSSH without X support. This allows to use OpenSSH without dragging X
> +;; libraries to the closure.
> +(define-public openssh-no-x
> +  (package
> +    (inherit openssh)
> +    (name "openssh-no-x")
> +    (inputs (alist-delete "xauth" (package-inputs openssh)))))

I like it!  Maybe use -sans-x?

Two remarks: after merging, we may want to use openssh-no-x for the Hurd
too and drop this from openssh proper:

             ,@(if (hurd-target?)
                   '()
                   `(("xauth" ,xauth)))))         ; for 'ssh -X' and 'ssh -Y'

We have this bit because xauth does not compile for the Hurd yet,
but it's kind of a kludge.

Not to hold this up and maybe we should take this discussion
elsewhere...but I have a ~5y old, bit-rotting stack of LilyPond cross
build packages (foo-SANS-X) laying around.  At the time--much to my
suprise--I found the LilyPond community less than enthousiastic to move
their binary package builds to Guix (yet); but adding these -NO-X
packages felt a bit weird.  So, I wonder if we could address the X'yness
of packages in a more structured way, that would be nice...

Greetings,
Janneke

-- 
Jan Nieuwenhuizen <janneke@gnu.org> | GNU LilyPond http://lilypond.org
Freelance IT http://JoyofSource.com | Avatar® http://AvatarAcademy.com

[bug#41735] [PATCH 1/2] gnu: Add openssh-no-x.

[Mathieu Othacehe]

Hey Jan,

> Two remarks: after merging, we may want to use openssh-no-x for the Hurd
> too and drop this from openssh proper:
>
>              ,@(if (hurd-target?)
>                    '()
>                    `(("xauth" ,xauth)))))         ; for 'ssh -X' and 'ssh -Y'
>
> We have this bit because xauth does not compile for the Hurd yet,
> but it's kind of a kludge.

I just did push the serie, so I think you could maybe adapt the openssh
package used by the Hurd and remove the snippet above, if it's ok for
you?

> Not to hold this up and maybe we should take this discussion
> elsewhere...but I have a ~5y old, bit-rotting stack of LilyPond cross
> build packages (foo-SANS-X) laying around.  At the time--much to my
> suprise--I found the LilyPond community less than enthousiastic to move
> their binary package builds to Guix (yet); but adding these -NO-X
> packages felt a bit weird.  So, I wonder if we could address the X'yness
> of packages in a more structured way, that would be nice...

Yes I share your concerns on that one, sadly I don't have much better to
propose, and I think it's important to have a bare-bones template as
minimal as possible. Anyway, this is not a big hack and it will be easy
to revert it if we find a better way.

Thanks,

Mathieu

[bug#41735] [PATCH 1/2] gnu: Add openssh-no-x.

[Jan Nieuwenhuizen]

Mathieu Othacehe writes:

Hi Mathieu,

> I just did push the serie, so I think you could maybe adapt the openssh
> package used by the Hurd and remove the snippet above, if it's ok for
> you?

Yes, sure; I'm right on it.

I just sent my final patch set for wip-hurd-vm, still using plain
"openssh" -- will create an and test updated patch.

>> Not to hold this up and maybe we should take this discussion
>> elsewhere...but I have a ~5y old, bit-rotting stack of LilyPond cross
>> build packages (foo-SANS-X) laying around.  At the time--much to my
>> suprise--I found the LilyPond community less than enthousiastic to move
>> their binary package builds to Guix (yet); but adding these -NO-X
>> packages felt a bit weird.  So, I wonder if we could address the X'yness
>> of packages in a more structured way, that would be nice...
>
> Yes I share your concerns on that one, sadly I don't have much better to
> propose, and I think it's important to have a bare-bones template as
> minimal as possible. Anyway, this is not a big hack and it will be easy
> to revert it if we find a better way.

Okay...I'm sure this will come around another time.

Janneke

-- 
Jan Nieuwenhuizen <janneke@gnu.org> | GNU LilyPond http://lilypond.org
Freelance IT http://JoyofSource.com | Avatar® http://AvatarAcademy.com

[bug#41735] [PATCH 1/2] gnu: Add openssh-no-x.

[Marius Bakke]

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

Mathieu Othacehe <m.othacehe@gmail.com> writes:

> * gnu/packages/ssh.scm (openssh-no-x): New variable.
> ---
>  gnu/packages/ssh.scm | 8 ++++++++
>  1 file changed, 8 insertions(+)
>
> diff --git a/gnu/packages/ssh.scm b/gnu/packages/ssh.scm
> index 73841db6b5..13cf305695 100644
> --- a/gnu/packages/ssh.scm
> +++ b/gnu/packages/ssh.scm
> @@ -239,6 +239,14 @@ Additionally, various channel-specific options can be negotiated.")
>                                 "See LICENSE in the distribution."))
>     (home-page "https://www.openssh.com/")))
>  
> +;; OpenSSH without X support. This allows to use OpenSSH without dragging X
> +;; libraries to the closure.
> +(define-public openssh-no-x
> +  (package
> +    (inherit openssh)
> +    (name "openssh-no-x")
> +    (inputs (alist-delete "xauth" (package-inputs openssh)))))

Maybe add a synopsis so users don't have to guess what this package
does.  I.e. (synopsis "OpenSSH client and server without X11 support").

I also think 'openssh-sans-x' is better, and is already a fairly common
convention.  :-)

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

bug#41735: [PATCH 1/2] gnu: Add openssh-no-x.

[Mathieu Othacehe]

Hello Marius,

> Maybe add a synopsis so users don't have to guess what this package
> does.  I.e. (synopsis "OpenSSH client and server without X11 support").
>
> I also think 'openssh-sans-x' is better, and is already a fairly common
> convention.  :-)

Fixed and pushed,

Thanks,

Mathieu