notes icon indicating copy to clipboard operation
notes copied to clipboard

App is crashing on Windows 7

Open nuttyartist opened this issue 6 years ago • 12 comments

A friend of mine tried to install the app, but get an error message that says that "MSVCP120.dll is missing".

nuttyartist avatar Oct 21 '17 17:10 nuttyartist

@theshadowx Maybe it's because our config on AppVeyour doesn't include that? Should we include that?

nuttyartist avatar Oct 21 '17 17:10 nuttyartist

I think the installer you made doesn't include it, as windeployqt doesn't add it. In the appveyor, I have manually added it to the folder from which the bunder will be generated : https://github.com/nuttyartist/notes/blob/dev/appveyor.yml#L63-L67

  - cd C:\ci\notes\bin
  - windeployqt .
  - if %compiler%==msvc2015 copy "C:\\Program Files (x86)\\Microsoft Visual Studio 14.0\\VC\\redist\\x86\\Microsoft.VC140.CRT\\msvcp140.dll" .
  - if %compiler%==msvc2015 copy "C:\\Program Files (x86)\\Microsoft Visual Studio 14.0\\VC\\redist\\x86\\Microsoft.VC140.CRT\\vcruntime140.dll" .
  - if %compiler%==msvc2015_64 copy "C:\\Program Files (x86)\\Microsoft Visual Studio 14.0\\VC\\redist\\x64\\Microsoft.VC140.CRT\\msvcp140.dll" .
  - if %compiler%==msvc2015_64 copy "C:\\Program Files (x86)\\Microsoft Visual Studio 14.0\\VC\\redist\\x64\\Microsoft.VC140.CRT\\vcruntime140.dll" .
  - iscc C:\ci\notes\packaging\windows\Notes_Inno_Script.iss
  - cd C:\ci\notes\installer

theshadowx avatar Oct 21 '17 17:10 theshadowx

But it states "MSVCP120.dll" on my friend's machine, not msvcp140.dll. And indeed, my fault, I probably forgot to add these necessary libraries.

nuttyartist avatar Oct 21 '17 17:10 nuttyartist

yes because you used msvc 120 (2013) to compile the source code and the version used in appveyor is 140 (2015).

the dll follows the version of the compiler which you used

msvc 120 (2013) needs msvcp120.dll / vcruntime120.dll msvc 140 (2015) needs msvcp140.dll / vcruntime140.dll

theshadowx avatar Oct 21 '17 18:10 theshadowx

just to check : go to your appveyor Notes page and select

selection_334

You'll find in Artifacts tab:

appveyor_artifacts

Donwload the one with long name.

And try it, in theory it won't tell you that dll is missing.

theshadowx avatar Oct 21 '17 18:10 theshadowx

Ok, I will check with my friend and update.

nuttyartist avatar Oct 21 '17 18:10 nuttyartist

Yeah, he said the link solved its problem. @theshadowx Are we still unable to create one Windows installer using AppVeyour (for both 32bit and 64bit)?

nuttyartist avatar Oct 24 '17 05:10 nuttyartist

I haven't found a solution yet.

theshadowx avatar Oct 24 '17 08:10 theshadowx

Ok, so I will create a new installer and upload it to the site. The real bummer here is that a new installer will probably lose all the popularity gained with the previous one, and thus, will be affected by Windows warnings. My fault on this.

nuttyartist avatar Oct 24 '17 10:10 nuttyartist

@theshadowx Is it good to use the AppVeyor's artifacts for the installer, rather for me to compile them every time on both 32bit and 64bit, and only packaging them together on 32bit VM?

nuttyartist avatar Nov 04 '17 14:11 nuttyartist

I think it's better to use the package generated from appVeyor, as it's clean environment.

theshadowx avatar Nov 04 '17 15:11 theshadowx

Ok 👍

nuttyartist avatar Nov 04 '17 15:11 nuttyartist

This was fixed by #399, so let's close this.

guihkx avatar Jan 15 '23 20:01 guihkx