Update framework-kendryte-standalone-sdk version and builder script #41
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
2 tasks
Default the value of upload.slow to avoid build errors for boards whe…
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Recent projects like https://github.com/kendryte/kendryte-standalone-demo/tree/develop/face_detect cannot be compiled under PlatformIO because the SDK (https://github.com/kendryte/kendryte-standalone-sdk) package it uses has become outdated with respect to the master branch, and so, errors like
are encountered. Sadly Sipeed doesn't seem to be making regular releases of the SDK anymore (V0.5.6 is from May 2019) but the SDK is still being further developed.
This PR mainly updates the builder script
builder/frameworks/kendryte-standalone-sdk.pyto adapt to the new SDK version and compilation settings. It also incorporates PR #39 because otherwise a Python error occurs. It also changes the version offramework-kendryte-standalone-sdkto point directly to thegitversion, however, after a merge this should changed back to a stable package version together with an updated version of that package (CC @ivankravets).A test project that showcases working compilation with this updated version and non-working of it with the current version is located at https://github.com/maxgerhardt/pio-kendryte-facedetect-demo.
This stems from a user's post about not being able to compile the reference project at https://community.platformio.org/t/kendryte-face-detect-example-with-sipeed-maix-bit/24763.
Also linked to #31.