825872 - Port bug 780561 - Overhaul the packager - to comm-central

Status: RESOLVED FIXED

(MailNews Core :: Build Config, defect)

Description

[Mark Banner (:standard8)]

+++ This bug was initially created as a clone of Bug #780561 +++

Bug 780561 is working on an overhaul of the packager, we need to have the same overhaul in comm-central to avoid bustage in comm-central.

I've tweaked the dependency list for things that I think we do need actually port, the other bugs I've removed are non-relevant things.

Comment 1

[Mike Hommey [:glandium]]

There /should/ be nothing to port from bug 780561 itself, except the flight.mk and the configure.in changes for MOZ_CHROME_FILE_FORMAT.

Comment 2

[Nomis101]

Attachment: port everything

This are every similar things I've found between m-c and c-c. But I'm confused if every of this changes are needed, because :glandium says only the parts in flight.mk and configure.in for MOZ_CHROME_FILE_FORMAT are needed. Should I remove the other parts I've ported from the patch?

Comment 3

[Mike Hommey [:glandium]]

Comment on attachment 697571 [details] [diff] [review]
port everything

Actually, that looks good like that. Some of these are not strictly needed, but it's good to have them. You'll also need a few changes in other */installer directories.

Comment 4

[Nomis101]

(In reply to Mike Hommey [:glandium] from comment #3)
> You'll also need a few changes in other
> */installer directories.

Ah yes, for suite and calendar, you are right. I will do that.

Comment 5

[Nomis101]

Attachment: port everything v2

Hm, who can I choose to review the calendar and the suite part?

Comment 6

[Philip Chee]

Comment on attachment 701307 [details] [diff] [review]
port everything v2

> Hm, who can I choose to review the calendar and the suite part?
I'd say Fallen

Comment 7

[Mike Hommey [:glandium]]

(In reply to Nomis101 from comment #5)
> Created attachment 701307 [details] [diff] [review]
> port everything v2
> 
> Hm, who can I choose to review the calendar and the suite part?

Note the MOZ_CHROME_FILE_FORMAT part is missing.

Comment 8

[Mark Banner (:standard8)]

Comment on attachment 701307 [details] [diff] [review]
port everything v2

I'm taking a look at this, I know Fallen is happy with me reviewing build config patches for Lightning as well.

Comment 9

[Mark Banner (:standard8)]

I haven't had time tonight, but this is my priority first thing in the morning.

Comment 10

[Mark Banner (:standard8)]

Attachment: port everything v3

This patch is pretty much the same as Nomis' patch apart from:

- Fixed bitrot in config.mk
- Ported the MOZ_CHROME_FILE_FORMAT changes in configure.in
- Ported one-line DIST_GARBAGE change in /Makefile.in from bug 785871

Comment 11

[Philip Chee]

Comment on attachment 705769 [details] [diff] [review]
port everything v3

This patch allows make package and make installer to complete successfully.

Comment 12

[Mark Banner (:standard8)]

This landed earlier:

https://hg.mozilla.org/comm-central/rev/ead82f3bfe13

We're still waiting on other bugs to be reviewed & land before the tree will be fully green and we can reopen.

Comment 13

[Mark Banner (:standard8)]

Tree is now green again.