Closed Bug 1021538 Opened 6 years ago Closed 6 years ago

Use mozmake to build comm-central


(MailNews Core :: Build Config, defect)

Windows 7
Not set


(Not tracked)

Thunderbird 33.0


(Reporter: glandium, Assigned: glandium)




(1 file, 1 obsolete file)

No description provided.
The change is straight out of m-c's pymake (bug 927672)
The tooltool manifest also comes from m-c (same bug).

Presumably, if the seamonkey slaves are not using tooltool, they won't get mozmake.exe, and as a result pymake will fall through to using plain pymake. But I don't know whether they do or not.

The python/foo thing in the try changeset is a workaround for bug 1021375.
Attachment #8435553 - Flags: review?(bugspam.Callek)
Attachment #8435553 - Flags: review?(Pidgeot18)
Assignee: nobody → mh+mozilla
Damn, that was close. The compile went through, but buildsymbols failed.
The fix for buildsymbols was trivial:

Then I hit check-sync-dirs, which complains about the manifest I added not matching the one in m-c. Meh, the difference is sccache and, those won't hurt and won't be used anyways, since m-c will look for it in $topsrcdir/mozilla and won't find it.

This last try is what this patch corresponds to.
Attachment #8435652 - Flags: review?(bugspam.Callek)
Attachment #8435652 - Flags: review?(Pidgeot18)
Attachment #8435553 - Attachment is obsolete: true
Attachment #8435553 - Flags: review?(bugspam.Callek)
Attachment #8435553 - Flags: review?(Pidgeot18)
Comment on attachment 8435652 [details] [diff] [review]
Use mozmake to build comm-central

Review of attachment 8435652 [details] [diff] [review]:

::: build/pymake/
@@ +23,5 @@
> +    if 'TINDERBOX_OUTPUT' in os.environ:
> +        # When building on mozilla build slaves, execute mozmake instead. Until bug
> +        # 978211, this is the easiest, albeit hackish, way to do this.
> +        mozmake = os.path.join(os.path.dirname(__file__), '..', '..',
> +            'mozmake.exe')

I was actually surprised that this works, but the try runs convinced me that it does.
Attachment #8435652 - Flags: review?(Pidgeot18) → review+
Attachment #8435652 - Flags: review?(bugspam.Callek)
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 33.0
Looks like the change is for Windows only, but the bug is filed under Linux... Can you fix the Platform?
OS: Linux → Windows 7
Hardware: x86_64 → All
You need to log in before you can comment on or make changes to this bug.