Closed Bug 437219 Opened 13 years ago Closed 13 years ago
Implement usage of application/app-config
.mk and application/app-rules .mk
Benjamin's blog post at http://benjamin.smedbergs.us/blog/2008-05-02/tricks-of-the-mozilla-build-system-myconfigmk-and-myrulesmk/ mentions "Brainstorm of possible improvement: load application/app-config.mk and application/app-rules.mk for application-specific build configuration and build rules." This would also e.g. enable us to have MOZ_SUITE defined for all .cpp files, so that configure doesn't need to AC_DEFINE it and can get rid of those app-specific things.
Assignee: nobody → benjamin
Status: NEW → ASSIGNED
Attachment #323916 - Flags: review?(ted.mielczarek)
Comment on attachment 323916 [details] [diff] [review] app-config.mk and app-rules.mk, rev. 1 wow, that's easy!
Attachment #323916 - Flags: review?(ted.mielczarek) → review+
Pushed to mozilla-central, rev d4910bfb3fab
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.1
Ben, would you object to this landing for CVS (1.9.0) users too?
No, it's pretty risk-free.
Comment on attachment 323916 [details] [diff] [review] app-config.mk and app-rules.mk, rev. 1 a=beltzner
Attachment #323916 - Flags: approval18.104.22.168? → approval22.214.171.124+
checked in to CVS Checking in config/config.mk; /cvsroot/mozilla/config/config.mk,v <-- config.mk new revision: 3.390; previous revision: 3.389 done Checking in config/rules.mk; /cvsroot/mozilla/config/rules.mk,v <-- rules.mk new revision: 3.596; previous revision: 3.595 done
Target Milestone: mozilla1.9.1 → mozilla1.9.1a1
You need to log in before you can comment on or make changes to this bug.