Skip to content

Replace ctx.ToolsDirs with ctx.BuiltInToolsDirs #418

New issue

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

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

Already on GitHub? # to your account

Merged
merged 2 commits into from
Sep 26, 2019

Conversation

cmaglie
Copy link
Member

@cmaglie cmaglie commented Sep 20, 2019

The ToolsDirs field has been replaced by BuiltInToolsDirs during the big refactoring in the arduino-builder repo.

The intention was to remove ToolsDirs once done, but for some reasons I forgot to remove it.

This commit completes the transition.

@cmaglie cmaglie self-assigned this Sep 20, 2019
The ToolsDirs field has been replaced by BuiltInToolsDirs during the big
refactoring in the arduino-builder repo. The intention was to remove
ToolsDirs once done, but for some reasons I forgot to remove it.

This commit completes the transition.
@cmaglie
Copy link
Member Author

cmaglie commented Sep 20, 2019

@ArduinoBot build this please

@ArduinoBot
Copy link
Contributor

✅ Build completed.

⬇️ Linux 64: https://downloads.arduino.cc/PR/arduino-cli/arduino-cli-155-PR418-linux64.tar.bz2

⬇️ Linux 32: https://downloads.arduino.cc/PR/arduino-cli/arduino-cli-155-PR418-linux32.tar.bz2

⬇️ Linux ARM 32: https://downloads.arduino.cc/PR/arduino-cli/arduino-cli-155-PR418-linuxarm32.tar.bz2

⬇️ Linux ARM 64: https://downloads.arduino.cc/PR/arduino-cli/arduino-cli-155-PR418-linuxarm64.tar.bz2

⬇️ Windows: https://downloads.arduino.cc/PR/arduino-cli/arduino-cli-155-PR418-windows.zip

⬇️ OSX: https://downloads.arduino.cc/PR/arduino-cli/arduino-cli-155-PR418-macosx.zip

ℹ️ To test this build:

  1. Just replace arduino-cli binary with the provided one

Copy link
Contributor

@masci masci left a comment

Choose a reason for hiding this comment

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

LGTM

@cmaglie cmaglie merged commit c23bb9c into arduino:master Sep 26, 2019
@cmaglie cmaglie deleted the fix-builtin-tools-legacy branch September 26, 2019 13:27
cmaglie added a commit to cmaglie/arduino-builder that referenced this pull request Sep 26, 2019
cmaglie added a commit to arduino/arduino-builder that referenced this pull request Sep 26, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants