445146 - Move editor/ui from mozilla-central to comm-central

Status: RESOLVED FIXED

(MailNews Core :: Build Config, defect)

Description

[Robert Kaiser]

editor/ui contains the compose part of mail and the SeaMonkey composer, this should probably move over from mozilla-central to comm-central in the future.

Comment 1

[Axel Hecht [:Pike]]

This is blocking compare-locales for mail and suite, I suggest to get this done before trying to do l10n builds on those.

Comment 2

[Mark Banner (:standard8)]

I'll see if I can knock up a quick patch tonight.

Comment 3

[Robert Kaiser]

(In reply to comment #1)
> This is blocking compare-locales for mail and suite, I suggest to get this done
> before trying to do l10n builds on those.

Actually, I'm pretty sure doing L10n builds and having compare-locales run are orthogonal and do NOT block each other.

Comment 4

[Mark Banner (:standard8)]

Attachment: [checked in] Editor Files to move

This is the list of editor files to move, generated from a mozilla-central patch. I can attach the full patches if you really want me to, but I doubt it.

Build config patches coming up in a moment.

Comment 5

[Mark Banner (:standard8)]

Attachment: [checked in] comm-central build changes

These are the build changes to comm-central, note: to be applied after moving the files (if you want the patches to test, let me know).

Comment 6

[Mark Banner (:standard8)]

Attachment: mozilla-central changes

The very minimal mozilla-central build changes.

Comment 7

[Mark Banner (:standard8)]

Comment on attachment 335952 [details] [diff] [review]
mozilla-central changes

I just realised, editor/composer shouldn't be within the MOZ_COMPOSER if statement. Additionally, MOZ_COMPOSER is essentially redundant in mozilla-central with this move of editor/ui. Therefore we should just drop it as well.

Comment 8

[Robert Kaiser]

(In reply to comment #7)
> (From update of attachment 335952 [details] [diff] [review])
> I just realised, editor/composer shouldn't be within the MOZ_COMPOSER if
> statement. Additionally, MOZ_COMPOSER is essentially redundant in
> mozilla-central with this move of editor/ui. Therefore we should just drop it
> as well.

From Thunderbird, yes - but I think we still will want to be able to build SeaMonkey with --disable-composer (probably along with --disable-mailnews).

Comment 9

[Mark Banner (:standard8)]

(In reply to comment #8)
> (In reply to comment #7)
> > (From update of attachment 335952 [details] [diff] [review] [details])
> > I just realised, editor/composer shouldn't be within the MOZ_COMPOSER if
> > statement. Additionally, MOZ_COMPOSER is essentially redundant in
> > mozilla-central with this move of editor/ui. Therefore we should just drop it
> > as well.
> 
> From Thunderbird, yes - but I think we still will want to be able to build
> SeaMonkey with --disable-composer (probably along with --disable-mailnews).

This won't affect the comm-central apps. It will just mean that mozilla-central configure will no longer have a --disable-composer option. The comm-central one will remain untouched.

Comment 10

[Robert Kaiser]

Ah, right, you wrote *mozilla*-central, not *comm*-central. In that case, sure, MOZ_COMPOSER is probably obsolete there.

Comment 11

[Mark Banner (:standard8)]

Attachment: mozilla-central changes v2

Updated patch for the mozilla-central changes. This removes the --disable-composer option from the mozilla-central configure.in, and sorts out some inaccuracies in toolkit-makefiles.sh (as well as removing the unneeded options).

Comment 12

[Robert Kaiser]

Comment on attachment 335951 [details] [diff] [review]
[checked in] comm-central build changes

>diff --git a/mail/build.mk b/mail/build.mk
>--- a/mail/build.mk
>+++ b/mail/build.mk
>@@ -56,6 +54,10 @@ ifdef MOZ_LDAP_XPCOM
> ifdef MOZ_LDAP_XPCOM
> tier_app_staticdirs += directory/c-sdk
> tier_app_dirs += directory/xpcom
>+endif
>+
>+ifdef MOZ_COMPOSER
>+tier_app_dirs += editor/ui
> endif

I don't think Thunderbird wants to make anything |ifdef MOZ_COMPOSER|, mailnews doesn't work without MOZ_COMPOSER and such ifdefs haven't been in Thunderbird-specific code for some time from what I've seen.

I haven't actually tested the code, but it looks good to me.

Comment 13

[Robert Kaiser]

Comment on attachment 336197 [details] [diff] [review]
mozilla-central changes v2

http://mxr.mozilla.org/comm-central/source/mozilla/config/autoconf.mk.in#125 probably should also be deleted

Comment 14

[[e-mail]]

Comment on attachment 335950 [details] [diff] [review]
[checked in] Editor Files to move

I'm assuming here you meant to include the images too but because they're binary they didn't show up in a normal diff.

Comment 15

[Mark Banner (:standard8)]

(In reply to comment #14)
> (From update of attachment 335950 [details] [diff] [review])
> I'm assuming here you meant to include the images too but because they're
> binary they didn't show up in a normal diff.

They did in the git diff, but I grepped wrong.

Comment 16

[Mark Banner (:standard8)]

Attachment: [checked in] mozilla-central changes v3

Include the config/autoconf.mk.in change

Comment 17

[Mark Banner (:standard8)]

Comment on attachment 335950 [details] [diff] [review]
[checked in] Editor Files to move

Checked in.

comm-central changeset id: 250:ee719a050249
mozilla-central changeset id: 18735:c0e82e7f2412