630176 - Add the comm-central source stamp to Thunderbird's source stamp file

Status: RESOLVED FIXED

(Thunderbird :: Build Config, defect)

Description

[Mark Banner (:standard8)]

Attachment: The fix

Now that bug 549958 has altered how the source stamp files are done, we should append the comm-central stamp to Thunderbird's version of the file.

Patch attached to do that (I would copy and paste to suite, but I expect that SM probably wants to do their extension repos as well).

Comment 1

[Justin Wood (:Callek)]

Comment on attachment 508382 [details] [diff] [review]
The fix

I'm happy with this as a standalone, but I *really* want this taken one step further for us... and basically reverse http://hg.mozilla.org/comm-central/rev/ce2bbf8d11fd

By doing s/MOZ_SOURCE_STAMP/COMM_SOURCE_STAMP/ in c-c (even with an export from our root Makefile) and even having it utilized for suite, etc. (for the sourcestamp file) suite can add the extensions to the sourcestamp file in another bug.

But if that is more than you have time for, or disagree with my plan you can land this as is, just please file a bug for the plan I outlined.

Comment 2

[Mark Banner (:standard8)]

Checked in: http://hg.mozilla.org/comm-central/rev/bbfc217dd3ff

Raised bug 630487 on the follow-up tidy up.

Comment 3

[Mark Banner (:standard8)]

Comment on attachment 508382 [details] [diff] [review]
The fix

Should consider for 3.1.9 once the dependent bug lands.

Comment 4

[Mark Banner (:standard8)]

Landed on 1.9.2 branch:

http://hg.mozilla.org/releases/comm-1.9.2/rev/9663e3b772d8