perl Makefile.PL

[takulark]

Description / Steps to reproduce the issue

perl Makefile.PL can't be done.

perl -MCPAN -e shell
cpan[1]>reload index
cpan[2]>make ExtUtils::MakeMaker
shell cmd <-stop

ctrl+c
exit <-Force quit with

cd ~/.cpan/build/ExtUtils-MakeMaker-7.70-0
patches(Needs a patch for each update)
perl Makefile.PL INSTALLDIRS=site
make install

Although it is a modification of Config_heavy.pl, it can be omitted.
+++ lib/ExtUtils/MM_Win32.pm
@@ -40,7 +40,14 @@
        my $BORLAND = $config->{cc} =~ /\bbcc/i ? 1 : 0;
        my $GCC     = $config->{cc} =~ /\bgcc\b/i ? 1 : 0;
        my $MSVC    = $config->{cc} =~ /\b(?:cl|icl)/i ? 1 : 0; # MSVC can come as clarm.exe, icl=Intel C
-
+#      for my $cv (keys %$config) {
+#          next if(!exists $config->{$cv});
+#          my $v=$config->{$cv};
+#          next if(!defined $v);
+#          if($v=~/\w:\\/i){
+#              $v=~ s/\\/\//g;
+#              $config->{$cv}=$v;
+#          }}
        return ( $BORLAND, $GCC, $MSVC );
 }

need this.
@@ -120,7 +127,7 @@

 =cut

-sub init_DIRFILESEP {
+sub init_DIRFILESEPd {
     my($self) = shift;

     # The ^ makes sure its not interpreted as an escape in nmake
@@ -139,7 +146,7 @@
 sub init_tools {
     my ($self) = @_;

-    $self->{NOOP}     ||= 'rem';
+#    $self->{NOOP}     ||= 'rem';
     $self->{DEV_NULL} ||= '> NUL';

     $self->{FIXIN}    ||= $self->{PERL_CORE} ?
@@ -226,7 +233,7 @@

 =cut

-sub specify_shell {
+sub specify_shelld {
     my $self = shift;
     return '' unless $self->is_make_type('gmake');
     "\nSHELL = $ENV{COMSPEC}\n";

Expected behavior

File::Spec::Win32 & ExtUtils::MM_Win32
is made for Command Prompt (CMD).
\ is not a dir separator on bash.
It cannot be used as is, so a patch is required.

+++ File/Spec.pm
@@ -15,8 +15,8 @@
              cygwin  => 'Cygwin',
              amigaos => 'AmigaOS');

-
-my $module = $module{$^O} || 'Unix';
+my $os=$^O;$os='cygwin' if $ENV{'MSYSTEM'} eq "MINGW64";
+my $module = $module{$os} || 'Unix';

 require "File/Spec/$module.pm";
 our @ISA = ("File::Spec::$module");

sample.
before patch
-INST_AUTODIR = $(INST_LIB)\auto$(FULLEXT)
-INST_ARCHAUTODIR = $(INST_ARCHLIB)\auto$(FULLEXT)
After patch
+INST_AUTODIR = $(INST_LIB)/auto/$(FULLEXT)
+INST_ARCHAUTODIR = $(INST_ARCHLIB)/auto/$(FULLEXT)

Actual behavior

Verification

• I have verified that my MSYS2 is up-to-date before submitting the report (see https://www.msys2.org/docs/updating/)

Windows Version

MINGW64_NT-10.0-19045

MINGW environments affected

• MINGW64
• MINGW32
• UCRT64
• CLANG64
• CLANG32
• CLANGARM64

Are you willing to submit a PR?

No response

[Biswa96]

Please fill up the issue template and explain the issue.

[zmughal]

It looks like this is trying to patch ExtUtils::MakeMaker, but I'm not sure why. Can you explain what you are using this for? These aren't the kinds of issues I've had with ExtUtils::MakeMaker under MSYS2's MinGW64.

[takulark]

Edited for clarity.
MSYS is patched.

[Biswa96]

What issue are you trying to solve?

[takulark]

cmd starts and ends.
Even if I fix the makefile here
Because the dir separator is different
cp lib/yyy/xxx.pm blib\yyy\xxx.pm ->blibyyyxxx.pm
become.

[zmughal]

What specifically are you trying to install and how are you running it? Info such as:

which perl
echo $PATH
echo $MSYSTEM
echo $SHELL

would help as well.

[takulark]

Discovered that XML::Parser is required.
perl -MCPAN -e shell,Trying to get install in.
OS is judgement only by $^O. not considering the shell.
Because it is also necessary for module creation of ActivePerl.

which perl
echo $PATH
echo $MSYSTEM
echo $SHELL
/mingw64/bin/perl
/mingw64/bin:/mingw64/bin/site_perl/5.32.1:/mingw64/bin/vendor_perl:/mingw64/bin/core_perl:/usr/local/bin:/usr/bin:/bin:/c/Windows/System32:/c/Windows
MINGW64
/usr/bin/bash

[revelator]

i think i know what he means, the XML::Parser module is missing from the mingw-w64-perl packages but is needed by other build packages. i ran into this one myself so i installed it to mingw-w64-perl and no problems since.
it builds fine so could we add it to the mingw-w64-perl packages ?.

[revelator]

made a package for it ->
mingw-w64-perl-xml-parser.zip

feel free to add it to the repo if usefull.

[takulark]

How about changing $^O?
For example MSWin32sh or MSWin64.
I don't need the patch anymore,
but will changing it cause other problems?

[revelator]

not a perl guru myself but i guess those two might be for msvc ?

[Kreijstal]

Had this issue too compiling sftl, see newsboat/stfl#7 (review)