Hi Uli, hi Orion,
On Mi 21 Nov 2018 01:13:15 CET, Ulrich Sibiller wrote:
On Tue, Nov 20, 2018 at 11:45 PM Orion Poplawski <orion@nwra.com> wrote:
See https://bugs.x2go.org/cgi-bin/bugreport.cgi?bug=1322 for my proposal for addressing this on the server side. I believe by adding the line above you are messing up many of the default options. According to the man
page of nxagent:-options filename path to an options file containing nx/nx options (see below).
Yes, I know. I have worked on that option handling stuff in nxagent several times. But x2go is not passing the options file via the command line option. It is embedding the path into the DISPLAY variable. My (quick) tests showed that it worked. I am still waiting for the original poster to specify in detail what problems he had after applying that configuration. Nevertheless I have not looked into the code to check if there's anything that will prevent this from working properly.
Regarding your proposal: While it is not the cleanest approach I think we should include it to help users in special cases.. The sleep stuff really should be handled by x2goclient. @Mihai Can you please include the patch for the next release?
I have just applied Orion's patch from #1322.
Nightly build of x2goserver is on its way...
DAS-NETZWERKTEAM mike gabriel, herweg 7, 24357 fleckeby mobile: +49 (1520) 1976 148 landline: +49 (4354) 8390 139
GnuPG Fingerprint: 9BFB AEE8 6C0A A5FF BF22 0782 9AF4 6B30 2577 1B31 mail: mike.gabriel@das-netzwerkteam.de, http://das-netzwerkteam.de