Eric Wilhelm wrote:
# from Michael G Schwern
# on Thursday 14 February 2008 00:54:

There's already a module to do this, File::pushd, and the sentinel
approach is a little wonky.  There's also File::chdir for another way
to do it.  Both will happily accept a Path::Class::Dir so I don't see
what's to be gained.

So. Did we decide that I should revert this patch and add a dependency to File::pushd or what?

Module installers cannot have dependencies, especially on something that use Module::Build. :)


Should I just call them chdir and unchdir?

Go with the patch I proposed to implement a simplified pushd/local_chdir() for Module::Build internal use.


--
29. The Irish MPs are not after “Me frosted lucky charms”.
    -- The 213 Things Skippy Is No Longer Allowed To Do In The U.S. Army
           http://skippyslist.com/list/

Reply via email to