Closed
Bug 367127
Opened 18 years ago
Closed 18 years ago
Make mozconfig-find space-safe (e.g. spaces in $HOME)
Categories
(Firefox Build System :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: benjamin, Assigned: benjamin)
Details
(Keywords: fixed1.8.0.10, fixed1.8.1.2)
Attachments
(1 file)
2.85 KB,
patch
|
dbaron
:
review+
jay
:
approval1.8.1.2+
jay
:
approval1.8.0.10+
|
Details | Diff | Splinter Review |
If your $HOME is /c/Documents and Settings/Benjamin Smedberg mozconfig-find doesn't find a .mozconfig in it.
Attachment #251639 -
Flags: review?(dbaron)
Comment on attachment 251639 [details] [diff] [review] Honor mozconfig with spaces, rev. 1 r=dbaron
Attachment #251639 -
Flags: review?(dbaron) → review+
Assignee | ||
Comment 2•18 years ago
|
||
Fixed on trunk. I'd like to get this on the active branches as well.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 3•18 years ago
|
||
Comment on attachment 251639 [details] [diff] [review] Honor mozconfig with spaces, rev. 1 As a build-config change this is very low-risk.
Attachment #251639 -
Flags: approval1.8.1.2?
Attachment #251639 -
Flags: approval1.8.0.10?
Comment 4•18 years ago
|
||
Comment on attachment 251639 [details] [diff] [review] Honor mozconfig with spaces, rev. 1 Approved for both branches, a=jay for drivers.
Attachment #251639 -
Flags: approval1.8.1.2?
Attachment #251639 -
Flags: approval1.8.1.2+
Attachment #251639 -
Flags: approval1.8.0.10?
Attachment #251639 -
Flags: approval1.8.0.10+
Assignee | ||
Comment 5•18 years ago
|
||
Fixed on MOZILLA_1_8_BRANCH and MOZILLA_1_8_0_BRANCH
Keywords: fixed1.8.0.10,
fixed1.8.1.2
Updated•6 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•