On Thu, Oct 23, 2008 at 12:38 AM, John E. Malmberg <[EMAIL PROTECTED]> wrote: > Craig, Ken, > > Can you review this patch?
I'll take a look, though I can't promise to do so immediately. At first glance the readdir part doesn't look right; there shouldn't need to be another completely different way of setting up the macros for one extension, but I don't know offhand why the existing macros would not be visible in File::Glob. There are really three separate patches here, the symlink one, the readdir one, and the Cwd one. It would be helpful in the future if things like this could be submitted as three separate patches in three separate messages so they can be more easily reviewed, considered, tested, integrated, or (worst case) rolled back individually without affecting the unrelated parts. For Cwd, the patches should be submitted here: http://rt.cpan.org/Dist/Display.html?Name=PathTools though for VMS-specific things I do appreciate a CC to vmsperl. Thanks for your work, and for your patience while I try to find time to digest it.
