"fix" mkdepend to stop short-circuiting dependencies

VERIFIED WONTFIX

Status

SeaMonkey
Build Config
P4
major
VERIFIED WONTFIX
16 years ago
13 years ago

People

(Reporter: hacker formerly known as seawood@netscape.com, Assigned: hacker formerly known as seawood@netscape.com)

Tracking

Trunk
mozilla1.2alpha

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 obsolete attachment)

mkdepend is a copy of X11's makedepend utility.  Both have a feature where they
only process the dependencies for a given file once.  This is fine for simple
files but we could wind up with wrong dependencies for some of our more complex
files in bigger directories.  Currently, we work around this feature by invoking
a separate mkdepend process for each source file but the addition process
creation is killing us on win32. (At least that's my story.)  So we need to add
a switch to make mkdepend stop short-circuiting dependencies or make it smarter
about the dependency list that it keeps.  If it doesn't do this already, we
could make it keep track of what defines where used for a particular dependency
and compare the stored list against the current list of defines that are being
used to determine the current dependency.
Depends on: 132232
Priority: -- → P2
Target Milestone: --- → mozilla1.0
Created attachment 75141 [details] [diff] [review]
Skip include short-circuiting

This patch requires the X11R6.6 patch from bug 132232.	It avoids the
short-circuiting issue by ifdefing out the lines that set & check whether the
file had already been processed.
Priority: P2 → P4
Target Milestone: mozilla1.0 → mozilla1.2alpha
Comment on attachment 75141 [details] [diff] [review]
Skip include short-circuiting

Patch has nasty side-effects.  Punting.
Attachment #75141 - Attachment is obsolete: true
Really punting.
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → WONTFIX

Comment 4

16 years ago
verified wontfix.
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.