Skip to content
This repository has been archived by the owner on Dec 4, 2023. It is now read-only.

Added in V8::C::Debug::EnableAgent() #310

Closed
wants to merge 2 commits into from
Closed

Conversation

calh
Copy link

@calh calh commented Aug 7, 2014

Also adding in some quick documentation. Let me konw if there's any issues with the code! Seemed like a pretty simple addition.

I still haven't been able to use Eclipse for anything interesting. It crashes 80% of the time. And when it's not crashing, it doesn't show any detailed information on the stack, variables, or file sources. I haven't had much luck with jamming JSON into my telnet client either.

Attempting something like {"seq":725,"type":"request","command":"setexceptionbreak","arguments":{"type":"all","enabled":true}} doesn't seem to have any effect. Any suggestions?

@calh
Copy link
Author

calh commented Aug 7, 2014

I don't understand why the array tests are segfaulting. ( array_spec.rb, line 5, doing V8::C::Object::New() ) Is it possible my class builder line in Init() is affecting that?

@ignisf
Copy link
Collaborator

ignisf commented Nov 20, 2014

@calh, could this be the same issue as rubyjs/libv8#156 ?

@calh
Copy link
Author

calh commented Nov 20, 2014

@ignisf I'm not quite sure, it looks similar, but I can't seem to get a deep stack trace like you have. Here's a gist output of my spec run, and here's a gist running each individual spec file

What do you think?

@ignisf
Copy link
Collaborator

ignisf commented Nov 20, 2014

ugh... Can't be certain until I see a stack trace from gdb... What distro are you running this on?

@calh
Copy link
Author

calh commented Nov 20, 2014

Ubuntu, Trusty

@cowboyd
Copy link
Collaborator

cowboyd commented Aug 14, 2015

I'm not sure where the debug apis live in v8 4.5, but EnableAgent doesn't seem to be around anymore.

@cowboyd cowboyd closed this Aug 14, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants