Skip to content

Rewrite of download #79

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

Closed
wants to merge 7 commits into from
Closed

Rewrite of download #79

wants to merge 7 commits into from

Conversation

matteosuppo
Copy link
Contributor

I moved the code to download tools into a package.

The downloadtool works mostly like before, with the only difference that now it ignores the url provided to it relying instead on a configuration variable.

Usage:

downloadtool avrdude

This command is sent from websocket

The agent will parse the package_index.json in the configuration, will retrieve the latest version of the tool and it will download it.
It will check the Sha-256 checksum to ensure that the file hasn't been corrupted or tampered with.

Right now it's not used but there is the possibility to specify a certain version of a tool, in order to keep them versioned for boards needing an older version, for example.

You can test it with this release: https://github.com/arduino/arduino-create-agent/releases/tag/test-3
Fix #72

@ArduinoBot
Copy link

Test FAILed.

1 similar comment
@ArduinoBot
Copy link

Test FAILed.

@ArduinoBot
Copy link

Test FAILed.

1 similar comment
@ArduinoBot
Copy link

Test FAILed.

@ArduinoBot
Copy link

Test FAILed.

@ArduinoBot
Copy link

Test FAILed.

@ArduinoBot
Copy link

Test FAILed.

2 similar comments
@ArduinoBot
Copy link

Test FAILed.

@ArduinoBot
Copy link

Test FAILed.

@ArduinoBot
Copy link

✅ Build completed.

Download at http://downloads.arduino.cc/CreateBridge/CreateBridge-PR115.zip

@facchinm
Copy link
Member

Merged as #95

@facchinm facchinm closed this May 13, 2016
@facchinm facchinm deleted the md5 branch May 13, 2016 16:05
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.

4 participants