Closed Bug 770386 Opened 12 years ago Closed 12 years ago

Update Metro installscript makefile so build won't fail on user accounts containing spaces

Categories

(Firefox Build System :: General, defect)

x86
Windows 8.1
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: jwilde, Unassigned)

References

Details

Attachments

(1 file)

The build currently fails if the account for the currently-logged-in user contains a space.  When the user name contains a space, $(L10NBASEDIR) in browser\metro\shell\installscripts\Makefile.in has a space (but lacks quotes), messing up the arguments passed to Preprocessor.py at line 42.
Assignee: nobody → jonathan
Attached patch patch v1Splinter Review
Attachment #638569 - Flags: review?(jmathies)
Status: NEW → ASSIGNED
Comment on attachment 638569 [details] [diff] [review]
patch v1

We currently don't use this variable in the shortcut script. So 

-DUSER=$(L10NBASEDIR)

can be removed.
Attachment #638569 - Flags: review?(jmathies)
Depends on: 772044
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → INVALID
Assignee: jonathan → nobody
OS: Windows 8 Metro → Windows 8.1
Component: Build Config → General
Product: Firefox → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: