Skip to content

Fix web VNC app#102

Merged
Tuxliri merged 7 commits intoentefrom
use-correct-launch-script-for-vnc-DTSW-7386
Nov 11, 2025
Merged

Fix web VNC app#102
Tuxliri merged 7 commits intoentefrom
use-correct-launch-script-for-vnc-DTSW-7386

Conversation

@Tuxliri
Copy link
Copy Markdown
Contributor

@Tuxliri Tuxliri commented Nov 11, 2025

Improve static file serving, update the noVNC launch process, and clean up legacy or problematic code.

Frontend and Static File Serving Improvements:

  • Updated Nginx routing.
  • Changed the VNC frontend to load static/novnc/vnc.html instead of static/vnc.html, aligning with the new static file structure.

noVNC Launch and Supervisor Configuration:

  • Changed supervisor configuration to launch novnc_proxy directly instead of using the older launch.sh script

Codebase Cleanup and Dependency Updates:

  • Cleaned up Dockerfile and dependencies by removing unused build steps and legacy camera calibration.

Testing

  1. Clone my fork of the duckietown-shell-commands repo and checkout the use-parsed-distro-in-dts-code-vnc-command-DTSW-7385 branch (here).
  2. Clone this branch of the dt-gui-tools and build it using dts devel build
  3. Run dts code vnc in any ente lx (for example the control lx available here)
  4. Follow the onscreen instructions to open the VNC viewer (if the page does not load correctly you might need to reload the page with SHIFT+CMD+R for it to load).

@Tuxliri
Copy link
Copy Markdown
Contributor Author

Tuxliri commented Nov 11, 2025

This has been tested and confirmed working on Ubuntu 24.04 (amd64) and macOS (devcontainer).

Copy link
Copy Markdown
Member

@tanij tanij left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested, works

@liammcalpineduckietown liammcalpineduckietown removed their request for review November 11, 2025 13:08
@Tuxliri Tuxliri merged commit e4025ab into ente Nov 11, 2025
2 checks passed
@Tuxliri Tuxliri deleted the use-correct-launch-script-for-vnc-DTSW-7386 branch November 11, 2025 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants