Closed
Bug 103988
Opened 23 years ago
Closed 23 years ago
cache is not its own module for windows REQUIRES
Categories
(SeaMonkey :: Build Config, defect, P2)
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla0.9.6
People
(Reporter: dbaron, Assigned: alecf)
Details
(Whiteboard: fix in hand)
Attachments
(1 file)
(deleted),
patch
|
dbaron
:
review+
|
Details | Diff | Splinter Review |
In the autoconf build system, cache is its own MODULE (nkcache), but on Windows it's part of the necko module. This makes editing REQUIRES confusing and can cause bustage when making the same change on different platforms (e.g., bug 100895).
Assignee | ||
Comment 1•23 years ago
|
||
yeah, it's pretty strange. but it looks like the change is very simple, since the windows REQUIRES lines already have "nkcache" listed...attaching a patch
Status: NEW → ASSIGNED
Priority: -- → P2
Target Milestone: --- → mozilla0.9.6
Assignee | ||
Comment 2•23 years ago
|
||
Reporter | ||
Comment 3•23 years ago
|
||
Comment on attachment 52917 [details] [diff] [review] switch from "necko" to "nkcache" r=dbaron if you've checked that it builds in a clobber build (and that the purge script worked so the headers aren't still in dist/include/necko)
Attachment #52917 -
Flags: review+
Assignee | ||
Comment 4•23 years ago
|
||
I'll be trying a clobber build tonight.. adding cls@seawood.org, added the wrong one before
Whiteboard: fix in hand
Assignee | ||
Comment 6•23 years ago
|
||
checked in.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•