Page MenuHomeSchine - Phabricator

Launcher | The launcher should ignore duplicate entries in the buildindex file
Closed, FinishedPublic

Description

An incorrect buildindex with multiple duplicate entries should not cause the launcher to misbehave and display them all.

Add some intelligence to the version parsing and remove duplicates.

Details

Commits
Restricted Diffusion Commit
Restricted Diffusion Commit
Task Type
Bug
Testing Results
Affected Gamemode(s)
none/unspecified
Reproducible
uncertain
Category
Launcher
Hardware/Software/System
OS-Specific
No
Hardware-Specific
No
Video Card Vendor
uncertain
Tester information (Internal use only)

Note:
version and build differ -- This ticket should apply to removing entries that are completely identical (path, version, and build all match), not just differing in the version that the launcher displays in the build-select dropdown.

(I may change how the launcher displays these versions in the future to prevent confusion, but that isn't the focus of this ticket)

Serverconfig (server.cfg)
<replace this line with the file content>
Clientconfig (settings.cfg)
<replace this line with the file content>

Event Timeline

calani updated the task description. (Show Details)Jan 29 2016, 1:30 AM
calani shifted this object from the S1 Public space to the S5 Launcher space.
calani changed Category from none/unspecified to Launcher.
calani edited Serverconfig (server.cfg). (Show Details)
calani edited Clientconfig (settings.cfg). (Show Details)
calani edited Tester information (Internal use only). (Show Details)
calani added a subscriber: calani.
calani created this task.
calani claimed this task.
calani raised the priority of this task from to High.
Restricted Application added a project: Issue Navigation. · View Herald TranscriptJan 29 2016, 1:30 AM
Restricted Application added a subscriber: AndyP. · View Herald Transcript
calani set Task Type to Bug.Jan 29 2016, 1:30 AM
calani set OS-Specific to No.
calani set Hardware-Specific to No.
calani set Video Card Vendor to uncertain.
calani set Affected Gamemode(s) to none/unspecified.
calani set Reproducible to uncertain.
calani changed the task status from Open to In Queue (Launcher).
Restricted Application edited projects, added Starmade, Launcher Development; removed Issue Navigation. · View Herald TranscriptJan 29 2016, 1:30 AM
calani added a commit: Restricted Diffusion Commit.Jan 29 2016, 1:38 AM
calani changed the task status from In Queue (Launcher) to Resolved by committing Restricted Diffusion Commit.
Restricted Application edited projects, added Quality Assurance; removed Launcher Development. · View Herald TranscriptJan 29 2016, 1:38 AM
calani added a commit: Restricted Diffusion Commit.Jan 29 2016, 7:52 AM

Works fine for the windows version. I'm not sure how it deals with hotfix release builds since those have the exact same release version as the previous one.
Would it filter out the older duplicates or ignore them?

AndyP changed the visibility from "Custom Policy" to "Public (No Login Required)".Mar 2 2016, 12:13 AM
AndyP changed the edit policy from "Task Author" to "Starmade (Project)".
AndyP moved this task from Unclassed to Issue affecting current release on the Starmade board.
AndyP claimed this task.
AndyP reassigned this task from AndyP to calani.
calani added a comment.Mar 2 2016, 1:29 AM

@lancake:
"This ticket should apply to removing entries that are completely identical (path, version, and build all match), not just differing in the version that the launcher displays in the build-select dropdown."

Basically, it should only disallow complete duplicates (not just duplicate version numbers), e.g. within http://files.star-made.org/devbuildindex

Matching version numbers with differing builds is instead covered by T1105 (adding hotfix indication) though honestly, that is just sugar and really should be handled with thein game's actual version numbers instead. (ex: using SemVer)

I second that recommendation to move to SemVer's scheme for version IDs, or at least something similar...

AndyP closed this task as Closed.Mar 5 2016, 9:39 PM

-QA Testing-
Already confirmed to work a while ago. =)

Restricted Application removed a project: Quality Assurance. · View Herald TranscriptMar 5 2016, 9:39 PM
Restricted Application removed a subscriber: AndyP. · View Herald Transcript