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

Update documentation on how to upgrade #896

Closed
LEV3LGames opened this issue Aug 25, 2018 · 5 comments
Closed

Update documentation on how to upgrade #896

LEV3LGames opened this issue Aug 25, 2018 · 5 comments
Labels

Comments

@LEV3LGames
Copy link

@LEV3LGames LEV3LGames commented Aug 25, 2018

Prerequisites

Tracing and log file not applicable, as it is not an issue of the implementation, it is an issue of installing / adding to project.

Description

I wanted to add Github for Unity to my project importing it from the Asset Store. After import I got the message that there is a newer version of it, which I wanted. It took me to a page to download it as a Unity package, so I selected to open it with Unity (in the same project) and import it. This is where the problem happened: the DLL sfw_x64.dll could not be updated, because it got loaded into memory

Steps to Reproduce

  1. Import an older version than the latest of Github for Unity to your Unity project
  2. Agree to download the newer version and try to import it
  3. Observe the error message

Expected behavior: Github for Unity can upgrade without errors OR Asset Store automatically has the latest version (which would eliminate this issue)

Actual behavior: Github for Unity cannot upgrade automatically, because DLLs stay in memory of UNity once loaded

Reproduces how often: 100% (always)

Additional Information

I'm using Unity 2018.2.0f2, tried to import Github for Unity version 1.0.0 (version in Asset Store), newer version is 1.0.2.

@StanleyGoldman
Copy link
Member

@StanleyGoldman StanleyGoldman commented Aug 27, 2018

Hey @LEV3LGames it's a known issue. It isn't your fault you hit this problem. This is something we need to document and handle better.

You have two choices:

  1. Delete the plugin while Unity is not running. Then install the plugin after they start Unity.
  2. You can also exclude those sfw_*.dll files from the update installer, as they don't normally change.

Our code uses an un-managed library to perform file watching and we can't unload it when people try to upgrade and they are running Unity at the same time.

@StanleyGoldman StanleyGoldman added bug documentation and removed bug labels Aug 27, 2018
@StanleyGoldman StanleyGoldman changed the title Upgrading from package after importing from Asset Store fails Update documentation on how to upgrade Aug 27, 2018
@StanleyGoldman
Copy link
Member

@StanleyGoldman StanleyGoldman commented Aug 27, 2018

And actually, I just went and tested this flow in updating 1.0.1 to 1.0.2 in Unity 2017.2
It did seem to realize that the files had not been changed and actually did not let me check them.
Not every game and/or version of Unity is going to be as lucky, but it should definitely work with those files unchecked.

image

@LEV3LGames
Copy link
Author

@LEV3LGames LEV3LGames commented Aug 28, 2018

Hey @StanleyGoldman! I managed to solve it the same way that you mentioned, but thanks for clarifying it! Actually the idea of creating this issue was to call this problem to your attention.

It is interesting that upgrading from 1.0.1 to 1.0.2 Unity 2017.2 recognized that the file has not changed. It made me realise that I tried to upgrade from 1.0.0 to 1.0.2, but in the description I wrote ("an older version". So either the sfw_x64.dll changed between 1.0.0 and 1.0.1, or the importing works worse in Unity 2018.2.

@StanleyGoldman
Copy link
Member

@StanleyGoldman StanleyGoldman commented Aug 28, 2018

@LEV3LGames thanks for hanging in there. I hope it is all smooth sailing from here on in!

The best way to get our attention is always with an issue. I'm in discord, but I'm not actively using it so the notifications slip by me sometimes.

Usually we write in the release notes when the sfw library has changes, but I will also double check the import process in 2018.2.

Cheers! 🍻

@shana shana closed this in #897 Aug 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.