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

Release 15 of the FreeOrionSDK #127

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

o01eg
Copy link
Contributor

@o01eg o01eg commented Jan 3, 2025

No description provided.

@o01eg o01eg requested review from Vezzra and geoffthemedio January 3, 2025 04:36
@o01eg o01eg self-assigned this Jan 3, 2025
ChangeLog.md Outdated Show resolved Hide resolved
@o01eg o01eg requested a review from geoffthemedio January 3, 2025 09:17
@geoffthemedio
Copy link
Member

Is there a download, or should I build locally?

@o01eg
Copy link
Contributor Author

o01eg commented Jan 3, 2025

It was the same as my latest 17th SDK from that PR #126. I cleaned it as there only remaining thing is to make textual changes.

@geoffthemedio
Copy link
Member

It was the same as my latest 17th SDK from that PR #126. I cleaned it as there only remaining thing is to make textual changes.

Not sure if / how that answers my question, but it seems to build and work for building FreeOrion on my system.

@o01eg
Copy link
Contributor Author

o01eg commented Jan 4, 2025

@geoffthemedio Does other text look fine?

@geoffthemedio
Copy link
Member

Not sure what other text you mean...

maybe After that, the SDK can be build with: -> After that, the SDK can be built with:

Copy link
Member

@Vezzra Vezzra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can build the SDK, and I can successfully build and run your test-release-v15 branch with that SDK. I can also build current master with that SDK, however, the resulting app crashes on startup (missing the SDL library), but that's probably expected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants