diff mbox series

[bug#67214] services: xvnc: Do not specify display number when using inetd

Message ID CAJ=Rwfadsuv18Gb0i_-OrkLq4kT+j_+z4v2L7ftDra_MG0dPXw@mail.gmail.com
State New
Headers show
Series [bug#67214] services: xvnc: Do not specify display number when using inetd | expand

Commit Message

Thompson, David Nov. 15, 2023, 10:53 p.m. UTC 
Hey everyone,

I've been struggling to get VNC working with multiple users. Upon
digging into it I realized that it's because the X display number is
fixed in the xvnc service, even when in inetd mode, so only the first
user who connects gets a desktop and everyone else gets an error. By
omitting the display number when using inetd, xvnc just picks an
unused display and I'm now able to log in with multiple user accounts.

The patch is a one-liner and hopefully not very controversial. :)

- Dave

Comments

Mathieu Othacehe Nov. 25, 2023, 6:11 p.m. UTC | #1 
Hello David,

> * gnu/services/vnc.scm (xvnc-configuration->command-line-arguments): Omit X
> display number when inetd? is #t.

Seems fair, I think you can go ahead.

Thanks,

Mathieu
Thompson, David Nov. 27, 2023, 4:03 p.m. UTC | #2 
Hey Mathieu,

On Sat, Nov 25, 2023 at 1:11 PM Mathieu Othacehe <othacehe@gnu.org> wrote:
>
>
> Hello David,
>
> > * gnu/services/vnc.scm (xvnc-configuration->command-line-arguments): Omit X
> > display number when inetd? is #t.
>
> Seems fair, I think you can go ahead.

Thanks for the review! Pushed as commit
dce3672f5a8eb440ae61a334384423d1e2ff5e65.

- Dave
diff mbox series

Patch

From a7aa18cbb8050ae228aeca2b4a1f724a797983ac Mon Sep 17 00:00:00 2001
Message-ID: <a7aa18cbb8050ae228aeca2b4a1f724a797983ac.1700088462.git.dthompson2@worcester.edu>
From: David Thompson <dthompson2@worcester.edu>
Date: Wed, 15 Nov 2023 17:46:34 -0500
Subject: [PATCH] services: xvnc: Do not specify display number when using
 inetd.

* gnu/services/vnc.scm (xvnc-configuration->command-line-arguments): Omit X
display number when inetd? is #t.

Change-Id: I0fc8e3068f3ae22421e60e96bd40e3b6e477ca99
---
 gnu/services/vnc.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/services/vnc.scm b/gnu/services/vnc.scm
index f90bd7258f..8b9ad0b179 100644
--- a/gnu/services/vnc.scm
+++ b/gnu/services/vnc.scm
@@ -149,7 +149,7 @@  (define (xvnc-configuration->command-line-arguments config)
     (xvnc display-number geometry depth port ipv4? ipv6? password-file xdmcp?
           inetd? frame-rate security-types localhost? log-level extra-options)
     #~(list #$(file-append xvnc "/bin/Xvnc")
-            #$(format #f ":~a" display-number)
+            #$@(if inetd? '() (list (format #f ":~a" display-number)))
             "-geometry" #$geometry
             "-depth" #$(number->string depth)
             #$@(if inetd?

base-commit: d987b75618a62c95c030e7ca53e0972e700c4f06
-- 
2.41.0