Status

()

Core
Build Config
6 years ago
2 months ago

People

(Reporter: ted, Unassigned)

Tracking

(Blocks: 1 bug)

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

6 years ago
layout/build is a leftover from when we supported shared builds, and it could actually produce a shared library. Nowadays it just produce a static library (and a fake one, at that), and adds another place that you're forced to rebuild when you're touching content/layout code.

We should move all the bits that it links directly into the libxul Makefiles in toolkit/library. The couple of source files in that directory can stay there or move elsewhere, I don't care.
Note that it's not needed if we fix bug 644608, which is a windows build failure away. (plus maybe some bitrotting)
Probably all the C++ files should move to layout/base/.  (Make sure it's an hg move, though.)
(Reporter)

Comment 3

6 years ago
Yeah, but it's still an unnecessary level of indirection, and it isn't in PARALLEL_DIRS at the moment because of this.
(In reply to David Baron [:dbaron] from comment #2)
> Probably all the C++ files should move to layout/base/.  (Make sure it's an
> hg move, though.)

Sure.
(Reporter)

Updated

6 years ago
Blocks: 694223
You need to log in before you can comment on or make changes to this bug.