- Notifications
You must be signed in to change notification settings - Fork 602
Msys2 upstream #18492
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
base: blead
Are you sure you want to change the base?
Msys2 upstream #18492
Changes from all commits
9ea625c 22ab84f 35122c6 a6a2c17 839250c 6ecbcb1 5a51a59 dcc3e7f 7a11631 a071b87 b4911b3 File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,33 @@ | ||
| package ExtUtils::CBuilder::Platform::msys; | ||
| Contributor There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. If this is exactly the same as | ||
| | ||
| use warnings; | ||
| use strict; | ||
| use File::Spec; | ||
| use ExtUtils::CBuilder::Platform::Unix; | ||
| | ||
| our $VERSION = '0.280234'; # VERSION | ||
| our @ISA = qw(ExtUtils::CBuilder::Platform::Unix); | ||
| | ||
| # TODO: If a specific exe_file name is requested, if the exe created | ||
| # doesn't have that name, we might want to rename it. Apparently asking | ||
| # for an exe of "foo" might result in "foo.exe". Alternatively, we should | ||
| # make sure the return value is correctly "foo.exe". | ||
| # C.f http://rt.cpan.org/Public/Bug/Display.html?id=41003 | ||
| sub link_executable { | ||
| my $self = shift; | ||
| return $self->SUPER::link_executable(@_); | ||
| } | ||
| | ||
| sub link { | ||
| my ($self, %args) = @_; | ||
| | ||
| my $lib = $self->{config}{useshrplib} ? 'libperl.dll.a' : 'libperl.a'; | ||
| $args{extra_linker_flags} = [ | ||
| File::Spec->catfile($self->perl_inc(), $lib), | ||
| $self->split_like_shell($args{extra_linker_flags}) | ||
| ]; | ||
| | ||
| return $self->SUPER::link(%args); | ||
| } | ||
| | ||
| 1; | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| | @@ -90,7 +90,7 @@ package DynaLoader; | |
| # Tim.Bunce@ig.co.uk, August 1994 | ||
| | ||
| BEGIN { | ||
| our $VERSION = '1.54'; | ||
| our $VERSION = '1.55'; | ||
| } | ||
| | ||
| # Note: in almost any other piece of code "our" would have been a better | ||
| | @@ -484,6 +484,9 @@ sub dl_findfile { | |
| <<$^O-eq-cygwin>> | ||
| push(@names,"cyg$_.$dl_so") unless m:/:; | ||
| <</$^O-eq-cygwin>> | ||
| <<$^O-eq-msys>> | ||
| Contributor There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This commit should also update the version to Author There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Done | ||
| push(@names,"msys-$_.$dl_so") unless m:/:; | ||
| <</$^O-eq-msys>> | ||
| push(@names,"lib$_.$dl_so") unless m:/:; | ||
| push(@names, $_); | ||
| } | ||
| | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,7 +1,16 @@ | ||
| use strict; | ||
| use ExtUtils::MakeMaker; | ||
| | ||
| our @LIBS; | ||
| if( $^O eq 'msys' ) { | ||
| @LIBS = ("-lgdbm -lgdbm_compat"); | ||
| } else { | ||
| @LIBS = ("-L/usr/local/lib -lndbm", "-ldbm -lucb"); | ||
| }; | ||
| | ||
| WriteMakefile( | ||
| NAME=> 'NDBM_File', | ||
| LIBS => ["-L/usr/local/lib -lndbm", "-ldbm -lucb"], | ||
| Contributor There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. cygwin uses a Contributor There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I suspect this entire patch isn't needed, can we double check that? | ||
| LIBS => \@LIBS, | ||
| XSPROTOARG => '-noprototypes', # XXX remove later? | ||
| VERSION_FROM => 'NDBM_File.pm', | ||
| ); | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,7 +1,16 @@ | ||
| use strict; | ||
| use ExtUtils::MakeMaker; | ||
| | ||
| our @LIBS; | ||
| if( $^O eq 'msys' ) { | ||
| @LIBS = ("-lgdbm -lgdbm_compat"); | ||
| } else { | ||
| @LIBS = ("-ldbm -lucb"); | ||
| }; | ||
| | ||
| WriteMakefile( | ||
| NAME => 'ODBM_File', | ||
| LIBS => ["-ldbm -lucb"], | ||
| LIBS => \@LIBS, | ||
| XSPROTOARG => '-noprototypes', # XXX remove later? | ||
| VERSION_FROM => 'ODBM_File.pm', | ||
| ); |
Uh oh!
There was an error while loading. Please reload this page.