You're right. -d seems better. Index: BuildOptions.pm =================================================================== --- BuildOptions.pm (revision 452813) +++ BuildOptions.pm (working copy) @@ -121,6 +121,11 @@ require Win32; $val = Win32::GetShortPathName($val); } + + if (!$val || !-d $val) { + error "MP_AP_PREFIX must point to a valid directory."; + die "\n"; + } }
if ($table->{$key}->{append}){ "Paul Johnson" <[EMAIL PROTECTED]> wrote in message news:[EMAIL PROTECTED] > On Wed, Oct 04, 2006 at 01:49:21PM +0300, Nikolay Ananiev wrote: > > > This patch adds a nice error message. > > > > Index: BuildOptions.pm > > =================================================================== > > --- BuildOptions.pm (revision 452813) > > +++ BuildOptions.pm (working copy) > > @@ -121,6 +121,11 @@ > > require Win32; > > $val = Win32::GetShortPathName($val); > > } > > + > > + if (!$val || !-e $val) { > > + error "MP_AP_PREFIX must point to a valid directory."; > > + die "\n"; > > + } > > Perhaps -d instead? > > -- > Paul Johnson - [EMAIL PROTECTED] > http://www.pjcj.net --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]