Closed Bug 1165135 Opened 5 years ago Closed 5 years ago

distribution directory not removed on pave over install

Categories

(Firefox :: Installer, defect)

Product:
Firefox
Component:
Installer
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
VERIFIED FIXED
Milestone:
Firefox 41
Tracking Flags:
Tracking Status
firefox38 + wontfix
firefox38.0.5 + verified
firefox39 + verified
firefox40 --- fixed
firefox41 --- fixed
firefox-esr31 --- wontfix
firefox-esr38 39+ verified

People

(Reporter: robert.strong.bugs, Assigned: robert.strong.bugs)

References

Details

Attachments

(1 file)

patch
910 bytes, patch
spohl
: review+
Sylvestre
: approval-mozilla-aurora+
Sylvestre
: approval-mozilla-beta+
Sylvestre
: approval-mozilla-release+
Sylvestre
: approval-mozilla-esr38+
Details | Diff | Splinter Review 
No description provided.
Note: the code that caused this was added for the update hotfix so we can support installing without removing the distribution dir for repacks such as the eu-ballot
Attached patch patch — — Splinter Review 
Meh!
Assignee: nobody → robert.strong.bugs
Status: NEW → ASSIGNED
Attachment #8606048 - Flags: review?(spohl.mozilla.bugs)
[Tracking Requested - why for this release]:
This causes bugs such as bug 1164858
tracking-firefox38: --- → ?
tracking-firefox38.0.5: --- → ?
tracking-firefox39: --- → ?
Attachment #8606048 - Flags: review?(spohl.mozilla.bugs) → review+
Comment on attachment 8606048 [details] [diff] [review]
patch

Approval Request Comment
[Feature/regressing bug #]: Bug 1164858 - added to support installing without removing the distribution dir for repacks such as the eu-ballot
[User impact if declined]: Users installing a non repack build over a repack build won't have the repack customizations removed.
[Describe test coverage new/current, TreeHerder]: Thoroughly tested locally.
[Risks and why]: Little to none. The patch is a simple one line fix that just initializes an nsis variable
[String/UUID change made/needed]: None
Attachment #8606048 - Flags: approval-mozilla-beta?
Attachment #8606048 - Flags: approval-mozilla-aurora?
Comment on attachment 8606048 [details] [diff] [review]
patch

Should also get this in esr38
Attachment #8606048 - Flags: approval-mozilla-esr38?
https://hg.mozilla.org/mozilla-central/rev/b2f7c610539c
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox41: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 41
Comment on attachment 8606048 [details] [diff] [review]
patch

Should be in the first beta of 39 and ESR 38.1.0.
Attachment #8606048 - Flags: approval-mozilla-esr38?
Attachment #8606048 - Flags: approval-mozilla-esr38+
Attachment #8606048 - Flags: approval-mozilla-beta?
Attachment #8606048 - Flags: approval-mozilla-beta+
Attachment #8606048 - Flags: approval-mozilla-aurora?
Attachment #8606048 - Flags: approval-mozilla-aurora+
Comment on attachment 8606048 [details] [diff] [review]
patch

Approval Request Comment: comment #5

Tracking for 38.0.5 was requested so I suspect this needs uplifting to release for the 38.0.5 release. Sylvestre, can you clarify?
Flags: needinfo?(sledru)
Attachment #8606048 - Flags: approval-mozilla-release?
Comment on attachment 8606048 [details] [diff] [review]
patch

Well, Robert didn't ask for it but I am happy to take it too.
Flags: needinfo?(sledru)
Attachment #8606048 - Flags: approval-mozilla-release? → approval-mozilla-release+
Flags: qe-verify+
QA Contact: alexandra.lucinet
Verified fixed with 38.0.5 beta 3 EME free build (Build ID: 20150518141916), under Windows 7 64-bit - distribution folder is correctly removed.
Status: RESOLVED → VERIFIED
status-firefox38.0.5: fixed → verified
Flags: qe-verify+
Flags: qe-verify+
Verified fixed with 39.0b4 EME free build 2 (Build ID: 20150609130336), under Windows 7 64-bit and Windows 8.1 32-bit.
status-firefox39: fixed → verified
Removing qe-verify+ since this was already verified on Firefox 38.0.5 and 39.
Flags: qe-verify+
Reproduced with Firefox 38 RC build 3 under Win 7 64-bit.
There are no repack builds for ESR, so I installed 38.1esr build over a regular 38 repack build and the distribution folder was correctly removed. Marking as verified.
status-firefox-esr38: fixed → verified
You need to log in before you can comment on or make changes to this bug.