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.

Sign up for GitHub

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

Jump to bottom

armv6 and v7 download link update #108

Merged
merged 8 commits into from
Oct 21, 2021
Merged

armv6 and v7 download link update #108

merged 8 commits into from
Oct 21, 2021

Conversation

ersonp
Copy link
Contributor

@ersonp ersonp commented Apr 23, 2021 •
edited
Loading

Fixes #

Changes:

  • Updated SKYWIRE_VERSION to v0.5.0
  • Renamed RASPBIAN_ARMHF_DOWNLOAD_URL to RASPBIAN_ARM_DOWNLOAD_URL
  • Removed SKYWIRE_ARMV6_DOWNLOAD_URL as skywire offers armv6 for both armv6 and armv7 in SKYWIRE_ARM_DOWNLOAD_URL
  • Updated build.sh to reflect the SKYWIRE_ARMV6_DOWNLOAD_URL change

Does this change need to mentioned in CHANGELOG.md?

Yes | No

@ersonp ersonp changed the title armv6 and v7 link update armv6 and v7 download link update Apr 23, 2021
jdknives
jdknives approved these changes Apr 23, 2021
View reviewed changes
Copy link
Member

@jdknives jdknives left a comment

Choose a reason for hiding this comment

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

This looks good and should work as soon as we have a new release out. Will leave it in this state for now and merge it close to the next release once the Skywire v0.5.0 is out.

@ersonp ersonp mentioned this pull request Aug 5, 2021
Merged
@ersonp ersonp marked this pull request as ready for review October 20, 2021 11:20
@jdknives jdknives merged commit d694a1e into skycoin:develop Oct 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jdknives jdknives jdknives approved these changes

Assignees
No one assigned
Labels
milestone 5
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@ersonp @jdknives @mrpalide