Closed awdeorio closed 1 year ago
CC @zjweiss
Makes sense to me.
The make debug
build should include the debugging symbols. I can tweak the 281_setup.md
, to explain that to students.
I agree that launch.json
should compile main_debug
, and not main
. Do we provide students with a template launch.json
?
I see #88 is suggesting making a template launch.json
.
I agree that launch.json should compile main_debug, and not main. Do we provide students with a template launch.json?
launch.json
doesn't compile anything. While it is possible to configure VS Code to compile something for you, we don't include that in the instructions.
launch.json
does include the path to the executable. In EECS 281, it's an anti-pattern to use the default release target because it lacks debugging symbols. The pattern we need is:
make debug
, not make
program
to ${workspaceFolder}/X_debug
, not ${workspaceFolder}/X
Working on this in #108.
I am not completely familiar with using VS Code for C++ development, but from reading the setup_vscode.html
guide, I want to confirm that students need to compile the code by typing make
/make debug
into the terminal, and then they can click the "debug icon" in VS Code, correct?
I am not completely familiar with using VS Code for C++ development, but from reading the setup_vscode.html guide, I want to confirm that students need to compile the code by typing make/make debug into the terminal, and then they can click the "debug icon" in VS Code, correct?
Yes, and they need to use main_debug
(or whatever *_debug
) in this step.
It's an easy mistake to make for EECS 281 students to fail to include debugging symbols in VS Code because
make
will compile the release target, not the debug target. Furthermore,launch.json
needs to be configurede to runmain_debug
notmain
.