447636 - build editor/ui with Mozilla build system as long as it lives in mozilla/

Status: RESOLVED FIXED

(MailNews Core :: Build Config, defect)

Description

[Robert Kaiser]

On IRC, I was pointed to our current way of building editor/ui being wonky, as its DEPTH makes it point to the Mozilla autoconf.mk but it can end up with setting topsrcdir to the comm top directory (probably when generated via make-makefile) and that makes it pull the comm rules.mk which uses MOZILLA_SRCDIR at a number of places which is not defined by the Mozilla autoconf.mk.

Potentially ending up with a mixed build system calls for trouble, and I wonder a bit that it merely works for us right now.

We should move building of mozilla/editor/ui completely to the Mozilla build system via shifting it around in build.mk so things get stable.

Once bug 445146 gets solved, it can make us build it from our build system as it should be.

Comment 1

[Robert Kaiser]

Attachment: move editor/ui to be built with the Mozilla build system for now

This patch moves building of editor/ui to the Mozilla build system for now. This should move back when we move this code to comm-central.

Comment 2

[Mark Banner (:standard8)]

Comment on attachment 331137 [details] [diff] [review]
move editor/ui to be built with the Mozilla build system for now

r=me and a=me for checkin to closed tree.

Comment 3

[Robert Kaiser]

Thanks, checked in.