debugger: --debug-brk should pause until debug client connects #3589

Activity

1. bnoordhuis

   added

   debugger

   on Oct 29, 2015
2. bnoordhuis commented on Oct 29, 2015

   

   bnoordhuis

   on Oct 29, 2015

   MemberAuthor

   More actions

   /cc @indutny
3. Trott commented on Jun 1, 2016

   

   Trott

   on Jun 1, 2016

   Last edited by Trott

   Member

   More actions

   It appears that, as it is currently written,test/parallel/test-debug-brk.js depends on this bug. If it were not for this bug, that test would not exit:

   'use strict';
   
   const common = require('../common');
   const assert = require('assert');
   const spawnSync = require('child_process').spawnSync;
   
   const args = [`--debug-brk=${common.PORT}`, '-e', '0'];
   const proc = spawnSync(process.execPath, args, {encoding: 'utf8'});
   assert(/Debugger listening on/.test(proc.stderr));
4. bnoordhuis commented on Jun 1, 2016

   

   bnoordhuis

   on Jun 1, 2016

   MemberAuthor

   More actions

   I added test/parallel/test-debug-brk.js in #3585 to catch regressions in the current behavior. I don't think the current behavior is the best or the intended behavior, though.
5. Trott commented on Jun 1, 2016

   

   Trott

   on Jun 1, 2016

   Member

   More actions

   Possible fix: #7089
6. Trott

   added 2 commits that reference this issue on Jun 4, 2016

   debugger: fix --debug-brk interaction with -e

   ...

   10c64e9

   debugger: fix --debug-brk interaction with -e

   ...

   1a21524
7. Trott

   closed this as completedin 1a21524on Jun 4, 2016
8. evanlucas

   added a commit that references this issue on Jun 15, 2016

   debugger: fix --debug-brk interaction with -e

   ...

   2961f06