Subject: Re: [Vim] toplevel configure script
From: Bram Moolenaar <[email-masked]>
To: Stephane Chazelas <[email-masked]>
Cc: [email-masked]

------------

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 ---- [email-masked] ---- [email-masked] ---\-\--
  \ \    www.vim.org/iccf      www.moolenaar.net       www.vim.org    / /