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

Some details didn't mentioned on the compiling guide about installing #863

Open
wzx1996 opened this issue Sep 19, 2016 · 0 comments

Comments

Projects
None yet
2 participants
@wzx1996
Copy link

commented Sep 19, 2016

After tons of fails attempting to compile and install GLFW via VS2015, I found out that, in order to compile successfully, the whole source directory must be put on a path that contains no space, otherwise some files can't be found during the installation.

Also, installing into "Program Files" directory requires admin privilege, which VS2015 doesn't acquire automatically. My workaround is, running cmake_install.cmake manually using a admin mode command prompt after building the library. However, I think that, maybe running the build tool in admin mode can solve this problem easier.

In order to prevent other people making the same mistake, maybe consider adding the following text into the compiling guide.


Generating build files with CMake

(...) Both of the paths must contain no space, otherwise the installation process after compiling may fail. (...)

Compiling the library

On Windows, CMAKE_INSTALL_PREFIX is set to C:\Program Files\GLFW ( C:\Program Files (x86)\GLFW when installing 32-bit version on 64-bit system) by default. In most cases, this directory requires administrator privilege to write into, therefore, if the installation process fails, run the build tool in administrator mode, or change CMAKE_INSTALL_PREFIX to something else. Alternatively, after building the "Release" branch, you may also run cmake -P cmake_install.cmake in the target path using command prompt in administrator mode to install the library manually.

@elmindreda elmindreda added the Windows label Sep 19, 2016

@elmindreda elmindreda changed the title Some details didn't mentioned on the compiling guide about installing on Windows Some details didn't mentioned on the compiling guide about installing Sep 19, 2016

@elmindreda elmindreda removed this from High Priority in Review Queue May 17, 2017

@elmindreda elmindreda added this to the 3.4 milestone May 17, 2017

@elmindreda elmindreda added the build label Oct 1, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.