jupyter-remote-desktop-proxy icon indicating copy to clipboard operation
jupyter-remote-desktop-proxy copied to clipboard

Define VNC servers to support

Open consideRatio opened this issue 2 years ago • 5 comments

This project ships with TigerVNC, but has support for use with TurboVNC as well, and really any vncserver binary:

If a vncserver executable is found in PATH it will be used, otherwise a bundled TightVNC server is run. You can use this to install vncserver with support for other features, for example the Dockerfile in this repository installs TurboVNC for improved OpenGL support.

Should we aim for something simpler to reduce complexity and ensure robustness on what we really can manage to test and maintain? I'm thinking that we could remove TigerVNC for example.

Related

  • With some users using one but not the other VNC server, it becomes harder to ensure bugs aren't introduced: #54
  • TigerVNC has a different kind of license: #27
  • TigerVNC doesn't support aarch64: https://github.com/jupyterhub/jupyter-remote-desktop-proxy/issues/27#issuecomment-1418646823

consideRatio avatar Aug 25 '23 10:08 consideRatio

The bundled TigerVNC is x86_64 only.

Debian/Ubuntu has packages for both x86_64 and AArch64

benz0li avatar Aug 25 '23 13:08 benz0li

I actually think we should just stop bundling a VNC server completely, and just require it be installed separately. This also solves the licensing issue.

yuvipanda avatar Aug 25 '23 16:08 yuvipanda

I'd like to reach consensus on a path forward among us who have involved in thinking about this so far @cmd-ntrf @benz0li @yuvipanda.

  • Question 1: Do we stop bundling TigerVNC?
  • Question 2: Do we provide install scripts for TigerVNC and/or TurboVNC?
  • Question 3: Do we commit to testing TigerVNC and/or TurboVNC?

Related

Answer template

> **Question 1: Do we stop bundling TigerVNC?**

> **Question 2: Do we provide install scripts for TigerVNC and/or TurboVNC?**

> **Question 3: Do we commit to testing TigerVNC and/or TurboVNC?**

consideRatio avatar Oct 11 '23 10:10 consideRatio

Question 1: Do we stop bundling TigerVNC?

Yes, its x86_64 specific and it has a licence we are breaking.

Question 2: Do we provide install scripts for TigerVNC and/or TurboVNC?

No, I don't think we should. If we do, we should have it for all the VNC clients we support. My motivation for not suggesting we introduce this is because of the maintenance burden we take on in this project that currently doesn't have tests setup. If we have tests setup, I'm open to re-evaluating this answer.

Question 3: Do we commit to testing TigerVNC and/or TurboVNC?

To me, this is the same as "Do we commit to supporting TigerVNC and/or TurboVNC". I'm open to supporting / testing both and, or just picking TurboVNC - but I'm hesitant to only pick TigerVNC based on what I've understood so far.

consideRatio avatar Oct 11 '23 10:10 consideRatio

I agree with @consideRatio's answers.

benz0li avatar Oct 23 '23 07:10 benz0li

We now have tests for turbovnc and tigervnc and its far more clear now, closing as resolved!

consideRatio avatar Jul 08 '24 17:07 consideRatio