Re: [patch@31780] fixes for cpanplus on VMS

2007-10-01 Thread Craig A. Berry
At 6:58 PM -0500 10/1/07, John E. Malmberg wrote: >Glenn Linderman wrote: > >> >>Perhaps should be... >> >>BEGIN { %old_env = %ENV if $^O eq 'VMS' } >> >>>+END { %ENV = %old_env if $^O eq 'VMS' } >>>+} >>>+ > >No. That could be catastrophic. Indeed. I'm not sure what all the fuss is ab

Re: [patch@31780] fixes for cpanplus on VMS

2007-10-01 Thread John E. Malmberg
Glenn Linderman wrote: Perhaps should be... BEGIN { %old_env = %ENV if $^O eq 'VMS' } +END { %ENV = %old_env if $^O eq 'VMS' } +} + No. That could be catastrophic. Much of what is in the VMS %ENV table is logical names. Now VMS logical names behave more like symbolic links and

Re: [patch@31780] fixes for cpanplus on VMS

2007-10-01 Thread John E. Malmberg
Jos I. Boumans wrote: On 03 Sep 2007, at 21:25, John E. Malmberg wrote: Finally, each test run for CPANPLUS is appending a path on to $ENV{PATH} AND $ENV{PERL5LIB}. As near as I can tell, it intends to clean these up, but that is not working on VMS. I have not figured out why yet. It does n

Re: [patch@31780] fixes for cpanplus on VMS

2007-10-01 Thread Jos I. Boumans
On 03 Sep 2007, at 21:25, John E. Malmberg wrote: Finally, each test run for CPANPLUS is appending a path on to $ENV {PATH} AND $ENV{PERL5LIB}. As near as I can tell, it intends to clean these up, but that is not working on VMS. I have not figured out why yet. It does not take too many run

Re: [patch@31780] fixes for cpanplus on VMS

2007-10-01 Thread Jos I. Boumans
On 03 Sep 2007, at 21:25, John E. Malmberg wrote: Jos I. Boumans wrote: On Sep 3, 2007, at 12:23 AM, John E. Malmberg wrote: While there are still some CPANPLUS tests that are not working on VMS, these patches improve things. I'll review these patches shortly, thanks for sending them john.

Re: [patch@31750] CPANPLUS extract.pm fixes for VMS.

2007-10-01 Thread Jos I. Boumans
On 01 Sep 2007, at 00:53, Craig A. Berry wrote: At 12:21 AM -0500 8/31/07, John E. Malmberg wrote: In lib/CPANPLUS/Internals/Extract.pm, VMS needs to use catfile to deal with combining path components that could include a file name component. If C will work for all platforms, then the tes

Re: remaining M::B issues on VMS (was Re: [patch@31988] RevisedModule::Build fixes for VMS.)

2007-10-01 Thread John E. Malmberg
John E. Malmberg wrote: John E. Malmberg wrote: Craig A. Berry wrote: At 5:48 PM +0100 9/29/07, Nicholas Clark wrote: At least one of the reasons ppm.t fails is that ppm is attempting to do things with the tar format that may never work on VMS. For example, you can't store object code i