Closed Bug 458008 Opened 17 years ago Closed 17 years ago

Handle even more .mozconfig files in .hgignore

Categories

(Firefox Build System :: General, defect)

defect
Not set
minor

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)

Details

Attachments

(1 file)

Attached patch PatchSplinter 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)
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.
(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 on attachment 341245 [details] [diff] [review] Patch Let's WONTFIX this
Attachment #341245 - Flags: review?(benjamin) → review-
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → WONTFIX
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: