Skip Menu |

This queue is for tickets about the TermReadKey CPAN distribution.

Report information
The Basics
Id: 96824
Status: resolved
Priority: 0/
Queue: TermReadKey
People
Owner: Nobody in particular
Requestors: josh.narins [...] gmail.com
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Sun Jun 29 12:30:51 2014 josh.narins [...] gmail.com - Ticket created
Subject: Tests hang under nohup
Date: Sun, 29 Jun 2014 12:30:37 -0400
To: bug-TermReadKey [...] rt.cpan.org
From: Josh Narins <josh.narins [...] gmail.com>
TermReadKey version 2.32 as part of Bundle::CPAN Perl 5.18.2 OS: 3.2.0-65-generic #98-Ubuntu SMP A minimal Makefile: all: OPENSSL_PREFIX=/chroot/a/openssl PERL_MM_USE_DEFAULT=1 \ perl -MCPAN -e 'install Bundle::CPAN' > log 2>&1 Run with nohup as follows: Show quoted text
> nohup make &
Hangs after printing "ok 7"
  Mon Jun 30 17:40:58 2014 josh.narins [...] gmail.com - Correspondence added
Subject: [rt.cpan.org #96824]
Date: Mon, 30 Jun 2014 17:40:48 -0400
To: bug-TermReadKey [...] rt.cpan.org
From: Josh Narins <josh.narins [...] gmail.com>
I did some more investigating. If test 8 is commented out, the END block also hangs under nohup. Term::ReadLine has a similar issue, but offers two environment variables to get around it, AUTOMATED_TESTING and PERL_MM_NONINTERACTIVE.
  Fri Jun 05 15:31:46 2015 jns [...] gellyfish.co.uk - Correspondence added
I thought this was fixed by testing whether /dev/tty was open to a terminal and skipping if not, but this was a bogus assumption so it isn't fixed in the current release. I've made https://github.com/jonathanstowe/TermReadKey/commit/ac5e11f8e58627370fd15faa82a932bbd7b74030 which will skip the whole t/02_terminal_functions.t if the STDIN isn't a terminal (and does stop if hanging when running in background,) but this is less than ideal and I may split the tests more before the next release. Obviously testing this without a terminal does not test properly whether the module works so is always going to be sub-optimal.
  Fri Jun 05 15:31:47 2015 The RT System itself - Status changed from 'new' to 'open'
  Fri Jun 05 15:31:53 2015 jns [...] gellyfish.co.uk - Status changed from 'open' to 'stalled'
  Sun Dec 04 15:23:42 2016 jns [...] gellyfish.co.uk - Status changed from 'stalled' to 'open'
  Sun Dec 04 15:24:04 2016 jns [...] gellyfish.co.uk - Correspondence added
I assume this is fixed now.
  Sun Dec 04 15:24:10 2016 jns [...] gellyfish.co.uk - Status changed from 'open' to 'resolved'