You are viewing a plain text version of this content. The canonical link for it is here.
Posted to apache-bugdb@apache.org by Yves Arrouye <yv...@apple.com> on 1998/08/18 19:36:29 UTC

config/2866: config.status should use "$@" rather than $* so it can pass arguments properly

>Number:         2866
>Category:       config
>Synopsis:       config.status should use "$@" rather than $* so it can pass arguments properly
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    apache
>State:          open
>Class:          sw-bug
>Submitter-Id:   apache
>Arrival-Date:   Tue Aug 18 10:40:00 PDT 1998
>Last-Modified:
>Originator:     yves@apple.com
>Organization:
apache
>Release:        1.3.1
>Environment:
Rhapsody yves 5.2 Kernel Release 5.2: Sun Aug  2 15:52:18 PDT 1998; root(rcbuilder):Objects/kernel-120.obj~4/RELEASE_PPC Copyright (c) 1988-1995,1997-1998 Apple Computer, Inc. All Rights Reserved.  Power Macintosh

The port to this system is mine. Once finished, it will be the object of a submission.
>Description:
The synopsis tells what the problem is. The config.status script generated by configure passes extra arguments to configure using $*. However, "$@" should be used instead, as in any script wanting to pass its arguments unchanged to another program (see sh(1) for more info if needed).
>How-To-Repeat:
An example would be:

% ./configure --prefix /tmp
% ./config.status proxycachedir="/this --isstupid but shows the problem"

and configure will complain about the --isstupid argument.
>Fix:
The following trivial patch fixes the problem:

--- configure.orig      Sun Jul 12 02:25:46 1998
+++ configure   Tue Aug 18 10:21:46 1998
@@ -275,7 +275,7 @@
             -e 's:$:" \\:' >>$configstatus
     done
 fi
-echo '$*' >>$configstatus
+echo '"$@"' >>$configstatus
 echo '' >>$configstatus
 chmod a+x $configstatus
>Audit-Trail:
>Unformatted:
[In order for any reply to be added to the PR database, ]
[you need to include <ap...@Apache.Org> in the Cc line ]
[and leave the subject line UNCHANGED.  This is not done]
[automatically because of the potential for mail loops. ]
[If you do not include this Cc, your reply may be ig-   ]
[nored unless you are responding to an explicit request ]
[from a developer.                                      ]
[Reply only with text; DO NOT SEND ATTACHMENTS!         ]