Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"IPersistFile::Save failed" during installation of VS Code Insiders #104545

Closed
Maxim-Mazurok opened this issue Aug 13, 2020 · 31 comments · Fixed by #105003
Closed

"IPersistFile::Save failed" during installation of VS Code Insiders #104545

Maxim-Mazurok opened this issue Aug 13, 2020 · 31 comments · Fixed by #105003
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug candidate Issue identified as probable candidate for fixing in the next release insiders-released Patch has been released in VS Code Insiders install-update VS Code installation and upgrade system issues verified Verification succeeded windows VS Code on Windows issues

Comments

@Maxim-Mazurok
Copy link

  • VSCode Version: 1.48.0-insider (user setup)
  • OS Version: Windows_NT x64 10.0.19041

Commit: db40434
Date: 2020-08-11T23:21:50.282Z
Electron: 7.3.2
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0

Steps to Reproduce:

  1. Download and run "VSCodeUserSetup-x64-1.48.0-insider.exe"

image

I'm not sure if it's relevant, but I had VS stable installed prior to the Insiders install. Also, I tried to remove stable and reinstall Insiders, which resulted in the same error. It seems like everything is functioning properly, but there's no shortcut on the desktop.

Does this issue occur when all extensions are disabled?: Yes

@llagerlof
Copy link

Windows 8.1 64

Same is happening with me, but with the stable installer (VSCodeUserSetup-x64-1.48.0.exe).

@llagerlof
Copy link

llagerlof commented Aug 13, 2020

I tried to uninstall and install again, but checked only the PATH option. All other options was left unchecked. No error happend. I don't know if was coincidence or that others checked options were causing the error.

@Maxim-Mazurok
Copy link
Author

I believe "Create desktop shortcut" option may be producing this error.

@ReyOscuro9999
Copy link

Same happened installing Visual Studio Code. The thing is it happened with a pretty old Windows (not that that old but quite) but it still let me download it. What does this mean then? I had no problem installing it in another more modern computer but I have no idea what is wrong here...

@cuibty
Copy link

cuibty commented Aug 16, 2020

I also encountered the same problem
Win10 2004 (19041.388)

@yabasha
Copy link

yabasha commented Aug 16, 2020

me too

image

@Seralyn
Copy link

Seralyn commented Aug 17, 2020

Same exact error, though I installed it on another PC in my network without issue- same version of Windows, same options checked.

@BertV1
Copy link

BertV1 commented Aug 17, 2020

Same exact error, also happened during creation of the shortcut.

@hexaleon
Copy link

hexaleon commented Aug 18, 2020

Having the same issue. Been using it for over an year but never faced any issue like this. It got updated yesterday when I was working and after that when I tried to use live server, the chrome tab keeps on loading and CPU usage gets above 90% for chrome. Strange.
Up till now, I have tried re-installation (thats when "ipersistfile save failed code 0x80070005" error occured), installing previous version, window update. Nothing seems to work and I think live server issue is happening because of it.
Any idea?

Update: So I tried a suggestion to leave the icon option on desktop unchecked and the error did not occur this time. Somethings wrong with my PC or its a bug? Also, live server issue remains same. Can someone check if they are facing the same issue?

@zeronyms
Copy link

create dekstop shortcut made this error ??

@AlexStox
Copy link

I have the same issue on Windows 7 - and seems like the checked "install link on the desktop" causes the problem, as at the end of installation VS saying "installed fine" and it's running OK.

@isidorn
Copy link
Contributor

isidorn commented Aug 18, 2020

Users seeing this with vscode insiders when did this start happening?
Users seeing this with vscode stable, do you also see it when you install vscode insiders? https://code.visualstudio.com/insiders/

@bpasero for potential smartness
@deepak1556 even though it is not an electron crash might have some ideas since it looks like a native crash

@deepak1556
Copy link
Contributor

Running with VSCodeUserSetup-x64-1.48.0.exe /LOG="D:\setup.log"

setup.log

2020-08-18 10:50:23.263   -- Icon entry --
2020-08-18 10:50:23.263   Dest filename: C:\Users\Public\Desktop\Visual Studio Code.lnk
2020-08-18 10:50:23.264   Creating the icon.
2020-08-18 10:50:23.923   Exception message:
2020-08-18 10:50:23.923   Message box (OK):
                          IPersistFile::Save failed; code 0x80070005.
                          Access is denied.

in my case it is a permission issue to the folder it is trying to write the shortcut link to. Users having similar error code please check if the setup executable under your current user has permissions to write into that folder.

If anyone else has a different error code, please paste the inno setup log using /LOG switch as above.

@deepak1556 deepak1556 added info-needed Issue requires more information from poster install-update VS Code installation and upgrade system issues windows VS Code on Windows issues labels Aug 18, 2020
@JohnHammell
Copy link

JohnHammell commented Aug 19, 2020

Same issue when installing VSCodeUserSetup-x64-1.48.0.exe stable. After clicking 'OK' everything else appeared to install correctly.

image

@bpasero
Copy link
Member

bpasero commented Aug 19, 2020

I saw this error as well when I tested the installer (using a Windows VM on Parallels 10).

