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

Apply changes from PR #82 to the develop [default] branch (aid on solution for fibercrypto/skywallet-mcu#342) #84

Conversation

stdevPavelmc
Copy link
Contributor

@stdevPavelmc stdevPavelmc commented Dec 13, 2019

Fixes #83 changes was wrongly applied to master when the default branch is develop, develop is the one used on fibercrypto/skywallet-mcu and this will aid to fix fibercrypto/skywallet-mcu#342

Changes:

  • Forces Python3 (python2 deprecation and fail)
  • Improve OS detection by adding windows under MSYS2
  • Introduce the windows build for the target build-c (under MSYS2) see Continuous integration on Windows skywallet-mcu#342 for details
  • Add the clean options for the windows build
  • Touch phony targets, etc.

Does this change need to mentioned in CHANGELOG.md?
No

Related issues:
PR fibercrypto/skywallet-mcu#342
PR #81

@stdevPavelmc
Copy link
Contributor Author

@olemis This PR is ready to merge

@stdevPavelmc stdevPavelmc self-assigned this Dec 13, 2019
@stdevPavelmc stdevPavelmc added bug Something isn't working ci Tests and continuous integration os-windows Specific to Microsoft Windows platform stdev wip Work in progress and removed bug Something isn't working labels Dec 13, 2019
@olemis olemis merged commit 81be0e9 into fibercrypto:develop Dec 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Tests and continuous integration os-windows Specific to Microsoft Windows platform stdev wip Work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants