Closed Bug 1106707 Opened 9 years ago Closed 9 years ago

Add in support for win32/win64 builds


(Release Engineering :: General, defect)

Not set


(Not tracked)



(Reporter: sfink, Assigned: sfink)




(4 files, 1 obsolete file)

I am slowly getting closer to enabling spidermonkey builds on win32 and win64. There are some necessary bits distributed among the mozharness scripts and config and the buildbot config that I need here.

The main build script for spidermonkey has now moved in-tree. is now just an adapter script to allow the in-tree to be invoked from buildbot. (Well, it also still contains the logic to do the full build, for older branches.)
--platform allows specifying win32 vs win64

The Windows builders require mozmake, but are currently running MozillaBuild 1.5.1 (!) which does not contain it, so we need to fetch it from tooltool.
Attachment #8531119 - Flags: review?(bhearsum)
Comment on attachment 8531119 [details] [diff] [review]
Add in support for win32/win64 builds

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

::: scripts/spidermonkey_builds/
@@ +12,1 @@

This is something that's defined in the production/staging configs, so it should probably be passed through from there if possible. (Eg: Everything else seems fine.
Attachment #8531119 - Flags: review?(bhearsum) → feedback+
I'll pass it through a command line option.
Attachment #8540971 - Flags: review?(bhearsum)
Attachment #8531119 - Attachment is obsolete: true
Give the tooltool server and the platform (so can distinguish between win32 and win64 builds.)
Attachment #8540972 - Flags: review?(bhearsum)
Attachment #8540972 - Flags: review?(bhearsum) → review+
Attachment #8540971 - Flags: review?(bhearsum) → review+
This didn't completely work. I think I'm pulling the tooltool_url_list from the wrong config. Trying out a fix now.
According to my diff, this should do the trick.
Attachment #8552044 - Flags: review?(bhearsum)
Blocks: 972089
Attachment #8552044 - Flags: review?(bhearsum) → review+
(In reply to Chris Cooper [:coop] from comment #10)
> In production:

I think this may have caused some spidermonkey-specific bustage, as reported by philor:
Argh! Put the arguments out of order. Fix the build script to allow any order.
Attachment #8553143 - Flags: review?(bhearsum)
Attachment #8553143 - Flags: review?(bhearsum) → review+
This is working, finally.
Closed: 9 years ago
Resolution: --- → FIXED
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.