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

Package manager startup errors on 2019.1.8f1 on Windows 10 #22

Closed
gpoole opened this issue Jul 3, 2019 · 8 comments
Closed

Package manager startup errors on 2019.1.8f1 on Windows 10 #22

gpoole opened this issue Jul 3, 2019 · 8 comments

Comments

@gpoole
Copy link

gpoole commented Jul 3, 2019

When I'm opening a project in 2019.1.8f1 with FluvioFX installed I'm often getting EPERM: operation not permitted errors related to VFXCodeGenerator.cs. If I select quit and reopen Unity a few times it'll eventually work.

Capture2

@lilyvelour lilyvelour added this to In progress in FluvioFX Jul 5, 2019
@lilyvelour lilyvelour moved this from In progress to To do in FluvioFX Jul 6, 2019
@lilyvelour
Copy link
Contributor

I can't reproduce this one, but it may be fixed with #23.

@gpoole
Copy link
Author

gpoole commented Jul 6, 2019

I also didn't have this problem when I tried installing on my Mac, so could it be a Windows-specific problem? I know Windows is a lot more strict with locking files and preventing writes to files while they're being read by other processes, could it be something to do with that?

@lilyvelour
Copy link
Contributor

lilyvelour commented Jul 6, 2019

This might be specific to your system. Can you provide more details?

  • What is your Windows version?
  • Do you have anything unique about your project folder structure (is it under a symlink, on a different drive, etc.)?
  • Does this happen with a completely empty (new) project?

@gpoole
Copy link
Author

gpoole commented Jul 8, 2019

I'm running Windows 10 (1803). There isn't anything unusual I can think of about the setup, no symlinks or drive differences or anything odd. I was able to reproduce it with a new project just now, here's the manifest.json: https://gist.github.com/gpoole/691e900dc4a2b954b1c4c3cc47ea062c If it helps I can also push the whole project up as a repo?

I had more of a poke around and found:

  • If I disable the Fluvio auto installer by removing [InitializeOnLoad] and don't manually run it then ut restarts fine
  • If I manually modify the files the installer is changing by adding a comment to them, vscode tells me the file is read only and removes the read only flag when I save, but I still don't get any errors on restart
  • If I then manually run the installer from the menu I get the error after a restart

I wondered if it might be something to do with the readonly flag, so tried commenting out just the code in the installer that restores the readonly flag after modifying the files (e.g. I commented https://github.com/thinksquirrel/fluviofx/blob/master/Install/FluvioFXInstall.cs#L338) and it seems to be fine restarting after that.

@lilyvelour
Copy link
Contributor

Hm, this is likely the case. It could be that marking that as read-only again causes problems on some systems.

I don't think there is any problem with keeping the file writable, to be honest. So I'll just remove that line.

Thanks for troubleshooting this!

@lilyvelour
Copy link
Contributor

I've just made and pushed this update. Can you confirm that the PR works as-is? If so I'll go ahead and merge.

@lilyvelour lilyvelour moved this from To do to Needs review in FluvioFX Jul 15, 2019
@lilyvelour lilyvelour removed this from Needs review in FluvioFX Jul 15, 2019
@gpoole
Copy link
Author

gpoole commented Jul 16, 2019

Great, thanks for the fix, yep it's not showing the error anymore when I restart. Very strange, it doesn't seem like setting and unsetting the flag should have mattered.

@lilyvelour
Copy link
Contributor

Fixed by #23

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants