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

Bananas: "Select upgrade" for content tries to download old versions #6383

Closed
DorpsGek opened this issue Oct 25, 2015 · 4 comments
Closed

Bananas: "Select upgrade" for content tries to download old versions #6383

DorpsGek opened this issue Oct 25, 2015 · 4 comments
Labels
bug Something isn't working flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) needs triage This issue needs further investigation before it becomes actionable stale Stale issues

Comments

@DorpsGek
Copy link
Member

Moriarty opened the ticket and wrote:

r27407

A number of NewGRF's and other content on the bananas system have poor version numbers and this means players are downloading and using the wrong versions.
I.e.:

- I have the North American Renewal Set 2.5 and 2.51 installed. Yet when I click "select upgrade", OTTD selects version 2.03.

- Industrial Stations renewal 1.0.2 wants to "upgrade" to 0.8.1

- ab8 Aviators Aircraft Set 2.0 and 2.21 want to upgrade to 1.81

- OpenGFX+ Landscape 1.1.2 wants to upgrade to 0.2.3

-----

I'd suggest there are two problems here:

a) Old versions remaining on Bananas. I'd suggest only allowing one version of any given NewGRF/content to be active on bananas at any given time.

b) The version system using by bananas is inadequate. Possible solution - utilise semantic versioning (http://semver.org/) or some other sort of consistent versioning system for upgrades and require that all uploaded content adhere to it. And/or Flag if an uploaded version has a "lower" number than the current version.

For bonus confusion, if I download say the "upgrade" 0.2.3 OpenGFX+ Landscape, in the NewGRF Settings, only 1.1.2 shows. So that part of the game is able to correctly handle the versioning, it's the download client that fails.

Reported version: trunk
Operating system: All


This issue was imported from FlySpray: https://bugs.openttd.org/task/6383
@DorpsGek
Copy link
Member Author

DorpsGek commented Nov 2, 2015

planetmaker wrote:

I don't quite follow, nor can I reproduce your observation.

Are you sure you didn't try to download NewGRFs when you entered the download dialogue via the load savegame dialogue or when joining a server which need those exact versions?

How is the versioning of the NewGRFs you cite anything but the semantic versioning?

Yes, however, there are two systems in place which determine 'new content': one on the server, one by the client:

OpenTTD iteself allows NewGRFs to define an integer version which does not need to be sequential. What values are assigned there is up to the author, but the highest value will be interpreted as the newest version by OpenTTD - a system which generally works well. This version is used to show the NewGRFs available ingame

OpenTTD's content server offers as update always the newest uploaded version, if the client doesn't have that very same version already. Thus an update may not necessarily reflect the newest version as given by the NewGRF itself. But that then is the concious choice of the NewGRF author. It's their responsibility to upload only newer contentent than previous versions to bananas.

Attachments


This comment was imported from FlySpray: https://bugs.openttd.org/task/6383#comment14077

@DorpsGek
Copy link
Member Author

Airbus wrote:

I must say I had exactly same problem with updating NewGRF. Try selecting ECS packages of version 1.2, downloading them, and then selecting "Add upgrades". For me it selected versions 1.1.2. Version 1.2 is dated at 2012, version 1.1.2 is dated at 2009-2010. Maybe it had to do with bigger number of digits in version, i.e. 12 versus 112.


This comment was imported from FlySpray: https://bugs.openttd.org/task/6383#comment14115

@DorpsGek
Copy link
Member Author

andythenorth wrote:

I can't repro the case by Airbus. I have ECS Agricultural Vector 1.1.2, I select upgrades, it offers 1.2. I have ECS Basic II Vector 1.2, I select upgrades, no upgrade is offered.

I repro two of the cases by Moriarty:
- I have ISR 1.0.2, I select upgrades, no upgrade is offered.
- I have NARS 2.5.1. I select upgrades, 2.0.3 is offered (as an additional download, not replacing the 2.5.1 grf).
- I have OpenGFX+ Landscape 1.1.2, I select upgrades, no upgrade is offered.
- I have AV8 2.2.1, I select upgrades, 1.81 is offered (as an additional download, not replacing the 1.81 grf).

I've attached screenshots showing NARS and AV8 grf properties. All downloaded via bananas. NARS 203 lacks a version in action 14. But AV8 provides A14 version in both cases.

Wondering if it's a manifestation of #6377? But eh, could be completely unrelated.

Attachments


This comment was imported from FlySpray: https://bugs.openttd.org/task/6383#comment14625

@DorpsGek DorpsGek added Core flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) labels Apr 7, 2018
@TrueBrain TrueBrain added needs triage This issue needs further investigation before it becomes actionable bug Something isn't working and removed bug from FlySpray labels Apr 12, 2018
@frosch123 frosch123 removed the Core label Apr 14, 2018
@andythenorth andythenorth added the stale Stale issues label Jan 5, 2019
@andythenorth
Copy link
Contributor

Thanks for this. At the time of writing this appears to be an isolated bug, and hard to reproduce. We do try to address bugs where they occur frequently and/or are easy to reproduce, but this one doesn't look like we can any go further, so I'm closing it. Thanks for contributing!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) needs triage This issue needs further investigation before it becomes actionable stale Stale issues
Projects
None yet
Development

No branches or pull requests

4 participants