* Vincent Lefevre wrote on Thu, Aug 07, 2008 at 05:30:20PM CEST:
> On 2008-08-07 16:48:24 +0200, Ralf Wildenhues wrote:
> > > The following would be cleaner and completely local (so, no risks
> > > of side effects):
> > > 
> > >   require File::Copy;
> > >   File::Copy::move (...
> 
> Well, not completely local, because once the require has been evaluated,
> its effects remain, but without the "import", you can't use "move" alone.

Hmm.  Not sure whether any cleanup patch is currently warranted then.

> In fact, the main advantage of this solution is to be able to use it in
> an eval, so that it is not fatal; and if the require is never evaluated,
> the module isn't needed (and at the same occasion, this should then be
> faster). For instance, File::Copy isn't available with perl 5.6.1 on the
> Zaurus (though the user may install his own copy).

AFAICS Autoconf currently requires File::Copy.  Making it cope without
would be a separate task.

I guess the only cleanup task left over now would be to add a 'use
File::Copy' to the atomic-rename patch.  Otherwise, IMVHO this bug
can be closed (but I'll leave that at Ben's discretion to decide).

> > Is this also just as portable to older perl versions?
> 
> It works at least with perl 5.005_03 (1999). I don't have working
> earlier versions.

That's fine for Autoconf.

Cheers,
Ralf



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to