Closed
Bug 458008
Opened 17 years ago
Closed 17 years ago
Handle even more .mozconfig files in .hgignore
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)
Details
Attachments
(1 file)
|
327 bytes,
patch
|
benjamin
:
review-
|
Details | Diff | Splinter Review |
Bug 389826 added .mozconfig, .mozconfig.mk and .mozconfig.out files to .hgignore. I tend to keep at least a couple of .mozconfig files named such as .mozconfig-dbg and .mozconfig-opt for debug and optimized builds, and when building an optimized version, for example, I do:
cp .mozconfig-opt .mozconfig
make -f client.mk build
I think we can further improve the .mozconfig handling in .hgignore to enable such use cases. See the attached patch for the approach I'm proposing.
Attachment #341245 -
Flags: review?(benjamin)
Comment 1•17 years ago
|
||
for this use, I would personally recommend expanding *your* hgrc file to have those additional mozconfig ignores, rather than do in tree.
existing .mozconfig ignores are good enough, and we do have the MOZCONFIG environ var that is also referenced for the multiple config uses.
| Assignee | ||
Comment 2•17 years ago
|
||
(In reply to comment #1)
> for this use, I would personally recommend expanding *your* hgrc file to have
> those additional mozconfig ignores, rather than do in tree.
>
> existing .mozconfig ignores are good enough, and we do have the MOZCONFIG
> environ var that is also referenced for the multiple config uses.
This is mostly a convenience issue, and I'm not that much urged to get this into the tree. So, I'll wait and see what Benjamin thinks here, and if he feels the same way, we can WONTFIX it. :-)
Comment 3•17 years ago
|
||
Comment on attachment 341245 [details] [diff] [review]
Patch
Let's WONTFIX this
Attachment #341245 -
Flags: review?(benjamin) → review-
Updated•17 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → WONTFIX
Updated•8 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•