[perl #52894] [CAGE] use more File::Temp to avoid permission issues

2008-06-12 Thread James Keenan via RT
Since no failures are being reported by our smoke tests, I am marking this ticket resolved. kid51

Re: [perl #52894] [CAGE] use more File::Temp to avoid permission issues

2008-06-11 Thread ajr
> Well, if I had RTFM 'perldoc File::Temp' correctly, I wouldn't have > gotten the arguments wrong. > > Sorry for the confusion. > Sometimes, you just have to explain it to the bear before you understand. :-)* -- Email and shopping with the feelgood factor! 55% of income to good causes. http

[perl #52894] [CAGE] use more File::Temp to avoid permission issues

2008-06-09 Thread James Keenan via RT
Well, if I had RTFM 'perldoc File::Temp' correctly, I wouldn't have gotten the arguments wrong. The attached patch, applied in r28211, should correct matters. Sorry for the confusion. kid51 Index: config/auto/perldoc.pm === --- conf

[perl #52894] [CAGE] use more File::Temp to avoid permission issues

2008-06-09 Thread James Keenan via RT
This is proving to be a real PITA. While we're now passing t/steps/auto_perldoc.*.t on all systems, I'm not getting the proper result out of Configure.pl itself on Mac or Linux. Determining whether perldoc is installedno. Generating makefiles and other build

Re: [perl #52894] [CAGE] use more File::Temp to avoid permission issues

2008-06-09 Thread François Perrad
2008/6/9 James Keenan via RT <[EMAIL PROTECTED]>: > Based on our experience with File::Temp in the test suite, I think we > can avoid calling 'chmod' entirely (regardless of OS) and can simply use > File::Temp's functional interface (which avoids that warning I mentioned > in an earlier post). > >

Re: [perl #52894] [CAGE] use more File::Temp to avoid permission issues

2008-06-09 Thread chromatic
On Monday 09 June 2008 05:17:19 James Keenan via RT wrote: > (We use File::Temp extensively in the testing, but I can't recall a > situation where I absolutely had to 'chmod' a tempfile.  This may just > be an artifact of the previous way of doing things.) I agree; it seems unlikely that File::Te

[perl #52894] [CAGE] use more File::Temp to avoid permission issues

2008-06-09 Thread James Keenan via RT
Based on our experience with File::Temp in the test suite, I think we can avoid calling 'chmod' entirely (regardless of OS) and can simply use File::Temp's functional interface (which avoids that warning I mentioned in an earlier post). François: If the patch attached works for you on Windows, yo

Re: [perl #52894] [CAGE] use more File::Temp to avoid permission issues

2008-06-09 Thread Francois Perrad
James Keenan via RT a écrit : On Mon Jun 09 05:00:17 2008, [EMAIL PROTECTED] wrote: Determining whether perldoc is installed... step auto::perldoc died during execution: The fchmod function is unimplemented at config/auto/perldoc.pm line 42. On Windows, the Perl function 'chmod' works with f

[perl #52894] [CAGE] use more File::Temp to avoid permission issues

2008-06-09 Thread James Keenan via RT
On Mon Jun 09 05:00:17 2008, [EMAIL PROTECTED] wrote: > Determining whether perldoc is installed... > step auto::perldoc died during execution: The fchmod function is > unimplemented at config/auto/perldoc.pm line 42. > > On Windows, the Perl function 'chmod' works with filename (use POSIX > ch

[perl #52894] [CAGE] use more File::Temp to avoid permission issues

2008-06-09 Thread James Keenan via RT
On Mon Jun 09 05:00:17 2008, [EMAIL PROTECTED] wrote: > > since r28188, on Windows with ActiveState Perl 5.8.8 (build 822) , I've > the following error : > > Determining whether perldoc is installed... > step auto::perldoc died during execution: The fchmod function is > unimplemented at config

Re: [perl #52894] [CAGE] use more File::Temp to avoid permission issues

2008-06-09 Thread François Perrad
James Keenan via RT a écrit : On Sun Jun 08 17:57:56 2008, [EMAIL PROTECTED] wrote: Patch applied in r28188. Will resolve ticket if no one complains within 1-2 days. Thanks, Andy. I had no problems running the two existing step tests for this step via 'prove -v t/steps/auto_perldoc*.t. But

[perl #52894] [CAGE] use more File::Temp to avoid permission issues

2008-06-08 Thread James Keenan via RT
On Sun Jun 08 17:57:56 2008, [EMAIL PROTECTED] wrote: > Patch applied in r28188. Will resolve ticket if no one complains within > 1-2 days. Thanks, Andy. I had no problems running the two existing step tests for this step via 'prove -v t/steps/auto_perldoc*.t. But I got this warning when runnin

[perl #52894] [CAGE] use more File::Temp to avoid permission issues

2008-06-08 Thread James Keenan via RT
Patch applied in r28188. Will resolve ticket if no one complains within 1-2 days. Thanks, Andy.