Received: (qmail 18321 invoked by uid 501); 14 Aug 2001 19:28:56 -0000
Message-Id: <20010814192856.18320.qmail@apache.org>
Date: 14 Aug 2001 19:28:56 -0000
From: Jack Tan <jack_s_tan@yahoo.com>
Reply-To: jack_s_tan@yahoo.com
To: submit@bugz.apache.org
Subject: Missing quotes around $(INSTDIR) in makefile.win
X-Send-Pr-Version: 3.110

>Number:         8184
>Category:       build
>Synopsis:       Missing quotes around $(INSTDIR) in makefile.win
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    apache
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          sw-bug
>Submitter-Id:   apache
>Arrival-Date:   Tue Aug 14 12:30:00 PDT 2001
>Closed-Date:    Wed Aug 15 18:18:49 PDT 2001
>Last-Modified:  Wed Aug 15 18:18:49 PDT 2001
>Originator:     jack_s_tan@yahoo.com
>Release:        1.3.20
>Organization:
>Environment:
Windows 2000 SP2
Microsoft Visual C++ 6.0 SP5
Compiling from command line using NMAKE
>Description:
I am building and installing Apache on Windows from the command line using NMAKE (not the IDE).  The current directory is src/.  I have successfully built Apache using "nmake /f Makefile.win _apacher" and am trying to execute

nmake /f Makefile.win installr INSTDIR="E:\Program Files\Apache Group\Apache"

The build fails on the following line in the makefile:

awk -f <<script2.awk "support\apxs.pl" $(INSTDIR) >"$(INSTDIR)\bin\apxs.pl"

The failure occurs because $(INSTDIR) contains embedded spaces, but the macro preceding the redirection is not quoted.  By adding quotation marks, the build finishes successfully:

awk -f <<script2.awk "support\apxs.pl" "$(INSTDIR)" >"$(INSTDIR)\bin\apxs.pl"
>How-To-Repeat:
Build from the command line and install into a directory that contains embedded spaces.
>Fix:
Yes.  Add quotation marks around the $(INSTDIR) macro on the following line in src/makefile.win:

Bad:  awk -f <<script2.awk "support\apxs.pl" $(INSTDIR) >"$(INSTDIR)\bin\apxs.pl"

Fix:  awk -f <<script2.awk "support\apxs.pl" "$(INSTDIR)" >"$(INSTDIR)\bin\apxs.pl"
>Release-Note:
>Audit-Trail:
State-Changed-From-To: open-closed
State-Changed-By: wrowe
State-Changed-When: Wed Aug 15 18:18:49 PDT 2001
State-Changed-Why:

Thanks!  Fixed in 1.3 (no problem in 2.0)

>Unformatted:
 [In order for any reply to be added to the PR database, you need]
 [to include <apbugs@Apache.Org> in the Cc line and make sure the]
 [subject line starts with the report component and number, with ]
 [or without any 'Re:' prefixes (such as "general/1098:" or      ]
 ["Re: general/1098:").  If the subject doesn't match this       ]
 [pattern, your message will be misfiled and ignored.  The       ]
 ["apbugs" address is not added to the Cc line of messages from  ]
 [the database 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!     ]