diff options
| author | Vladislav Vaintroub <vvaintroub@mysql.com> | 2010-02-23 12:32:57 +0100 |
|---|---|---|
| committer | Vladislav Vaintroub <vvaintroub@mysql.com> | 2010-02-23 12:32:57 +0100 |
| commit | a4af6e87083aec7f27c70ccc6c669e3d3fbcaf0d (patch) | |
| tree | 4f9c1f81a4ebb7e603667f81a216eec55951653d /BUILD | |
| parent | 71f8615fd66c03af06c8b8547153d35494c1ac2a (diff) | |
| download | mariadb-git-a4af6e87083aec7f27c70ccc6c669e3d3fbcaf0d.tar.gz | |
Bug #51414: Arguments with embedded spaces are not correctly handled by configure wrapper.
The bug was that ./configure was passing paramers to subscripts as $@, and to handle embedded spaces it needs
to be quoted as "$@".
This resulting into a bug when ./configure was called e.g with CFLAGS='-m64 -Xstrconst'..
Additionally, fixed cmake/configure.pl did not handle environment variables passed on the command line.
this is fixed in this push
Diffstat (limited to 'BUILD')
| -rw-r--r-- | BUILD/choose_configure.sh | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/BUILD/choose_configure.sh b/BUILD/choose_configure.sh index 71243ea09b6..476b8b51657 100644 --- a/BUILD/choose_configure.sh +++ b/BUILD/choose_configure.sh @@ -7,8 +7,8 @@ cmake -P cmake/check_minimal_version.cmake >/dev/null 2>&1 || HAVE_CMAKE=no perl --version >/dev/null 2>&1 || HAVE_CMAKE=no if test "$HAVE_CMAKE" = "no" then - sh ./configure.am $@ + sh ./configure.am "$@" else - perl ./cmake/configure.pl $@ + perl ./cmake/configure.pl "$@" fi |