-
Notifications
You must be signed in to change notification settings - Fork 42
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
DRAFT: Fix for VSCode Windows Extension #89
Closed
wyattearp
wants to merge
33
commits into
DangerousPrototypes:main
from
wyattearp:fix-87-windows-auto-build-failing
Closed
DRAFT: Fix for VSCode Windows Extension #89
wyattearp
wants to merge
33
commits into
DangerousPrototypes:main
from
wyattearp:fix-87-windows-auto-build-failing
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
…co SDK 1-wire program (not working)
now using the default cmake instead of the python one to align to the operting system's preferred directions. this shouldn't really be an issue as there's nothing extremely fancy going on in cmake also moving the docker compose to my personal repo for now, need to talk with the bp team if they're fine with me actually pushing a real org for them.
now using the default cmake instead of the python one to align to the operting system's preferred directions. this shouldn't really be an issue as there's nothing extremely fancy going on in cmake also moving the docker compose to my personal repo for now, need to talk with the bp team if they're fine with me actually pushing a real org for them.
with the inclusion of these changes, we address the python weirdness for cmake as well as we now correct the issue with picotool being unable to bootstrap. it comes at the cost that if we update picotool to a different version, we have to update the docker, but that appears to be a documented issue in their bugs so ... until that's better we're going to have to stick with that. also added the .devcontainers for vscode users. this should just "bring up" a build environment for the vscode system. tested only on linux, but looking for windows volunteers. do note that if it fails to build, it's likely that you have a permissions issue on your current build/ directory and need to clean it up first.
…ipx-docker Fix: update for pip and docker
Add legacy mode for third parties flashrom avrdude
I think I'm really hitting something strange here, like the Windows command line character limit being 8191 characters. It's odd as well that cmake says the line builds but then errors out: |
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.
Looking to fix #87 - I believe the pico-vscode.cmake is likely missing from being commited, but I also think it should probably live in the .vscode directory, which is currently under the .gitignore list.
This is just a draft so I can make sure I'm getting the right items into place, will update with a corrected .vscode/ and .gitignore as well as the corrected pico-vscode.cmake (when I find it).