Skip to content

fix: Correct PIO builds#52

Merged
faustbrian merged 4 commits intoArkEcosystemArchive:masterfrom
ciband:chore/update_bip39
Jan 28, 2019
Merged

fix: Correct PIO builds#52
faustbrian merged 4 commits intoArkEcosystemArchive:masterfrom
ciband:chore/update_bip39

Conversation

@ciband
Copy link
Contributor

@ciband ciband commented Jan 26, 2019

Proposed changes

Updated BIP39 submodule and PIO configs to be more explicit in determining dependencies. This allows for more control and better determinism as to what PIO will actually compile into the project.

Types of changes

  • Bugfix (non-breaking change which fixes an issue)
  • Build (changes that affect the build system)

Checklist

  • I have read the CONTRIBUTING documentation
  • Lint and unit tests pass locally with my changes

Further comments

This should fix the current build issues in the open PRs.

Allow individual projects to install dependencies.
PIO was getting the automated dependency checking wrong.  Turn it off in favor of being explicit
@ciband ciband mentioned this pull request Jan 26, 2019
12 tasks
@sleepdefic1t
Copy link
Contributor

Shows "Expected..."; going to CircleCI, you can see it passed 👍

Thank you, @ciband

@sleepdefic1t sleepdefic1t mentioned this pull request Jan 27, 2019
12 tasks
@faustbrian faustbrian added the Bounty: Tier 5 Awarded for minor features, refactorings, improvements. This is valued at 0 USD. label Jan 28, 2019
@faustbrian faustbrian merged commit fa0468c into ArkEcosystemArchive:master Jan 28, 2019
@sleepdefic1t sleepdefic1t mentioned this pull request Feb 5, 2019
12 tasks
ciband added a commit to ciband/cpp-crypto that referenced this pull request Feb 18, 2019
* chore:  Update bip39 submodule to tips

* fix:  Remove explicit install of dependencies

Allow individual projects to install dependencies.

* fix:  Add explicit version requirement for BIP39

* fix:  Disable dependency checking in PIO

PIO was getting the automated dependency checking wrong.  Turn it off in favor of being explicit
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bounty: Tier 5 Awarded for minor features, refactorings, improvements. This is valued at 0 USD.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants