Remove files deprecated in Fx7/Fx8 when updating to Fx9

RESOLVED FIXED in Firefox 11

Status

()

Firefox
Build Config
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: nthomas, Unassigned)

Tracking

9 Branch
Firefox 11
x86
All
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
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.
Attachment #577469 - Flags: review?(khuey)
(Reporter)

Updated

6 years ago
Depends on: 670880
(Reporter)

Updated

6 years ago
Depends on: 504645
(Reporter)

Updated

6 years ago
Version: 3.6 Branch → 9 Branch
(Reporter)

Updated

6 years ago
Summary: Remove old files on upgrade to Fx7/Fx8 → Remove files deprecated in Fx7/Fx8 when updating to Fx9
Attachment #577469 - Flags: review?(khuey) → review+
(Reporter)

Comment 1

6 years ago
Comment on attachment 577469 [details] [diff] [review]
Adjust removed-files.in

http://hg.mozilla.org/integration/mozilla-inbound/rev/de7161dd4620
https://hg.mozilla.org/mozilla-central/rev/de7161dd4620
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 11
(Reporter)

Comment 3

6 years ago
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.
Attachment #577469 - Flags: approval-mozilla-beta?
Attachment #577469 - Flags: approval-mozilla-aurora?
(Reporter)

Comment 4

6 years ago
Ahem, .... it's not a problem that isn't possible.

Comment 5

6 years ago
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.
Attachment #577469 - Flags: approval-mozilla-beta?
Attachment #577469 - Flags: approval-mozilla-beta-
Attachment #577469 - Flags: approval-mozilla-aurora?
Attachment #577469 - Flags: approval-mozilla-aurora-
(Reporter)

Comment 6

6 years ago
Ok, all done here then.
Blocks: 714609
You need to log in before you can comment on or make changes to this bug.