To: Stephane Chazelas Cc: vim-dev@vim.org Subject: Re: [Vim] toplevel configure script In-Reply-To: <19990921180654.A26070@raytheon.com> Fcc: outbox From: Bram Moolenaar ------------ Stephane Chazelas wrote: > > #! /bin/sh > > > > # This is just a stub for the Unix configure script, to provide support for > > # doing "./configure" in the top Vim directory. > > > > cd src && ./configure $* > > cd src && ./configure "$@" > > is better for the case one of the argument contains blanks. > > example: > /configure --prefix="/My home" Indeed, that works much better. Let's make a patch for this: Patch 5.5.001 Problem: Configure in the top directory did not pass on an argument with a space correctly. For example "./configure --previs="/My home". (Stephane Chazelas) Solution: Use '"$@"' instead of '$*' to pass on the arguments. Files: configure *** ../vim-5.5/configure Sun Sep 19 18:51:05 1999 --- configure Wed Sep 22 09:59:19 1999 *************** *** 3,6 **** # This is just a stub for the Unix configure script, to provide support for # doing "./configure" in the top Vim directory. ! cd src && ./configure $* --- 3,6 ---- # This is just a stub for the Unix configure script, to provide support for # doing "./configure" in the top Vim directory. ! cd src && ./configure "$@" *** ../vim-5.5/src/version.c Sun Sep 19 18:50:31 1999 --- src/version.c Wed Sep 22 10:04:36 1999 *************** *** 420,420 **** --- 420,421 ---- { /* Add new patch number below this line */ + 1, -- hundred-and-one symptoms of being an internet addict: 144. You eagerly await the update of the "Cool Site of the Day." --/-/---- Bram Moolenaar ---- Bram@moolenaar.net ---- Bram@vim.org ---\-\-- \ \ www.vim.org/iccf www.moolenaar.net www.vim.org / /