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

t/run/switches.t fails under miniperl #16159

Closed
p5pRT opened this issue Sep 20, 2017 · 7 comments
Closed

t/run/switches.t fails under miniperl #16159

p5pRT opened this issue Sep 20, 2017 · 7 comments

Comments

@p5pRT
Copy link

p5pRT commented Sep 20, 2017

Migrated from rt.perl.org#132138 (status was 'resolved')

Searchable as RT132138$

@p5pRT
Copy link
Author

p5pRT commented Sep 20, 2017

From @jkeenan

As documented in https://rt-archive.perl.org/perl5/Ticket/Display.html?id=132137,
'miniperl' is not well documented, particularly underneath the 'pod/'
directory where one would expect such documentation.

Consequently, I can't definitively say why the following test fails, but
I suspect that modules cannot be 'require'd under 'miniperl'.

#####
$ ./miniperl t/run/switches.t

1..137
ok 1 - -0
ok 2 - -0 after a -l
ok 3 - -0 before a -l
...
ok 110 - no warnings/errors without backup extension
ok 111 - correct output without backup extension
Can't locate File/Spec.pm in @​INC (you may need to install the
File​::Spec module) (@​INC contains​: ../lib) at t/run/switches.t line 424.
# Looks like you planned 137 tests but ran 111.

#####

Attached is a 'grep' of t/run/switches.t showing where File​::Spec is
required and invoked.

What is the recommended fix?

Thank you very much.
Jim Keenan

@p5pRT
Copy link
Author

p5pRT commented Sep 20, 2017

From @jkeenan

424​: require File​::Spec;
425​: my $work = File​::Spec->catfile("inplacetmp", "foo");
559​: my $abs_work = File​::Spec->rel2abs($work);
587​: my $check = File​::Spec->catfile("inplacetmp", "check");
632​: unlink map File​::Spec->catfile("inplacetmp", $_), @​names;

@p5pRT
Copy link
Author

p5pRT commented Sep 21, 2017

From @tonycoz

On Wed, 20 Sep 2017 16​:12​:44 -0700, jkeenan@​pobox.com wrote​:

As documented in https://rt-archive.perl.org/perl5/Ticket/Display.html?id=132137,
'miniperl' is not well documented, particularly underneath the 'pod/'
directory where one would expect such documentation.

Consequently, I can't definitively say why the following test fails, but
I suspect that modules cannot be 'require'd under 'miniperl'.

#####
$ ./miniperl t/run/switches.t

1..137
ok 1 - -0
ok 2 - -0 after a -l
ok 3 - -0 before a -l
...
ok 110 - no warnings/errors without backup extension
ok 111 - correct output without backup extension
Can't locate File/Spec.pm in @​INC (you may need to install the
File​::Spec module) (@​INC contains​: ../lib) at t/run/switches.t line 424.
# Looks like you planned 137 tests but ran 111.

#####

Attached is a 'grep' of t/run/switches.t showing where File​::Spec is
required and invoked.

Thanks, fixed by 8200f80.

Tony

@p5pRT
Copy link
Author

p5pRT commented Sep 21, 2017

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

@p5pRT
Copy link
Author

p5pRT commented Sep 21, 2017

@tonycoz - Status changed from 'open' to 'pending release'

@p5pRT
Copy link
Author

p5pRT commented Jun 23, 2018

From @khwilliamson

Thank you for filing this report. You have helped make Perl better.

With the release yesterday of Perl 5.28.0, this and 185 other issues have been
resolved.

Perl 5.28.0 may be downloaded via​:
https://metacpan.org/release/XSAWYERX/perl-5.28.0

If you find that the problem persists, feel free to reopen this ticket.

@p5pRT
Copy link
Author

p5pRT commented Jun 23, 2018

@khwilliamson - Status changed from 'pending release' to 'resolved'

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

1 participant