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

Test::Stream $root not initialized when END block runs #592

Closed
dolmen opened this issue May 6, 2015 · 4 comments
Closed

Test::Stream $root not initialized when END block runs #592

dolmen opened this issue May 6, 2015 · 4 comments

Comments

@dolmen
Copy link
Contributor

dolmen commented May 6, 2015

With Test::Stream 1.301001_110:

$ pmvers Test::Stream
1.30100111
Can't call method "concurrency_driver" on an undefined value at .../Test/Stream.pm line 25.
END failed--call queue aborted at /usr/bin/pmvers line 42.

My /usr/bin/pmvers on Ubuntu 14.04 is exactly the same (except $VERSION) as pmvers from distribution pmtools.

@dolmen dolmen changed the title Testy::Stream $root not initialized Test::Stream $root not initialized May 6, 2015
@dolmen dolmen changed the title Test::Stream $root not initialized Test::Stream $root not initialized when END block runs May 6, 2015
@dolmen
Copy link
Contributor Author

dolmen commented May 6, 2015

Simpler test:

$ perl -mTest::Stream -e0
Can't call method "concurrency_driver" on an undefined value at /home/omengue/.plenv/versions/5.10.1/lib/perl5/5.10.1/Test/Stream.pm line 25.
END failed--call queue aborted.

As -M instead of -m solves the issue, I suppose that import is initializing $root.

@karenetheridge
Copy link
Member

It seems odd that so much code is being run merely from the loading of the module. Is this necessary?

@exodist
Copy link
Member

exodist commented May 6, 2015

much of it is not necessary. I am fixing this along with other issues. this is related to the email I sent perl-qa about the work Test-Stream still needs :-)

@exodist
Copy link
Member

exodist commented May 10, 2015

This has been addressed in the new repo we are using for the Test-Stream components, closing here.

@exodist exodist closed this as completed May 10, 2015
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

3 participants