Closed Bug 1840506 Opened 1 years ago Closed 1 years ago

Remove code that generates old update directories and migration from them

Categories

(Toolkit :: Application Update, task)

Unspecified
Windows
task

Tracking

()

RESOLVED INVALID

People

(Reporter: bytesized, Assigned: bytesized)

References

Details

(Whiteboard: [fidedi-ope])

The update directory was moved in Bug 1458314 which was merged in version 65. There was a watershed update affecting all platforms in version 72.0.2. We can therefore remove the code that migrates the old update directory location to the new location.

Technically there is one thing that could happen improperly because of this: A paveover installation that upgrades Firefox from <65 to the current version will never clean up the old update directory. But (a) this should be a pretty rare occurrence, and (b) the only ill result is a small amount of storage not being cleaned up. I think it makes sense to get rid of it rather than maintain this code forever.

I'm about to move some code around related to this, and I'd rather remove it than maintain it. So I should be done with this this cycle and thus I'm marking this a P1, even though it isn't really a high priority change in and of itself.

Blocks: 1545119
No longer blocks: 1545119

Oops, I forgot that we moved the directory again in version 97 (Bug 1732435). We need to wait for yet another watershed to pass before this change is feasible. Sorry for the noise.

Status: NEW → RESOLVED
Closed: 1 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.