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

Split PERL_MB_OPT as on Unix #13

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Split PERL_MB_OPT as on Unix #13

wants to merge 1 commit into from

Conversation

Leont
Copy link
Member

@Leont Leont commented Jul 20, 2015

Currently in Module::Build and Module::Build::Tiny we're using platform specific splitting for PERL_MB_OPT, which means using Text::ParseWords on all platforms except Windows, where a custom and widely presumed buggy function is used.

There is no need for it to do this (it's not handled by a shell). local::lib isn't doing anything Windows specific there either as far as I understand.

Hence, I propose we change the behavior in the spec and in the implementations to using Unix semantics even on Windows, just like ExtUtils::MakeMaker is with PERL_MM_OPT.

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.

1 participant