Closed Bug 855056 Opened 7 years ago Closed 7 years ago

Disable Win64 by default on try and update trychooser


(Release Engineering :: General, defect, P2)

Windows Server 2008


(Not tracked)



(Reporter: armenzg, Assigned: armenzg)




(1 file, 1 obsolete file)

Attached patch disable win64 by default on try (obsolete) — Splinter Review
I have tried testing this patch locally without any luck.

I'm using this:
buildbot sendchange --master localhost:9101 --revision default    --branch try --username armenzg --comments "try: -b do -p all -u none -t none" doit

To setup the master:
./ master bm31-try1

To avoid setting up a mail server:
--- a/mozilla/
+++ b/mozilla/
@@ -142,6 +142,6 @@ BRANCHES = {
         'download_base_url': '',
         'mobile_download_base_url': '',
-        'enable_mail_notifier': True,
-        'notify_real_author': True,
+        'enable_mail_notifier': False,
+        'notify_real_author': False,
         'package_url': '',
         'talos_masters': [],
My main test is to dump out all the try schedulers, find the prettyNames for what you're interested in, and make sure that the values you're interested in contain the substring "try-nondefault". If I do that with your patch, it looks good for the regular 'try' branch, but it doesn't get the 'try-comm-central' win64; did you want that too?
Blocks: 814009
OS: Mac OS X → Windows Server 2008
Hardware: x86 → x86_64
Summary: Disable Win64 by default on try → Disable Win64 by default on try and update trychooser
I've posted a patch that adjusted the build and try sides:

I decided to move the try part of it to it. That way more people can see where it got done.
Attachment #729788 - Attachment is obsolete: true
Attachment #730686 - Flags: review?(sphink)
Priority: -- → P2
Attachment #730686 - Flags: review?(sphink) → review+
Attachment #730686 - Flags: checked-in+
win64 builds can be triggered with: "try: -b o -p win64 -u none -t none"
Closed: 7 years ago
Resolution: --- → FIXED
Product: → Release Engineering
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.