Closed Bug 770896 Opened 8 years ago Closed 8 years ago

Win8 Elm zip installs / installer currently missing vccorlib110.dll

Categories

(Firefox :: General, defect)

x86_64
Windows 7
defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 16

People

(Reporter: jimm, Assigned: jimm)

References

Details

Attachments

(1 file)

Two issues here - 

1) The redist library vccorlib110.dll currently doesn't get included in the distribution.
2) To run as a metro enabled desktop browser, we rely on api-ms-win-core-* winrt runtime. The current win8 release doesn't include these bu default.

#1 is easy to address. #2 is a little strange. One guess is that they aren't distributing these files in the release previews but will in the final release of the os. 

Both of these issues are preventing set up pre-release win8 testing slaves which normally don't have visual studio installed.

https://wiki.mozilla.org/ReferencePlatforms/Test/WinXP

I would suggest we fix #1 on elm, ping ms to confirm the runtime will be in the final release, and try to find some way of getting the pre-release runtime installed without installing visual studio.

Note found this through a search, not sure what this is but it looks promising:

http://www.microsoft.com/downloads/details.aspx?FamilyID=78F5DE9A-47B6-4D3B-BEE6-D878E271832D&displaylang=ru&displaylang=en
FTR we might not be running the latest Win8 build:
http://cl.ly/HrLc
"Windows 8 Consumer Preview: Evaluation copy. Build 8250"

Maybe I can ask IT to install the latest copy?
Blocks: metro-releng
Confirmed that the Release Preview has the api-ms-win-core-* runtime installed. Morphing this bug.
Summary: Fresh installs of Win8 Release Preview are missing key runtime files → Win8 Elm zip installs / installer currently missing vccorlib110.dll
(In reply to Armen Zambrano G. [:armenzg] - Release Engineer from comment #1)
> FTR we might not be running the latest Win8 build:
> http://cl.ly/HrLc
> "Windows 8 Consumer Preview: Evaluation copy. Build 8250"
> 
> Maybe I can ask IT to install the latest copy?

Yep, we need to update these slaves to the release preview.
I have asked in bug 770953 for IT to re-image one of the machines.

Is there a workaround meanwhile?
(In reply to Armen Zambrano G. [:armenzg] - Release Engineer from comment #4)
> I have asked in bug 770953 for IT to re-image one of the machines.
> 
> Is there a workaround meanwhile?

For the consumer preview you would have to install visual studio I would think. Best option is to just upgrade to the release preview.  

For the vccorlib problem we need to land a patch on elm that adds vccorlib110.dll here - 

http://mxr.mozilla.org/mozilla-central/source/browser/installer/package-manifest.in#95
http://mxr.mozilla.org/mozilla-central/source/browser/installer/removed-files.in#1388
Attached patch patchSplinter Review
AFAICT there are two locations we need to mention this file.
Assignee: nobody → jmathies
Attachment #640198 - Flags: review?(robert.bugzilla)
https://hg.mozilla.org/projects/elm/rev/aeecd761a36d

I'll land this on mi once I've confirmed this dll is showing up in zip installs on elm.
(In reply to Jim Mathies [:jimm] from comment #7)
> https://hg.mozilla.org/projects/elm/rev/aeecd761a36d
> 
> I'll land this on mi once I've confirmed this dll is showing up in zip
> installs on elm.

I also had to add this file to the redist file copy logic in build/win32 - 

https://hg.mozilla.org/projects/elm/rev/8a1293d96653

This is working correctly on Elm now.
https://hg.mozilla.org/mozilla-central/rev/e308e75d1b5b
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 16
No longer blocks: 737975
Blocks: elm-merge
You need to log in before you can comment on or make changes to this bug.