Skip to content

Comments

Update README.md examples and project structure#3

Open
DryByte wants to merge 4 commits intoopenmultiplayer:masterfrom
DryByte:update_example
Open

Update README.md examples and project structure#3
DryByte wants to merge 4 commits intoopenmultiplayer:masterfrom
DryByte:update_example

Conversation

@DryByte
Copy link

@DryByte DryByte commented Feb 16, 2026

The example in README.md is outdated, so you can't actually use it for creating a component from a "basic setup", having the need to dig a little bit in the server and the SDK code to understand what's happening...

And the target_link_libraries is trying to link the wrong library name, so i updated it as well.

Updated the project structure to include the tools folder

@DryByte
Copy link
Author

DryByte commented Feb 16, 2026

Just curious, but do we really need to add it as a subdirectory and link it? Since it's a single header, it will already be compiled by the program when the header is included.

Since the file isn't in the same folder,  doesn't make sense to use quotes instead of brackets.
@AmyrAhmady
Copy link
Member

Thanks for your contribution
And no, you don't have to do that, it's a single header file so you can just use it directly. CMake support is only added for those who are more comfortable with controlling it through CMake.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants