Skip to content

Set up a working debug configuration#6

Open
shawwn wants to merge 3 commits into
avaer:masterfrom
shawwn:2018-06-14/resample
Open

Set up a working debug configuration#6
shawwn wants to merge 3 commits into
avaer:masterfrom
shawwn:2018-06-14/resample

Conversation

@shawwn

@shawwn shawwn commented Jun 14, 2018

Copy link
Copy Markdown

This PR adds the ability to step through LabSound in Visual Studio's debugger.

To do this, I run build-windows-debug using MSBuild command prompt, then I run node-gyp-debug, which is a custom script in my PATH:

CALL node-gyp configure rebuild --nodedir="B:\node" --debug %*

B:\node is where I ran .\vcbuild debug x64 openssl-no-asm to build the latest Node master branch from source.

At that point, you can successfully use VS's debugger! And you can even see the stack trace all the way to the bottom of Node.

I was going to continue pushing the sample rate fixes, but this seemed like a good spot to submit a PR.

@shawwn

shawwn commented Jun 14, 2018

Copy link
Copy Markdown
Author

For reference, here's how I set up my VS locally:

image

image

image

Comment thread binding.gyp
}],
'libraries': [
'LabSound.lib',
'avformat.lib',

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spacing?

Comment thread binding.gyp
'<(module_root_dir)/labsound/build/x64/Release',
['OS=="win"', {
'configurations': {
'Debug': {

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can release come first? Might be easier to scan between the os flavors.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants