Update README.md to include x64 build guides #17
Conversation
README.md
Outdated
| 2. Download the [EditorConfig plugin for Notepad++][] and extract it. | ||
| 3. Open `cmake-gui`, select the extracted folder as source code directory, and create a directory to build the binaries. Then click the `Configure` button. When asked for a generator, select a generator without suffix to build the x86 version, or a generator with `Win64` suffix to build the x64 version. |
There was a problem hiding this comment.
Do you wanna say the BUILD_TYPE should be set to Release?
There was a problem hiding this comment.
-
I think
Releaseshould be used to compile the dll for daily use. Release builds should be smaller and runs faster than Debug builds. However using the Debug one is also fine. -
There are 2 places where we can change the option. The first place is when we "Configure" the CMake project, we can specify a set of available configuration types in the
CMAKE_CONFIGURATION_TYPESoption. The second place is when we opened the project using VS, there is a drop down menu on the toolbar so we can select the configuration from the set.CMAKE_CONFIGURATION_TYPESdefaults toDebug;Release;MinSizeRel;RelWithDebInfoand VS defaults to the first one, which isDebug. -
Maybe I should call it "configuration type" instead of "build type". Do I need to change it or just remove the suggestion to use
Release?
There was a problem hiding this comment.
- Please just mention the existence of
Release. Otherwise, many people may forget. - Either place is OK.
- Sure, as long as we use some name that is consistent with what is shown in the cmake GUI/VS, it is OK.
|
Thanks! |
No description provided.