On Sep 28, 2007, at 11:42 PM, Craig A. Berry wrote:

It applied cleanly against blead, where I've checked it in as 31995.
I wonder if we've had some drift between blead and the independent
version.

Indeed we have, here are the VMS-related outstanding differences. Could someone verify that the blead version is correct, and let me know who to assign credit to in the Changes file?

 -Ken

diff -ur /Users/ken/Downloads/perl/bleadperl/lib/Module/Build/Base.pm lib/Module/Build/Base.pm --- /Users/ken/Downloads/perl/bleadperl/lib/Module/Build/Base.pm 2007-09-28 23:43:35.000000000 -0500
+++ lib/Module/Build/Base.pm    2007-09-28 20:57:28.000000000 -0500
@@ -447,7 +446,10 @@
   my $exe = $c->get('exe_ext');
   foreach my $thisperl ( @potential_perls ) {

-    if (defined $exe and $proto->os_type ne 'VMS') {
+    if ($proto->is_vmsish) {
+      # VMS might have a file version at the end
+      $thisperl .= $exe unless $thisperl =~ m/$exe(;\d+)?$/i;
+    } elsif (defined $exe) {
       $thisperl .= $exe unless $thisperl =~ m/$exe$/i;
     }

diff -ur /Users/ken/Downloads/perl/bleadperl/lib/Module/Build/ Platform/VMS.pm lib/Module/Build/Platform/VMS.pm --- /Users/ken/Downloads/perl/bleadperl/lib/Module/Build/Platform/ VMS.pm 2007-09-28 23:43:35.000000000 -0500
+++ lib/Module/Build/Platform/VMS.pm    2007-09-28 20:51:56.000000000 -0500
@@ -195,9 +195,8 @@

   # Need to create with the same name as DynaLoader will load with.
   if (defined &DynaLoader::mod2fname) {
- my $file = $$spec{module_name} . '.' . $self->{config}->get ('dlext');
-    $file =~ tr/:/_/;
-    $file = DynaLoader::mod2fname([$file]);
+    my $file = DynaLoader::mod2fname([$$spec{base_name}]);
+    $file .= '.' . $self->{config}->get('dlext');
     $$spec{lib_file} = File::Spec->catfile($$spec{archdir}, $file);
   }

@@ -220,36 +219,6 @@
   return $result;
 }

-=item dist_dir
-
-Inherit the standard version but replace embedded dots with underscores because
-a dot is the directory delimiter on VMS.
-
-=cut
-
-sub dist_dir {
-  my $self = shift;
-
-  my $dist_dir = $self->SUPER::dist_dir;
-  $dist_dir =~ s/\./_/g;
-  return $dist_dir;
-}
-
-=item man3page_name
-
-Inherit the standard version but chop the extra manpage delimiter off the front if
-there is one.  The VMS version of splitdir('[.foo]') returns '', 'foo'.
-
-=cut
-
-sub man3page_name {
-  my $self = shift;
-
-  my $mpname = $self->SUPER::man3page_name( shift );
-  my $sep = $self->manpage_separator;
-  $mpname =~ s/^$sep//;
-  return $mpname;
-}

 =item expand_test_dir

@@ -275,9 +244,7 @@

 =head1 AUTHOR

-Michael G Schwern <[EMAIL PROTECTED]>
-Ken Williams <[EMAIL PROTECTED]>
-Craig A. Berry <[EMAIL PROTECTED]>
+Michael G Schwern <[EMAIL PROTECTED]>, Ken Williams <[EMAIL PROTECTED]>

 =head1 SEE ALSO


Reply via email to