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 failures with perl 5.14.4 (possibly others) #15

Closed
leejo opened this issue Aug 20, 2015 · 4 comments
Closed

test failures with perl 5.14.4 (possibly others) #15

leejo opened this issue Aug 20, 2015 · 4 comments

Comments

@leejo
Copy link

leejo commented Aug 20, 2015

You're missing 5.12, 5.14, and 5.16 from your travis so you're probably not seeing these.

/Users/leejohnson/working/Pod-Markdown > prove -Ilib t/encoding.t
t/encoding.t .. 1/? Can't locate object method "detected_encoding" via package "Pod::Markdown" at /Volumes/code_partition/Pod-Markdown/lib/Pod/Markdown.pm line 746.
# Tests were run but no plan was declared and done_testing() was not seen.
t/encoding.t .. Dubious, test returned 255 (wstat 65280, 0xff00)
All 12 subtests passed
        (less 1 skipped subtest: 11 okay)

Test Summary Report
-------------------
t/encoding.t (Wstat: 65280 Tests: 12 Failed: 0)
  Non-zero exit status: 255
  Parse errors: No plan found in TAP output
Files=1, Tests=12,  0 wallclock secs ( 0.03 usr  0.01 sys +  0.08 cusr  0.01 csys =  0.13 CPU)
Result: FAIL

perl:

/Users/leejohnson/working/Pod-Markdown > perl --version

This is perl 5, version 14, subversion 4 (v5.14.4) built for darwin-2level

t/pod2markdown.t also fails with the same error, which can be bisected:

94f488beb143c251e116b0cc133745545b9a125b is the first bad commit
commit 94f488beb143c251e116b0cc133745545b9a125b
Author: Randy Stauner <randy@magnificent-tears.com>
Date:   Fri Aug 14 20:45:26 2015 -0700

    Add new encoding attributes as command line options

:100644 100644 63ef39b875ad35fa697fa8f17004624ca041b2c5 c7e26360cffc75968863eb9a12b166074e9e5033 M      Changes
:040000 040000 24266683eb70abccc1915e5c775b8be8ab323bac 8e042716bd31158aafd763400894679b8b6a2abc M      bin
:040000 040000 fe0679d4a846602efe7eff656dae10ba759e2c1e b84c3d9087d6f607844fd348734acd7c2ab26452 M      corpus
:100644 100644 35083eb883559e22e3f4098eef1fa2669e253702 3c527f93e431554de85abcbe8e5280994cdb9d7a M      dist.ini
:040000 040000 ab8d8cef0606739e2ab0fec858d1928285fbe676 4f1d0698b67fcf8e9b5afa0c4602c7fa6bf5319f M      t
bisect run success
@leejo
Copy link
Author

leejo commented Aug 20, 2015

I just noticed your bug tracker is still pointing at RT in the dist. I'm going to assume you'll see this before raising the issue on RT as well.

@rwstauner
Copy link
Owner

Looks I need to check what version of Pod::Simple added detected_encoding.
Thanks for the heads up, cpantesters is down so i haven't been able to see the reports.

@rwstauner
Copy link
Owner

3.002 released.
Thanks for the help!

rwstauner added a commit that referenced this issue Aug 21, 2015
@leejo
Copy link
Author

leejo commented Aug 21, 2015

Cool! Thanks for the quick fix.

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