-
-
Notifications
You must be signed in to change notification settings - Fork 53
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
Update readme to new build system #200
Conversation
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.
Hey, this change seems pretty good to me, but there's a few notes I have:
- You have added build instructions, but the old build instructions still seem to be there. They are listed right underneath the runtime dependencies.
- The new build instructions don't list the requirements for building on Windows. I think this is worth writing down?
- In "Supported Operating Systems", it still links to the Prerequisites section, which is missing.
The points @Lucky4Luuk makes here are very good, Especially the steps for windows as that would be me :) |
Sorry @Lucky4Luuk if its not clear, but the windows build works the same. That was most of the point of this rewrite of the build system - ill fix the point about the old instructions. |
So @Starystars67 the same steps on windows, you just open the cmakelists with visual studio and it should just work |
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.
Looks good to me now, I have no more comments!
Now building is like 2 steps, so the readme reflects that now.