Closed Bug 833863 Opened 7 years ago Closed 7 years ago

The wrong SplayTree.h is ending up in dist/include/mozilla, causing build error: "Cannot open include file: 'lifoalloc.h'

Categories

(Firefox Build System :: General, defect)

x86_64
Linux
defect
Not set

Tracking

(firefox20+ fixed, firefox21 fixed)

RESOLVED FIXED
mozilla21
Tracking Status
firefox20 + fixed
firefox21 --- fixed

People

(Reporter: benjamin, Assigned: benjamin)

References

Details

Attachments

(1 file)

I don't know why the tinderbox and everyone isn't seeing this, but it may have to do with the relative timestamp of files or something.

There are currently two SplayTree.h files in the tree: mfbt/SplayTree.h and js/src/ds/SplayTree.h. In my local clobber build, the js (wrong) one is ending up in dist/include/mozilla/SplayTree.h

This is because when we are building in js/src, we include mfbt/exported_headers.mk. This tries to install SplayTree.h, but js/src/Makefile.in puts $(srcdir)/ds in front of mfbt in the VPATH.
Assignee: nobody → benjamin
Status: NEW → ASSIGNED
Attachment #705424 - Flags: review?(khuey)
Comment on attachment 705424 [details] [diff] [review]
Put MFBT first in the VPATH, rev. 1

Review of attachment 705424 [details] [diff] [review]:
-----------------------------------------------------------------

::: js/src/Makefile.in
@@ +57,5 @@
> +###############################################
> +# BEGIN include sources for low-level code shared with mfbt
> +#
> +MFBT_ROOT = $(srcdir)/../../mfbt
> +VPATH += $(MFBT_ROOT)

Why += instead of = like we had before?
Attachment #705424 - Flags: review?(khuey) → review+
Why not? Makes it easier to refactor in the future if warranted.
https://hg.mozilla.org/mozilla-central/rev/cf2a37861425
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
Duplicate of this bug: 835735
For me that patch is needed to build Firefox 20 at all. Can it be considered for a patch in 20?
Comment on attachment 705424 [details] [diff] [review]
Put MFBT first in the VPATH, rev. 1

[Approval Request Comment]
Bug caused by (feature/regressing bug #): bug 822906
User impact if declined: Failure to build aurora
Testing completed (on m-c, etc.): Landed on m-c a while ago
Risk to taking this patch (and alternatives if risky): Very low
String or UUID changes made by this patch: None
Attachment #705424 - Flags: approval-mozilla-aurora?
Comment on attachment 705424 [details] [diff] [review]
Put MFBT first in the VPATH, rev. 1

Tracking and approving on behalf of developers and Linux distros.
Attachment #705424 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.