Closed
Bug 1161661
Opened 9 years ago
Closed 9 years ago
Provide progress and state feedback via the stub installer taskbar icon
Categories
(Firefox :: Installer, defect)
Firefox
Installer
Tracking
()
RESOLVED
FIXED
Firefox 41
Tracking | Status | |
---|---|---|
firefox41 | --- | fixed |
People
(Reporter: robert.strong.bugs, Assigned: robert.strong.bugs)
Details
Attachments
(2 files)
25.25 KB,
patch
|
bbondy
:
review+
|
Details | Diff | Splinter Review |
1.37 KB,
patch
|
spohl
:
review+
|
Details | Diff | Splinter Review |
Patch coming up
Assignee | ||
Comment 1•9 years ago
|
||
Assignee: nobody → robert.strong.bugs
Status: NEW → ASSIGNED
Attachment #8601642 -
Flags: review?(spohl.mozilla.bugs)
Comment 2•9 years ago
|
||
I'm planning to use this bug to read up on NSIS and its scripting language to speed up reviews in the future. Let me know if this bug is too urgent for that.
Comment 3•9 years ago
|
||
Comment on attachment 8601642 [details] [diff] [review] patch rev1 Review of attachment 8601642 [details] [diff] [review]: ----------------------------------------------------------------- I'll take the review in the meantime spohl. Very cool rstrong.
Attachment #8601642 -
Flags: review?(spohl.mozilla.bugs) → review+
Assignee | ||
Updated•9 years ago
|
Flags: in-testsuite-
Target Milestone: --- → Firefox 40
Comment 6•9 years ago
|
||
And we only build the stub installer for nightlies, so the only way to find out whether a patch actually works is to land it, get it merged to m-c, and then burn a nightly? Fun! Backed out in https://hg.mozilla.org/mozilla-central/rev/8f21d63b310a for https://treeherder.mozilla.org/logviewer.html#?job_id=1462107&repo=mozilla-central, 06:05:56 INFO - !insertmacro: macro "CallArtificialFunction" already being inserted! 06:05:56 INFO - Error in macro _WinVerAtLeast on macroline 2 06:05:56 INFO - Error in macro _If on macroline 9 06:05:56 INFO - Error in macro ITBL3SetProgressState_ on macroline 2 06:05:56 INFO - Error in macro CallArtificialFunction on macroline 11 06:05:56 INFO - Error in macro ITBL3SetProgressStateCall on macroline 2 06:05:56 INFO - Error in script "stub.nsi" on line 1300 -- aborting creation process
Status: RESOLVED → REOPENED
Flags: needinfo?(robert.strong.bugs)
Resolution: FIXED → ---
Target Milestone: Firefox 40 → ---
Assignee | ||
Comment 7•9 years ago
|
||
I built it locally and this must have ran into a problem when compiling with the old NSIS that I asked to have upgraded / deployed to the build slaves over a year ago in bug 989531
Flags: needinfo?(robert.strong.bugs)
Assignee | ||
Comment 8•9 years ago
|
||
This is in addition to the existing patch. It removes the windows version checks from the ITBL3SetProgressValue and ITBL3SetProgressState macros. These aren't needed since the ITBL3Create macro which doesn't use the CallArtificialFunction macro checks the Windows version and if it isn't compatible with IID_ITaskbarList3 it leaves the $ITaskbarList3 var set to 0.
Attachment #8604443 -
Flags: review?(spohl.mozilla.bugs)
Assignee | ||
Comment 9•9 years ago
|
||
I also verified this works with NSIS 2.46 Unicode locally.
Updated•9 years ago
|
Attachment #8604443 -
Flags: review?(spohl.mozilla.bugs) → review+
Assignee | ||
Comment 10•9 years ago
|
||
Also, try push https://treeherder.mozilla.org/#/jobs?repo=try&revision=435a6592eff9
Assignee | ||
Comment 11•9 years ago
|
||
Missed the confvars change in the above try push... trying again https://treeherder.mozilla.org/#/jobs?repo=try&revision=987b874dce19
Assignee | ||
Comment 12•9 years ago
|
||
Try looks good and the stub was built. I'll land later tonight
Comment 13•9 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/638f0db43a3f https://hg.mozilla.org/integration/fx-team/rev/932176dec8d1
https://hg.mozilla.org/mozilla-central/rev/638f0db43a3f https://hg.mozilla.org/mozilla-central/rev/932176dec8d1
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
status-firefox41:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 41
Assignee | ||
Updated•9 years ago
|
status-firefox40:
fixed → ---
You need to log in
before you can comment on or make changes to this bug.
Description
•