You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
With the current makefiles, one may not change their VS Code config without it being reset to the no-OS defaults during the next build, which greatly hinders development. For instance, my WSL dev setup requires changing the gdb options in launch.json, but doing so is futile because the makefile resets my changes when building.
Options
I suggest (and will shortly open a PR) adding a condition to only write to these files if they are empty or don't exist (i.e., during project setup), but keep their current content otherwise, to allow for customization of the config.
Despite some effort, I didn't manage to do it using just GNU Make syntax, but rather by changing the file call from filename.json to filename.json.default, and having some shell commands conditionally copy filename.json.default to filename.json if the latter doesn't exist. Finally, shell commands remove the .default files to keep the directory tidy.
The text was updated successfully, but these errors were encountered:
trupples
added a commit
to trupples/no-OS
that referenced
this issue
May 14, 2024
Referencing issue analogdevicesinc#2181. $(PROJECT_TARGET)_configure targets in maxim.mk and stm32.mk
don't check if .vscode config files already exist and forcefully overwrite them with
the defaults during every build.
This adds some shell commands to only write the defaults if the files don't exist or
are empty = when setting up the project = when we actually need those defaults.
Signed-off-by: Ioan Dragomir <ioan.dragomir@analog.com>
Referencing issue #2181. $(PROJECT_TARGET)_configure targets in maxim.mk and stm32.mk
don't check if .vscode config files already exist and forcefully overwrite them with
the defaults during every build.
This adds some shell commands to only write the defaults if the files don't exist or
are empty = when setting up the project = when we actually need those defaults.
Signed-off-by: Ioan Dragomir <ioan.dragomir@analog.com>
Behaviour
The
$(PROJECT_TARGET)_configure
targets in tools/scripts/stm32.mk and tools/scripts/maxim.mk use the GNU Makefile
function to set up a default VS Code configuration for projects, with no regard to the contents of those files.no-OS/tools/scripts/maxim.mk
Lines 139 to 143 in 3cb5f58
For context, the content of the files is generated in vsc_intellisense.mk, vsc_openocd_debug.mk, and vsc_tasks.mk.
Impact
With the current makefiles, one may not change their VS Code config without it being reset to the no-OS defaults during the next build, which greatly hinders development. For instance, my WSL dev setup requires changing the gdb options in
launch.json
, but doing so is futile because the makefile resets my changes when building.Options
I suggest (and will shortly open a PR) adding a condition to only write to these files if they are empty or don't exist (i.e., during project setup), but keep their current content otherwise, to allow for customization of the config.
Despite some effort, I didn't manage to do it using just GNU Make syntax, but rather by changing the
file
call fromfilename.json
tofilename.json.default
, and having some shell commands conditionally copyfilename.json.default
tofilename.json
if the latter doesn't exist. Finally, shell commands remove the.default
files to keep the directory tidy.The text was updated successfully, but these errors were encountered: