Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support Remote Development Board Selection #1459

Merged
merged 3 commits into from
Feb 17, 2022

Conversation

davidcooper1
Copy link
Contributor

@davidcooper1 davidcooper1 commented Feb 14, 2022

Currently, the port of the workspace extension on the remote host is not automatically forwarded in the local VS Code instance. This PR uses vscode.env.asExternalUri in arduinoContentProvider instead of localhost, so the port is automatically forwarded. This way, calls to the API don't fail from the local UI. To isolate the issue, I opened up the dev tools for VS Code and manually forwarded the port the Arduino extension was using for the web server. The board view showed up after forwarding, indicating it is not automatically forwarded.

Microsoft Docs Reference
Fixes #1064

How I Tested:

Host - Windows

  • VS Code: 1.64.2
  • Arduino v0.4.11 (Installed from Extension Marketplace)

Remote - Raspberry Pi Running Latest Rasbian 64 OS

  • VS Code: 1.64.2
  • No Arduino Extension Installed
  • C/C++ Extension v1.8.4 (Installed from Extension Marketplace)
  • Node: 14.19.0
  • NPM: 6.14.16

Steps

  1. Remote SSH into Pi and clone repository.
  2. Run the build task manually using gulp build, as the preLaunchTask for Launch Extension throws an error for cannot find gulp.
  3. Comment out the preLaunchTask for Launch Extension.
  4. Run Launch Extension.
  5. VS Code window should launch with the newly compiled Arduino extension. When checking the extension tab, the extension still shows as grayed out.
  6. Open a .ino file to load the extension.
  7. Try to select a board. Page should load.
  8. Compare results to just installing the Arduino extension from Marketplace.

@ghost
Copy link

ghost commented Feb 14, 2022

CLA assistant check
All CLA requirements met.

@benmcmorran
Copy link
Member

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@benmcmorran benmcmorran merged commit a0a5f95 into microsoft:main Feb 17, 2022
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.

Board Manager / Library Manager are Empty on VirtualBox / Remote SSH
2 participants