-
Notifications
You must be signed in to change notification settings - Fork 914
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
Fix loading of PICO_TOOLCHAIN_PATH #262
Conversation
josesimoes
commented
Mar 16, 2021
•
edited
Loading
edited
- Add double quotes because build option it's a string.
- Remove comment as requested by @kilograham.
- Resolves Build option PICO_TOOLCHAIN_PATH has no effect #258.
Nice 👍 Apologies if any of my uninformed comments threw you off the scent! 👃 |
cmake/pico_pre_load_toolchain.cmake
Outdated
@@ -1,6 +1,6 @@ | |||
# PICO_CMAKE_CONFIG: PICO_TOOLCHAIN_PATH, Path to search for compiler, default=none (i.e. search system paths), group=build | |||
# Set your compiler path here if it's not in the PATH environment variable. | |||
set(PICO_TOOLCHAIN_PATH "" CACHE INTERNAL "") | |||
# Set your compiler path here with PICO_TOOLCHAIN_PATH CMake option, if it's not in the PATH environment variable. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd remove this comment completely - the PICO_CMAKE_CONFIG comment explains what the var is for.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
sorry, can you please rebase this onto develop-1.1.1 (it'll get released sooner that way) - still working thru the best branching policy! |
- Add double quotes because build option it's a string. - Remove comment as requested by @kilograham. - Resolves raspberrypi#258.
@kilograham I've started started over from develop1.1.1 and changed the base as develop1.1.1. |