Closed
Bug 759767
Opened 12 years ago
Closed 12 years ago
Firefox 4-9 should never update past FF12 (and same for Thunderbird)
Categories
(Release Engineering :: General, defect, P2)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: akeybl, Assigned: nthomas)
References
Details
Attachments
(4 files)
2.25 KB,
patch
|
bhearsum
:
review+
nthomas
:
checked-in+
|
Details | Diff | Splinter Review |
68.84 KB,
patch
|
bhearsum
:
review+
nthomas
:
checked-in+
|
Details | Diff | Splinter Review |
2.12 KB,
patch
|
standard8
:
review+
nthomas
:
checked-in+
|
Details | Diff | Splinter Review |
37.46 KB,
patch
|
bhearsum
:
review+
nthomas
:
checked-in+
|
Details | Diff | Splinter Review |
Because Win2K and WinXP SP 1 users on older versions of Firefox will never be updated to FF13, we need to push them to FF12 indefinitely such that they receive the desupport add-on hotfix/billboard. Those that can update past FF12 will do so within a couple of days. tl;dr When FF13 is released on 6/5, FF4-9 should still update to FF12 (from then on).
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → nrthomas
Priority: -- → P2
Assignee | ||
Comment 1•12 years ago
|
||
This will mean we only generate snippets for 10.0+ to latest. Any preference for what we do about update verify ? We could check that 4-9 can update to 12 each release, or just drop them from the verify config.
Attachment #628458 -
Flags: review?(bhearsum)
Reporter | ||
Comment 2•12 years ago
|
||
It'd be preferable that we verify on each release if it isn't too much extra work.
Comment 3•12 years ago
|
||
Comment on attachment 628458 [details] [diff] [review] [cvs] drop older releases from sequence Review of attachment 628458 [details] [diff] [review]: ----------------------------------------------------------------- It seems like a waste of machine time to me to continue to run update verify on these versions - we'd be doing the exact same tests over and over. If we really want to do it, we'll have to remove the "full" tests, because they'll end up trying to compare themselves against whatever the latest version is.
Attachment #628458 -
Flags: review?(bhearsum) → review+
Assignee | ||
Comment 4•12 years ago
|
||
Comment on attachment 628458 [details] [diff] [review] [cvs] drop older releases from sequence Checking in mozRelease-branch-patcher2.cfg; /cvsroot/mozilla/tools/patcher-configs/mozRelease-branch-patcher2.cfg,v <-- mozRelease-branch-patcher2.cfg new revision: 1.30; previous revision: 1.29 done
Attachment #628458 -
Flags: checked-in+
Assignee | ||
Comment 5•12 years ago
|
||
We could probably do something like diff --git a/release/updates/mozRelease-firefox-win32.cfg b/release/updates/mozRelease-firefox-win32.cfg --- a/release/updates/mozRelease-firefox-win32.cfg +++ b/release/updates/mozRelease-firefox-win32.cfg @@ -16,1 +16,1 @@ release="10.0" product="Firefox" platfor -release="9.0.1" product="Firefox" platform="WINNT_x86-msvc" build_id="20111220165912" locales="de en-US ru" channel="betatest" from="/firefox/releases/9.0.1/win32/%locale%/Firefox Setup 9.0.1.exe" +release="9.0.1" product="Firefox" platform="WINNT_x86-msvc" build_id="20111220165912" locales="de en-US ru" channel="betatest" from="/firefox/releases/9.0.1/win32/%locale%/Firefox Setup 9.0.1.exe" to="/firefox/nightly/12.0-candidates/build1/win32/%locale%/Firefox Setup 12.0.exe" ie, add a 12.0 'to' var at the first old release, if we wanted to keep testing (but maybe chunking makes that not possible?). But I think it would be sufficient to check that bouncer links for 12.0 are still working - ideally just a HEAD request but that's not possible in the current framework, so we'd query AUS then download the mar file.
Assignee | ||
Comment 6•12 years ago
|
||
In a perfect world we'd do fx4-9.0.1 in final verification only, or at least only in test mode only for update verify (a curl -I style HEAD request to bouncer). This is the expedient solution for now. I can make the locale lists alpha-sorted if you want, it was an attempt to make the diff less hideous but I'm not sure it worked.
Attachment #628650 -
Flags: review?(bhearsum)
Assignee | ||
Comment 7•12 years ago
|
||
TB is in too: <Standard8> so if I'm on TB 7 and win2k and we publish an update to TB 13, then I won't get updated to 12? ouch yeah, we should do that
Summary: Firefox 4-9 should never update past FF12 → Firefox 4-9 should never update past FF12 (and same for Thunderbird)
Assignee | ||
Comment 8•12 years ago
|
||
Attachment #628651 -
Flags: review?(mbanner)
Seamonkey? I think they are still on older compiler so maybe they still support the systems mentioned here.
Assignee | ||
Comment 10•12 years ago
|
||
Same again for TB - remove the full checks and paste in de en-US ru into the quick checks.
Attachment #628654 -
Flags: review?(mbanner)
Assignee | ||
Comment 11•12 years ago
|
||
(In reply to :aceman from comment #9) > Seamonkey? > I think they are still on older compiler so maybe they still support the > systems mentioned here. Actually, SeaMonkey releases are handled separately by that community. Thunderbird is include because they recently transitioned to using the same MoCo release-infrastructure as Firefox.
Comment 12•12 years ago
|
||
Comment on attachment 628650 [details] [diff] [review] [tools] update verify configs - disabling full checks Review of attachment 628650 [details] [diff] [review]: ----------------------------------------------------------------- I agree that this is the best solution we have right now.
Attachment #628650 -
Flags: review?(bhearsum) → review+
Updated•12 years ago
|
Attachment #628651 -
Flags: review?(mbanner) → review+
Comment 13•12 years ago
|
||
Comment on attachment 628654 [details] [diff] [review] [tools] update verify configs - disabling full checks for TB This looks fine, but I'm going to defer on complexity of reading/checking.
Attachment #628654 -
Flags: review?(mbanner) → review?(bhearsum)
Updated•12 years ago
|
Attachment #628654 -
Flags: review?(bhearsum) → review+
Assignee | ||
Comment 14•12 years ago
|
||
Comment on attachment 628650 [details] [diff] [review] [tools] update verify configs - disabling full checks http://hg.mozilla.org/build/tools/rev/e11a2673c6bd
Attachment #628650 -
Flags: checked-in+
Assignee | ||
Comment 15•12 years ago
|
||
Comment on attachment 628651 [details] [diff] [review] [cvs] drop older TB releases from sequence Checking in mozRelease-thunderbird-branch-patcher2.cfg; /cvsroot/mozilla/tools/patcher-configs/mozRelease-thunderbird-branch-patcher2.cfg,v <-- mozRelease-thunderbird-branch-patcher2.cfg new revision: 1.33; previous revision: 1.32 done
Attachment #628651 -
Flags: checked-in+
Assignee | ||
Comment 16•12 years ago
|
||
Comment on attachment 628654 [details] [diff] [review] [tools] update verify configs - disabling full checks for TB http://hg.mozilla.org/build/tools/rev/64a403268632
Attachment #628654 -
Flags: checked-in+
Assignee | ||
Updated•12 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 17•12 years ago
|
||
For the purposes of verifying this bug, I'll add a couple 4-9 builds into the 13RC update checks.
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•