-
Notifications
You must be signed in to change notification settings - Fork 80
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
GitHub Landing Page updates #880
Conversation
<dl class="wider"> | ||
<dt>32-bit Debug:</dt> | ||
<dd><tt>out\install\x86-Debug\client</tt></dd> | ||
|
||
<dt>32-bit Release:</dt> | ||
<dd><tt>out\install\x86-Release\client</tt></dd> | ||
|
||
<dt>64-bit Debug:</dt> | ||
<dd><tt>out\install\x64-Debug\client</tt></dd> | ||
|
||
<dt>64-bit Release:</dt> | ||
<dd><tt>out\install\x64-Release\client</tt></dd> | ||
</dl> |
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.
I think this section needs more detail. When I initially saw it, my reaction was: "wait, what are we talking about?" There's no mention of arch or build types before now by default.
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.
I don't know how to tell people where to find the files when the output folder varies, without also mentioning the fact that it varies 😛
</li> | ||
|
||
<li> | ||
<p>Compile the project by using <strong>Build > Install Plasma</strong>. All required dependencies will automatically be downloaded from vcpkg as part of the build process.</p> |
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.
This will build a debug client.
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.
Is that what we want the default to be? Can we change the default by attempting to set a default CMAKE_BUILD_TYPE
?
Co-authored-by: Adam Johnson <AdamJohnso@gmail.com>
Co-authored-by: Michael Hansen <zrax0111@gmail.com>
Preview: https://dpogue.github.io/Plasma
Expands the GitHub landing page for the project to include information about downloading builds from CI runs, how to compile Plasma with the newer vcpkg build system, how to run Plasma builds locally, and how to get involved.
This is intended to be the canonical instruction guide for building and running Plasma, rather than having poorly maintained information scattered across GitHub readme files, GoW and OpenUru wikis, and IRC logs.