Closed
Bug 1259219
Opened 9 years ago
Closed 9 years ago
Top-level mozconfig is ignored (while .mozconfig works)
Categories
(Thunderbird :: Build Config, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 49.0
People
(Reporter: aleth, Assigned: aleth)
References
Details
(Keywords: regression)
Attachments
(1 file)
1.24 KB,
patch
|
clokep
:
review+
|
Details | Diff | Splinter Review |
A mozconfig in the c-c dir is ignored, while a .mozconfig is read.
Most common symptom: "Cannot find project browser".
Regression from the configure changes in bug 1259174.
Assignee | ||
Updated•9 years ago
|
Blocks: 1259174
Keywords: regression
Assignee | ||
Comment 1•9 years ago
|
||
(In reply to aleth [:aleth] from comment #0)
> Regression from the configure changes in bug 1259174.
Sorry, bug 1254987 of course.
Blocks: 1254987
Assignee | ||
Comment 2•9 years ago
|
||
Attachment #8734540 -
Flags: review?(Pidgeot18)
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → aleth
Status: NEW → ASSIGNED
Assignee | ||
Comment 3•9 years ago
|
||
Comment on attachment 8734540 [details] [diff] [review]
Support both mozconfig and .mozconfig
Review of attachment 8734540 [details] [diff] [review]:
-----------------------------------------------------------------
Bump.
Attachment #8734540 -
Flags: review?(Pidgeot18) → review?(clokep)
Comment 4•9 years ago
|
||
Comment on attachment 8734540 [details] [diff] [review]
Support both mozconfig and .mozconfig
Review of attachment 8734540 [details] [diff] [review]:
-----------------------------------------------------------------
Seems sane. I'm not sure how this worked before?
Attachment #8734540 -
Flags: review?(clokep) → review+
Assignee | ||
Comment 5•9 years ago
|
||
https://hg.mozilla.org/comm-central/rev/0e1299ac02ccbd74c33a0f63099056de7319a634
Bug 1259219 - Support both mozconfig and .mozconfig. r=clokep
Assignee | ||
Comment 6•9 years ago
|
||
(In reply to Patrick Cloke [:clokep] from comment #4)
> Seems sane. I'm not sure how this worked before?
It's fallout from the move of configure to python. There's code to find the mozconfig, but it finds the mozilla/ mozconfig. Making this fully split-build-system-aware seems like overkill given that cypress is coming (hopefully).
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 49.0
And what are the uses for the different files? I always had a .mozconfig so what feature am I missing?
Assignee | ||
Comment 8•9 years ago
|
||
(In reply to :aceman from comment #7)
> And what are the uses for the different files? I always had a .mozconfig so
> what feature am I missing?
I don't think there is any difference. Likely historic reasons...
Comment 9•9 years ago
|
||
Windows is a bit stupid with dot files. For example you can't rename the file because Windows wants something before the dot. OS X can't show the hidden files in some folders only. You have to enable this for the whole system (or is there a possibility, aleth?).
The question could also be, why should this mozconfig be a hidden file?
Comment 10•9 years ago
|
||
(In reply to Richard Marti (:Paenglab) from comment #9)
> The question could also be, why should this mozconfig be a hidden file?
Just a unix convention that configurable state lives in dot files. Like .thunderbird/
You need to log in
before you can comment on or make changes to this bug.
Description
•