issue #34: use result of shutil.which... to include correct compiler path in vscode c_cpp_properties.json

raspberrypi / pico-project-generator

Tool to automatically generate a Pico C SDK Project

BSD 3-Clause "New" or "Revised" License

289 stars 74 forks source link

issue #34: use result of shutil.which... to include correct compiler path in vscode c_cpp_properties.json #57

Closed ajbt closed 1 year ago

ajbt commented 1 year ago

I made these changes locally for my own use, and thought I'd submit this pull request since it addresses an issue (issue #34 ). I won't be offended if it's disposed of!

Note that this change uses the result of shutil.which(COMPILER_NAME) via CheckPrerequisites() and plumbs it through to generateProjectFiles(), as opposed to the proposal in issue #34 to provide the compiler path as a command line argument.

JamesH65 commented 1 year ago

Have used this as a basic for a simpler fix, and allows allows a command line parameter to override the system location. 6f2b85fcf85dae3f302d7815fefa0730a6ec1a60

JamesH65 commented 1 year ago

Thanks. First time I had heard of f-strings! Very useful.