Skip to content

added OTA Update S3 example with a sample bin #445

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

Merged
merged 2 commits into from
Jun 14, 2017

Conversation

arvindr21
Copy link
Contributor

OTA Update example with AWS S3. Sample bin added along with sketch for quick validation.

Build & upload
Step 1 : Preferences > Show verbose output during `compilation`
Step 2 : ctrp + r (Verify sketch or build sketch)
Step 3 : In the verbose log, you will find a path t0 `sketch-name.iono.bin` on your machine. Example from Apple OSX:
Copy link
Member

Choose a reason for hiding this comment

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

You have a menu Sketch -> Export compiled binary which puts the bin in the sketch folder. Maybe use that?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks. Did not know about that option 👍 . Fixing it.

// chill
}

void execOTA() {
Copy link
Member

Choose a reason for hiding this comment

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

I will advise towards moving your functions above setup() and loop(). Reason is that while Arduino IDE and some other tools that use Arduino declare all your functions before compiling, this is not a standard, so if you grab this code and paste it into "make" based build, it will fail with a bunch of errors.

Copy link
Member

Choose a reason for hiding this comment

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

In this order :)

String getHeaderValue(String header, String headerName);
void execOTA();
void setup();
void loop();

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Will Do.

@me-no-dev me-no-dev merged commit 8965358 into espressif:master Jun 14, 2017
@arvindr21
Copy link
Contributor Author

Thanks.

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