@muhammad-farhan-bakht
Copy link

Same Issue with me today, as @Maxim-Mazurok suggested I install it again without "Creating desktop shortcut" worked and install without any errors.

@deepak1556
Copy link
Contributor

Sorry about the early conclusion, a bit more investigation looking at the logs of 1.47.3 installation, it does install the shortcut link under the desktop for current user rather than public folder. So this behavior change with 1.48 is indeed a bug.

Looking at the changelog, the only relevant changes that went into the setup code path are 5f5b75a cbc8bd8 , will check if the change in setup style brought in this behavior.

@deepak1556 deepak1556 added bug Issue identified by VS Code Team member as probable bug candidate Issue identified as probable candidate for fixing in the next release and removed info-needed Issue requires more information from poster labels Aug 19, 2020
@deepak1556 deepak1556 added this to the August 2020 milestone Aug 19, 2020
@isidorn
Copy link
Contributor

isidorn commented Aug 19, 2020

@deepak1556 thanks for looking into this. I have cycles so let me know if I can help somehow.

@isidorn
Copy link
Contributor

isidorn commented Aug 19, 2020

Dear users that see this can you please try to install this insiders build and let us know if you can still reproduce. This would be very valuable to us as it contains a potential fix for this issue:

As far as I understand the issue this only happnes for the user install. If somebody sees this issue for a non user install please let us know.

@Maxim-Mazurok
Copy link
Author

Dear users that see this can you please try to install this insiders build and let us know if you can still reproduce.

Tried to install your x64 version over my existing installation, same error:

image

@isidorn
Copy link
Contributor

isidorn commented Aug 19, 2020

@Maxim-Mazurok thanks a lot. We figured out that it did not fix the issue.
We have another candidate that is being built now, soon it will be out and I will share the links so you can try again.
Really appreicate the help.

@isidorn
Copy link
Contributor

isidorn commented Aug 19, 2020

@isidorn
Copy link
Contributor

isidorn commented Aug 19, 2020

Win64 build: https://az764295.vo.msecnd.net/insider/d514f5cbfc789c2f67f583ec20dc129ee7756546/VSCodeUserSetup-x64-1.48.1-insider.exe

So please try it out and let us know.

What is not 100% clear to me are repro steps. It only happens with the user install that creates a desktop shortcut.
@deepak1556 did I get the steps correctly? Just wondering since there are not that many users that complain...

Edit: yeah now the steps are 100% clear to me.
User install + check off create a desktop shorcut -> repros always.
However the VS Code still gets installed, just the desktop shortuct does not get created.

@isidorn
Copy link
Contributor

isidorn commented Aug 19, 2020

I have verified that the Inno-updater revert of version indeed fixes the issue.

@muhammad-farhan-bakht
Copy link

Win64 build: https://az764295.vo.msecnd.net/insider/d514f5cbfc789c2f67f583ec20dc129ee7756546/VSCodeUserSetup-x64-1.48.1-insider.exe
So please try it out and let us know.

Worked for me without any error now with Check in to create a desktop shortcut.

Screenshot

@isidorn
Copy link
Contributor

isidorn commented Aug 19, 2020

Great. It also worked for me.

So to recap:
The inno-updated update broke this. We are looking into two options for the recovery build:

  1. disable the create shortcut button for this release
  2. fix the actual issue captured by Inno breaking change with 6.0.0 for mapping user form of common folder constants #104981

We will decide on which one to do depending on the risk.
We will not revert back to the old inno-update version due to other chagnes that have gone in.

fyi @lszomoru @deepak1556

@Maxim-Mazurok
Copy link
Author

It only happens with the user install that creates a desktop shortcut.

Correct, I did check "Create desktop shortcut"

Win64 build: https://az764295.vo.msecnd.net/insider/d514f5cbfc789c2f67f583ec20dc129ee7756546/VSCodeUserSetup-x64-1.48.1-insider.exe

Worked for me without any errors, created shortcut. Thanks!

@lszomoru
Copy link
Member

Thanks @Maxim-Mazurok! We have a PR with a fix that will be included in the recovery build.

@Iceberg86300
Copy link

All with all options checked:

VSCodeUserSetup-x64-1.48.0.exe >> error
VSCodeUserSetup-x64-1.48.0.exe >> run as admin >> "you should use system" popup >> no error
VSCodeSystemSetup-x64-1.48.0.exe >> no error
VSCodeUserSetup-x64-1.48.1-insider.exe >> no error

@lszomoru lszomoru added the verification-needed Verification of issue is requested label Aug 20, 2020
@lszomoru
Copy link
Member

Verification steps:

  • Uninstall VS Code
  • Download user setup
  • In setup choose the option to create a desktop icon
  • Complete setup wizard

Setup should complete successfully, and a desktop icon should be created

@alexr00 alexr00 added the verified Verification succeeded label Aug 20, 2020
@lszomoru lszomoru removed the verification-needed Verification of issue is requested label Aug 20, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Oct 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug candidate Issue identified as probable candidate for fixing in the next release insiders-released Patch has been released in VS Code Insiders install-update VS Code installation and upgrade system issues verified Verification succeeded windows VS Code on Windows issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.