On 05/30/2015 04:12 PM, Mike Gabriel wrote:
Hi Salvador,
On Fr 29 Mai 2015 18:18:56 CEST, Salvador Fandiño wrote:
See the commit message on the attached patch for the details.
This is a minimal version introducing as few changes as possible in the nx-libs code.
thanks for sending in your patch. Can you do this:
Clone https://github.com/ArcticaProject/nx-libs
[Switch to (default) branch 3.6.x.]
Rebase your patch against that branch (which is probably zero work, because the 3.5.0.x branch and the 3.6.x branch don't differ that much concerning nxcomp).
File a pull request (PR) on Github against the 3.6.x branch.
The patch won't be accepted for 3.5.0.x, if we will include it in 3.6.x needs some discussion (at our X2Go Gathering, I suggest, or earlier on IRC).
In general, I want to have Unix file socket support in NX rather sooner than later (also for the nxproxy <-> nxproxy/nxagent) connection itself, so I am in favour of shipping this patch already in 3.6.x and not postponing it until 3.7.x. But we should reach a consensus with the other devs.
Also, the channels in NX are not used in X2Go and will neither be used in Arctica, so you patch will not affect either product that much.
So, is there actually anybody using the native channel support in nx besides us (QVD)? I ask because the current design (and implementation) is quite bad. Specially, having a hard-coded set of channels is a nuisance.
If backward compatibility is negotiable I would consider redoing it in a different way, more in line with the channel support in SSH.