Closed
Bug 382217
Opened 18 years ago
Closed 18 years ago
move browser/ Makefiles from allmakefiles.sh to browser/makefiles.sh
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla3 alpha6
People
(Reporter: whimboo, Assigned: whimboo)
References
Details
Attachments
(1 file, 1 obsolete file)
5.19 KB,
patch
|
bent.mozilla
:
review+
benjamin
:
review+
|
Details | Diff | Splinter Review |
This removes browser/ Makefiles from allmakefiles.sh and puts them into
browser/makefiles.sh which gets called by allmakefiles automatically now when
--enable-application=browser
I hope that's all?
Attachment #266350 -
Flags: review?(bent.mozilla)
![]() |
||
Comment 1•18 years ago
|
||
Comment on attachment 266350 [details] [diff] [review]
create browser/makefiles.sh, kill those entries from allmakefiles
>Index: browser/makefiles.sh
>===================================================================
>+# Portions created by the Initial Developer are Copyright (C) 2006
>+# the Initial Developer. All Rights Reserved.
nit: It's 2007, not 2006 ;-)
Assignee | ||
Comment 2•18 years ago
|
||
Attachment #266350 -
Attachment is obsolete: true
Attachment #266359 -
Flags: review?(bent.mozilla)
Attachment #266350 -
Flags: review?(bent.mozilla)
Comment on attachment 266359 [details] [diff] [review]
Patch v1.1 with corrected year
Looks good to me, thanks!
Attachment #266359 -
Flags: review?(bent.mozilla) → review+
Assignee | ||
Comment 4•18 years ago
|
||
Comment on attachment 266359 [details] [diff] [review]
Patch v1.1 with corrected year
Do I need a second review to get this patch checked in?
Attachment #266359 -
Flags: review?(benjamin)
Comment 5•18 years ago
|
||
Comment on attachment 266359 [details] [diff] [review]
Patch v1.1 with corrected year
Not really, I've delegate this to bent.
Attachment #266359 -
Flags: review?(benjamin) → review+
Assignee | ||
Updated•18 years ago
|
Whiteboard: [checkin needed]
Fixed on trunk.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•18 years ago
|
Status: RESOLVED → VERIFIED
Whiteboard: [checkin needed]
Target Milestone: --- → Firefox 3 alpha6
Updated•6 years ago
|
Component: Build Config → General
Product: Firefox → Firefox Build System
Updated•6 years ago
|
Target Milestone: Firefox 3 alpha6 → mozilla3 alpha6
You need to log in
before you can comment on or make changes to this bug.
Description
•