Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

lib/autodie/t/recv fails without TTY #9920

Open
p5pRT opened this issue Oct 20, 2009 · 6 comments
Open

lib/autodie/t/recv fails without TTY #9920

p5pRT opened this issue Oct 20, 2009 · 6 comments

Comments

@p5pRT
Copy link

p5pRT commented Oct 20, 2009

Migrated from rt.perl.org#69929 (status was 'open')

Searchable as RT69929$

@p5pRT
Copy link
Author

p5pRT commented Oct 20, 2009

From vmpdemo@gmail.com

lib/autodie/t/recv fails without TTY

This test assumes that STDIN is a TTY, and works fine as such.

Failed test 'recv dies on returning undef'
# at ../lib/autodie/t/recv.t line 45.
Use of uninitialized value $obj_name in concatenation (.) or string at
../lib/Test/More.pm line 599.
Use of uninitialized value $obj_name in concatenation (.) or string at
../lib/Test/More.pm line 618.
# Failed test ' isa autodie​::exception'
# at ../lib/autodie/t/recv.t line 46.
# isn't a class or reference
# Looks like you failed 2 tests of 8.

However remote compilation via SSH does not meet that criteria, so this fails​:
ssh user@​box "cd perl ; make test" </dev/null

And this hangs indefinitely​:
ssh user@​box "cd perl ; make test"

@p5pRT
Copy link
Author

p5pRT commented Oct 22, 2009

From @rgarcia

2009/10/20 Virtual Machine Provisioning Demo <perlbug-followup@​perl.org>​:

# New Ticket Created by  Virtual Machine Provisioning Demo
# Please include the string​:  [perl #69929]
# in the subject line of all future correspondence about this issue.
# <URL​: http​://rt.perl.org/rt3/Ticket/Display.html?id=69929 >

lib/autodie/t/recv fails without TTY

This test assumes that STDIN is a TTY, and works fine as such.

Failed test 'recv dies on returning undef'
#   at ../lib/autodie/t/recv.t line 45.
Use of uninitialized value $obj_name in concatenation (.) or string at
../lib/Test/More.pm line 599.
Use of uninitialized value $obj_name in concatenation (.) or string at
../lib/Test/More.pm line 618.
#   Failed test ' isa autodie​::exception'
#   at ../lib/autodie/t/recv.t line 46.
#      isn't a class or reference
# Looks like you failed 2 tests of 8.

However remote compilation via SSH does not meet that criteria, so this fails​:
ssh user@​box "cd perl ; make test" </dev/null

And this hangs indefinitely​:
ssh user@​box "cd perl ; make test"

That might be only a documentation (INSTALL document) issue. We have a
test_notty test target. runtests is supposed to detect that no tty is
present. However I can't reproduce this bug on my machine...

@p5pRT
Copy link
Author

p5pRT commented Oct 22, 2009

The RT System itself - Status changed from 'new' to 'open'

@p5pRT
Copy link
Author

p5pRT commented Oct 28, 2009

From vmpdemo@gmail.com

The TTY detection appears to work fine, as I can see make do "cd t &&
PERL_SKIP_TTY_TEST=1 ./perl TEST" - but I see no evidence that
lib/autodie/t/recv or anything else is trying to obey this detection.
Do you?

@p5pRT
Copy link
Author

p5pRT commented Oct 29, 2009

From vmpdemo@gmail.com

Also this bug did not exist in perl 5.8.7, so it's a regression
somewhere between that and 5.10.1.

@p5pRT
Copy link
Author

p5pRT commented Jul 21, 2016

From @dcollinsn

On Thu Oct 29 09​:20​:15 2009, vmpdemo@​gmail.com wrote​:

Also this bug did not exist in perl 5.8.7, so it's a regression
somewhere between that and 5.10.1.

I routinely build through SSH or through scripts that leave Configure and make without a proper TTY, regardless, I've never seen this test failure. In fact, it seems like no one has been able to reproduce it. Is there any objection to this ticket being closed (or is there any more information that someone can add, for people trying to diagnose this in the future)?

--
Respectfully,
Dan Collins

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants