-
-
Notifications
You must be signed in to change notification settings - Fork 968
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
Change: Always build baseset metadata, instead of including it in the repo. #7077
Conversation
0b70c54
to
8dca002
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where is openttd_vs140.sln ?
8dca002
to
6f5e5a6
Compare
Missed that. Have added it now, but haven't got a version that'll open it. |
Hmm, thinking about it, it probably needs to depend on all the language files, not just english.txt. Can't see how to do that, replacing english.txt with *.txt doesn't work at least! |
Tried VS2015, works |
I think you'd need, at least for MSBuild, a dummy-output generated from all the language inputs, and then you can depend other things on that dummy-output. |
… repo. This adds a script and project files to generate these files within MSVC.
6f5e5a6
to
635020c
Compare
Latest push adds code to build a list of language files and include it in the dependences. Seems to work for me with MSVC 2017. |
Seems to work with MSVC 2015 and MSYS2 |
I guess the final question now is... is this change a good thing to do? Seeing as eints likely doesn't have the build environment to update these files, this seems the best way to me. |
Yeah, IMO this is good. It's less prebuilt files to host in the repository. If it can reasonably be generated during build it should. |
Baseset metadata files are usually out of date as they are not updated when translations are changed.
This also adds a script and project files to generate these files within MSVC.