PadreIDE/Debug-Client

t/07-initialize.t hangs with Term-ReadLine-Gnu-1.26

Closed this issue · 8 comments

Upgrading Term-ReadLine-Gnu from 1.25 to 1.26, the t/07-initialize.t dead-locks here:

$ prove -b -v t/07-initialize.t
t/07-initialize.t ..
1..4
ok 1 - initialize with prams
LOCK
ok 2 - quit with prams
ok 3 - initialize without prams
ok 4 - quit witout prams
ok

This is caused by Term-ReadLine-Gnu commit:

r481 | hayashi | 2015-01-31 13:07:49 +0100 (So, 31 led 2015) | 2 lines
make handling of iostreams simple (make _rl_store_iostream() return void and remove _rl_fetch_iostream()) [rt.cpan.org #101078]

The issue is that the "c" command is sent by run() method to before debugger emits the "DB<1>" prompt. It also possible it is a bug in the debugger because the prompt is never emitted if the debugger is run from the t/07-initialize.t test while Term-ReadLine-Gnu-1.26 is installed. If run the debugger from another session, the prompt appears and tests pass.

See https://bugzilla.redhat.com/show_bug.cgi?id=1189459 (perl-Debug-Client bug for Fedora Linux distribution) and https://rt.cpan.org/Public/Bug/Display.html?id=101078 (Term-ReadLine-Gnu bug introducing the Term-ReadLine-Gnu change.

ppisar
thanks for the bug
I will added to my todo list, thanks

I happy to write that fix for this specific issue published on the Term-ReadLine-Gnu side.

I still think some improvements could be done in the synchronization in the Debug-Client tests to eliminate the seldom and race-prone "sleep" commands.

Is there a way to fix this bug? Padre won't install due to it.

On Tue, Mar 31, 2015 at 05:20:36AM -0700, nbdsp wrote:

Is there a way to fix this bug? Padre won't install due to it.

You can apply a patch to Term-ReadLine-Gnu
https://rt.cpan.org/Public/Bug/Display.html?id=101078#txn-1463705.

-- Petr

Thanks! That worked.

Please let me know about the files in which i have to apply below mentioned patch. I am unable to find files Gnu.pm or Gnu.xs either. Thanks in advance...

You can apply a patch to Term-ReadLine-Gnu
https://rt.cpan.org/Public/Bug/Display.html?id=101078#txn-1463705.

Term-ReadLine-Gnu is at 1.35 already. I assume this is not an issue any more.

It's still an issue with Term-ReadLine-Gnu 1.35, so the suggested fixes for 1.25/1.26 are probably not applicable. Padre (for me) won't install :(