Closed Bug 514332 Opened 15 years ago Closed 15 years ago

WinCE builder on try server

Categories

(Release Engineering :: General, defect)

ARM
Windows CE
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: vlad, Assigned: jhford)

Details

Attachments

(2 files, 2 obsolete files)

(Thought there was a bug about this, but I can't find it -- bug 465039 is about windows mobile, not wince)

We need a WinCE builder on the try server; we've had a few burnings on the main Firefox tinderbox due to WinCE burning, despite folks sending builds off to the try server.
I can look at this soon, is there any time frame aside from sooner being better?
Moving to future until we get more details about time line
Component: Release Engineering → Release Engineering: Future
I'm not sure what I can give you other than "sooner being better"; "yesterday"?
I'll try: can you have this in place by the end of next week (Sep 18)?  The sooner it is in place, the faster our WinCE development will go, and the fewer regressions we'll have to chase after the fact for our upcoming 3.6 release.
It looks like the required heavy lifting has already been done in bug 485109.  I will look into getting this ready today/tonight.  It will definitely need some time to bake in staging.
Component: Release Engineering: Future → Release Engineering
Attached patch buildbot-configs patch (obsolete) — Splinter Review
This should be enough to get WinCE try builds going on Try as they are using the standard build factories.  A symlink to the mozconfig file is needed in the buildbot directory for the try master to be able to send the file to the client (this bit me on setting up maemo/winmo try server).

I will be testing this as soon as the try staging environment opens up.
Attached patch wince-try-v2 (obsolete) — Splinter Review
Copied mozconfig instead of symlinking it and added the builders.append() call that is needed.
Assignee: nobody → jford
Attachment #399630 - Attachment is obsolete: true
Status: NEW → ASSIGNED
If the failure mode of an unbaked-on-stage version is that we get people telling us that the wince builds aren't quite right, it seems like we should just push it to live and let our mass of developers do some faster baking for us.  A note to dev.platform and/or planet that says "wince added to try, let us know in bug 514332 if it's wonky" would set expectations appropriately, I think.
I'd much rather see this pass just once on try staging, tbh.
I'm hoping try staging will free up today or tomorrow, and we can land the patch this week.
Got builds going but they were failing because of issues building in the source checkout directory.
Attached patch wince-try-v4Splinter Review
Here is the patch.  Tested in Staging and works.  I modified how the MOZ_OBJDIR is set without testing, but this should be quick to verify in production.
Attachment #399633 - Attachment is obsolete: true
Attachment #399843 - Flags: review?(aki)
Attachment #399843 - Flags: checked-in?
Attachment #399843 - Flags: review?(aki) → review+
Comment on attachment 399843 [details] [diff] [review]
wince-try-v4

revision 1493:934ee1fa9780

John is reconfig'ing try master and running a test wince build.
Attachment #399843 - Flags: checked-in? → checked-in+
These should go from the mozconfig:
+ac_add_options --enable-update-channel=nightly
+ac_add_options --enable-update-packaging

Did you have a chance to compare the windows mozconfig for mozilla-central and try ?
Attached patch follow up patchSplinter Review
Thanks Nick, removed them.  I will reconfig after this lands
Attachment #399851 - Flags: review?(aki)
Attachment #399851 - Flags: checked-in?
Attachment #399851 - Flags: review?(aki) → review+
Comment on attachment 399851 [details] [diff] [review]
follow up patch

revision 1494:6df0fb2c8a6f
Attachment #399851 - Flags: checked-in? → checked-in+
I have completed a successful job (https://build.mozilla.org/tryserver-builds/jford@mozilla.com-1252624187/) which has a wince zip file.  

I am going to wait to hear from Vlad or Justin Dolske before I mark resolved fixed.
tested on device, works.
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: