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.

Sign up for GitHub

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

Jump to bottom

Rt 120862 no dot in default inc #6

Merged
merged 2 commits into from Apr 18, 2017
Merged

Rt 120862 no dot in default inc #6

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
70acf12
perl-5.26.0-readiness. Account for absence of '.' in default @INC.
jkeenan Apr 16, 2017
6d1d8c4
Provide a .gitignore.
jkeenan Apr 16, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
11 changes: 11 additions & 0 deletions .gitignore
View file
Open in desktop
@@ -0,0 +1,11 @@
/MYMETA.json
/MYMETA.yml
/Makefile
/SSLeay.bs
/SSLeay.c
/SSLeay.o
/blib/
/openssl-version
/openssl-version.o
/pm_to_blib
/test.config
1 change: 1 addition & 0 deletions Makefile.PL
View file
Open in desktop
Expand Up @@ -8,6 +8,7 @@ use ExtUtils::MakeMaker;
use File::Spec::Functions qw( catfile rel2abs );
use Getopt::Long qw( GetOptionsFromArray );

push @INC, '.';
unless( require( catfile qw(inc IO Interactive Tiny.pm) ) ) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Surely, the right thing to do here would be to augment the call to "require" to start with "./", as modifying @INC for the entirety of Makefile.PL is not actually necessary here.

die 'Your distribution is incomplete: Failed to load bundled IO::Interactive::Tiny';
}
Expand Down