Skip to content
This repository has been archived by the owner on Oct 1, 2024. It is now read-only.

Plugin appends 'Arduino.app' to path on MacOS #1221

Closed
recursivecodes opened this issue Mar 31, 2021 · 5 comments · Fixed by #1227
Closed

Plugin appends 'Arduino.app' to path on MacOS #1221

recursivecodes opened this issue Mar 31, 2021 · 5 comments · Fixed by #1227

Comments

@recursivecodes
Copy link

Why?

If I'm using the CLI, I put /Users/bin/ and arduino-cli for the command path, but it ends up looking in /Users/bin/Arduino.app/arduino-cli. If I'm trying to use the CLI, it should not append anything to the path like that.

if (directoryExistsSync(path.join(scanPath, "Arduino.app"))) {

@recursivecodes
Copy link
Author

Any thoughts on this?

@awmichel
Copy link
Contributor

@recursivecodes I ran into the same issue and I believe the PR I just opened #1227 resolves this issue. I'm not too familiar with VSCode extension development, so I'm not sure about the steps to install an extension from a git repo, but I manually edited the source in ~/.vscode/extensions/vsciot-vscode.vscode-arduino-0.4.0/out/src/ to test out my changes.

@adiazulay
Copy link
Contributor

Thanks @awmichel for fixing this!

@recursivecodes this was an artifact of how the extension would handle accessing the IDE in older builds.

@adiazulay
Copy link
Contributor

This should be fixed in v0.4.1. @awmichel do you mind giving this a testing it out and letting me know if it works?
https://github.com/microsoft/vscode-arduino/releases/tag/v0.4.1-rc1

@awmichel
Copy link
Contributor

@adiazulay Installed 0.4.1-rc1 locally and was able to verify and upload to a ESP32 using arduino-cli.

For anyone else using a MacOS M1 w/ arduino-cli installed via Homebrew, your VSCode settings should look something like:

{
    "arduino.useArduinoCli": true,
    "arduino.path": "/opt/homebrew/bin/",
    "arduino.commandPath": "arduino-cli",
}

On a non-M1 Mac (untested):

{
    "arduino.useArduinoCli": true,
    "arduino.path": "/usr/local/bin/",
    "arduino.commandPath": "arduino-cli",
}

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants