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

Popup with showing new version, but this is the newest version #289

Closed
supernevi opened this issue Apr 5, 2022 · 6 comments · Fixed by #299
Closed

Popup with showing new version, but this is the newest version #289

supernevi opened this issue Apr 5, 2022 · 6 comments · Fixed by #299

Comments

@supernevi
Copy link
Contributor

Issue type: Bug report

Actual behaviour

On Windows 10 64bit with the current version:

image

Expected behaviour

Popup should not be shown currently and version number in left bottom corner should show the right version

Steps to reproduce

  1. Go to https://github.com/UltraStar-Deluxe/Play/releases
  2. Download StandaloneWindows64.zip
  3. Start the game

the commit hash seems to be right

@supernevi supernevi changed the title Popup with showing new version, but this is the current version Popup with showing new version, but this is the newest version Apr 5, 2022
@achimmihca
Copy link
Collaborator

achimmihca commented Apr 5, 2022

Whoops! How did the wrong version number end up in the Windows build?
Will take a look at it soon.

Are the other builds with correct version number?

@l3ztum
Copy link

l3ztum commented Apr 5, 2022

@achimmihca linux build is also effected
Screenshot from 2022-04-05 16-34-27

@achimmihca
Copy link
Collaborator

The version was set already in January

Weird.

@achimmihca
Copy link
Collaborator

The new GitHub Actions build artifacts have the correct version number.
I don't know what went wrong in the release.

Will check this when creating the next release.

@achimmihca
Copy link
Collaborator

Indeed, the new builds have again the wrong version number. It shows 0.4.0 but VERSION.txt already has 0.5.0.

Furthermore, the git commit hash is missing now because of a Git security update that prevents running Git commands as before.
See

Guess I have to setup docker to debug this. Sucks...

@achimmihca
Copy link
Collaborator

achimmihca commented May 11, 2022

I debugged the issue, not via docker, but via GitHub Actions and Debug.Log statements in a dedicated fork.

Looks like the Unity API returns an old value for Application.version and PlayerSettings.bundleVersion.
But the bundleVersion value in the file ProjectSettings/ProjectSettings.asset is correct.
No idea where Unity's C# API gets the old value from. I consider this a Unity bug.

Anyway, it works when fetching the value directly from the file instead of using Unity's API.

The commit hash can also be read from a file inside .git folder to avoid the issue with Git.

Will open a PR soon.

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

Successfully merging a pull request may close this issue.

3 participants