Last Comment Bug 705974 - Remove files deprecated in Fx7/Fx8 when updating to Fx9
: Remove files deprecated in Fx7/Fx8 when updating to Fx9
Status: RESOLVED FIXED
:
Product: Firefox
Classification: Client Software
Component: Build Config (show other bugs)
: 9 Branch
: x86 All
: -- normal (vote)
: Firefox 11
Assigned To: Nobody; OK to take it and work on it
:
Mentors:
Depends on: 504645 670880
Blocks: 714609
  Show dependency treegraph
 
Reported: 2011-11-28 19:41 PST by Nick Thomas [:nthomas]
Modified: 2012-01-02 04:49 PST (History)
4 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Adjust removed-files.in (1.11 KB, patch)
2011-11-28 19:41 PST, Nick Thomas [:nthomas]
khuey: review+
akeybl: approval‑mozilla‑aurora-
akeybl: approval‑mozilla‑beta-
Details | Diff | Review

Description Nick Thomas [:nthomas] 2011-11-28 19:41:02 PST
Created attachment 577469 [details] [diff] [review]
Adjust removed-files.in

Bug 670880 removed modules/SpatialNavigation.js in Fx8 for all platforms
Bug 504645 removed icons/document.png in Fx7 for Linux

Both are not removed when applying an update from earlier versions, eg 3.6.24, and this fixes that up.
Comment 1 Nick Thomas [:nthomas] 2011-12-04 16:12:37 PST
Comment on attachment 577469 [details] [diff] [review]
Adjust removed-files.in

http://hg.mozilla.org/integration/mozilla-inbound/rev/de7161dd4620
Comment 2 Matt Brubeck (:mbrubeck) 2011-12-05 10:32:19 PST
https://hg.mozilla.org/mozilla-central/rev/de7161dd4620
Comment 3 Nick Thomas [:nthomas] 2011-12-05 19:29:40 PST
Comment on attachment 577469 [details] [diff] [review]
Adjust removed-files.in

Drivers, this patch removes two files when users apply an update, which we stopped shipping in Fx7 or 8 "installers". The result is an old install + updates will be the same as a new install (excluding old empty directories we don't remove with the updater).  Both files were minor ones that were unused or little used before we stopped shipping them in "installers".

I'd like to at least get this on Aurora, and beta if it's an appropriate change at this point in the cycle, but it's not a problem if that is not problem.
Comment 4 Nick Thomas [:nthomas] 2011-12-05 19:30:46 PST
Ahem, .... it's not a problem that isn't possible.
Comment 5 Alex Keybl [:akeybl] 2011-12-06 14:55:04 PST
Comment on attachment 577469 [details] [diff] [review]
Adjust removed-files.in

[Triage Comment]
Let's let this ride the train. Doesn't appear to have major user benefit, and QA said this doesn't affect their testing.
Comment 6 Nick Thomas [:nthomas] 2011-12-06 14:56:40 PST
Ok, all done here then.

Note You need to log in before you can comment on or make changes to this bug.