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

fix: append build number to version in JenkinsRepositoryAdapter #631

Merged
merged 2 commits into from
Feb 21, 2021

Conversation

skaldarnar
Copy link
Member

All preview builds of the Omega bundle triggered from engine's develop
branch have the same display version string (currently "alpha-18"). As
this version string is not unique, the launcher treats all those
releases as the same (at least in the UI).

This PR appends the build number to the version string in the
GameIdentifier to be able to distinguish the builds. It's only a quick
fix as the launcher should not mangle with version IDs on its own, but
just consume them from whatever repository source it connects to...

Quick fix for #630

All preview builds of the Omega bundle triggered from engine's `develop`
branch have the same display version string (currently "alpha-18"). As
this version string is not unique, the launcher treats all those
releases as the same (at least in the UI).

This PR appends the build number to the version string in the
GameIdentifier to be able to distinguish the builds. It's only a quick
fix as the launcher should not mangle with version IDs on its own, but
just consume them from whatever repository source it connects to...

Quick fix for #630
@skaldarnar skaldarnar added the Type: Bug Bug reports for launcher releases (reproducible from master) label Feb 20, 2021
@jdrueckert jdrueckert merged commit 7c5be81 into master Feb 21, 2021
@jdrueckert jdrueckert deleted the fix/indistinguishable-game-releases branch February 21, 2021 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Bug reports for launcher releases (reproducible from master)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants