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

Push '.' to @INC when custom builder is used #216

Merged
merged 2 commits into from
Feb 20, 2017
Merged

Conversation

syohex
Copy link
Collaborator

@syohex syohex commented Feb 9, 2017

@INC does not contain current directory from Perl 5.25.9.
This is related to #215.

@inc does not contain current directory from Perl 5.25.9.
@skaji
Copy link
Collaborator

skaji commented Feb 9, 2017

I have confirmed that with this change,
perl compiled with -Ddefault_inc_excludes_dot could configure/build/test/install Compiler-Lexer and Hash-FieldHash.

@@ -75,6 +75,9 @@ use strict;
use warnings;
use utf8;

? if ( $project->build_class ne 'Module::Build' ) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

if minil.toml is

name = "Foo"
module_maker = "ModuleBuild"

[XSUtil]
needs_compiler_c99 = 1

then build_class is Module::Build::XSUtil. I think we can exclude 'Module::Build::XSUtil' too.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

3c7f080

Fixed

@toddr
Copy link

toddr commented Feb 9, 2017

I don't think this will fix it. I think this is going to require PERL_USE_UNSAFE_INC but I'll need to test it later. If I could see a Build.PL generated from this, I could tell you if it works. I'll see if I can figure out how to re-generate the one for Mouse.

@syohex
Copy link
Collaborator Author

syohex commented Feb 13, 2017

Thanks for comments. I'll see them later.

@skaji
Copy link
Collaborator

skaji commented Feb 18, 2017

LGTM

@toddr Could you test this please?

@toddr
Copy link

toddr commented Feb 20, 2017

@skaji Minilla now builds the correct Build.PL for Mouse. Once this is merged and released, I'll let the 5 dists that need a new Build.PL to update Minilla and release.

@syohex syohex merged commit 3dfc4b7 into master Feb 20, 2017
@syohex syohex deleted the push-current-dir-to-INC branch February 20, 2017 03:36
@syohex
Copy link
Collaborator Author

syohex commented Feb 20, 2017

Thanks for reviewing. I have merged.

syohex added a commit that referenced this pull request Feb 20, 2017
Changelog diff is:

diff --git a/Changes b/Changes
index 8a31200..c3261a5 100644
--- a/Changes
+++ b/Changes
@@ -1,6 +1,8 @@
 Revision history for Perl extension Minilla

 {{$NEXT}}
+
+v3.0.10 2017-02-20T16:30:43Z
     - Set '.' to @inc when using custom builder for newer Perl(#216)

 v3.0.9 2017-01-12T04:30:41Z
skaji added a commit that referenced this pull request May 22, 2017
Changelog diff is:

diff --git a/Changes b/Changes
index c3261a5..8e09edd 100644
--- a/Changes
+++ b/Changes
@@ -2,6 +2,10 @@ Revision history for Perl extension Minilla

 {{$NEXT}}

+v3.0.11 2017-05-22T10:22:00Z
+    - Fix tests for no-dot-in-inc (eserte #220, #221)
+    - Now Minilla requires perl 5.10.1+ (#222)
+
 v3.0.10 2017-02-20T16:30:43Z
     - Set '.' to @inc when using custom builder for newer Perl(#216)
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

Successfully merging this pull request may close these issues.

3 participants