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

Update Arduino IDE version to 1.8.7 #141

Merged
merged 2 commits into from
Oct 9, 2018
Merged

Conversation

frazar
Copy link

@frazar frazar commented Oct 8, 2018

This kind of changes could be performed automatically by setting the CI script to always use the latest version of the Arduino.

@frazar
Copy link
Author

frazar commented Oct 8, 2018

The build seems to be hitting this unresolved bug in the Arduino IDE.

In particular, since commit e209f7d, Travis builds of arduino-lmic use the cached .arduino15 folder from previous builds. Thus, when the command

arduino --install-boards mcci:samd

is run, it finds that the mcci:samd board is already installed, prints the following message

java.lang.Exception: Platform is already installed!
	at cc.arduino.contributions.packages.ContributionInstaller.install(ContributionInstaller.java:77)
	at processing.app.Base.<init>(Base.java:324)
	at processing.app.Base.main(Base.java:145)

and fails with error code 255.

@terrillmoore
Copy link
Member

Thanks for the request.

I'm perhaps too conservative about using the "latest" version of the IDE in CI testing.

Anyway, I think the thing to do is to check what boards are installed and then don't try to install things that are already installed. That's easy enough to do in the ci script. I think I need to resolve this, will first fix the caching issue, then accept this pull request, then look into #140.

@terrillmoore terrillmoore self-assigned this Oct 8, 2018
@terrillmoore
Copy link
Member

Adjusted to take into account fixes from #143.

@terrillmoore terrillmoore merged commit 849cfb2 into mcci-catena:master Oct 9, 2018
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