vendor: Fix overflown callstack trace bug

[linuxion]

The problem and the fix are described here.

If @olebedev merges my PR, you can undo the changes using

govendor update gopkg.in/olebedev/go-duktape.v3::gopkg.in/olebedev/go-duktape.v3
govendor update gopkg.in/olebedev/go-duktape.v3

if necessary.

[linuxion]

@karalabe, just a reminder. You have done a lot of cool geth improvements last 2 months, but it is a little inconvenient to recompile geth each update. :(

[AyushyaChitransh]

Pinging to check if this PR is ready.

[holiman]

I'm not a fan of this. It changes the vendoring into gopkg.in/linuxion/go-duktape.v3. I'd much prefer it if the original maintainer merged that PR instead of us vendoring a fork.

[karalabe]

I think our choices here are 1) upstream the change; 2) make this thing configurable and not baked in; 3) pull the entire ducktape code into our repo. The best would be 2, but someone needs to champion that upstream.

[karalabe]

If not 2, I'd go with 3 (we already have ducktape vendored in, so it's just moving form vendor to embed).

[holiman]

Just pinged upstream

[holiman]

Closing this, as we'd rather use an upstream fix