690840 - Get rid of layout/build

Status: NEW

(Firefox Build System :: General, defect)

Description

[(not currently active) Ted Mielczarek]

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.

Comment 1

[Mike Hommey [:glandium]]

Note that it's not needed if we fix bug 644608, which is a windows build failure away. (plus maybe some bitrotting)

Comment 2

[David Baron :dbaron: (⌚️UTC-5, no longer working on Mozilla)]

Probably all the C++ files should move to layout/base/.  (Make sure it's an hg move, though.)

Comment 3

[(not currently active) Ted Mielczarek]

Yeah, but it's still an unnecessary level of indirection, and it isn't in PARALLEL_DIRS at the moment because of this.

Comment 4

[Robert O'Callahan (:roc) (email my personal email if necessary)]

(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.