-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Malformed path for Qt5::Core in build using macOS homebrew. Update build instructions? #2177
Comments
This is a macOS homebrew bug, it doesn't add the qt5 binaries to the path correctly. Can be worked around by manually adding /usr/local/opt/qt5/bin to the PATH. |
Closing as AFAIK this can't be fixed on our side. |
@gcp hmm... manually adding the qt5 bin path to my PATH did not work. Upon further inspection, it looks like it might be a bug in Qt5 itself, as it is referencing the malformed path. Thanks! |
Wipe out the build dir after fixing the PATH and rerun cmake. |
That worked for me, thanks! Would it be possible to add this to the macOS build instructions, as Homebrew is probably how users will be installing QT5 |
Ah I might have closed this prematurely. Apologies. |
thanks for remembering i dont use mac myself (i'm on linux), but i had already written these clearer instructions quite some time ago : #2071 i added specific mac instructions based on the feedback of mac users, see the PR discussion (i documented the links of the discussion and summarized it) the PR was then kinda forgotten for some reason |
Should probably try to find the bug filed against Homebrew and link to that. I'd rather not have done this but this issue has been there for so long it looks like they're unable to fix it. |
so, what do we do with the mac instructions ? put them back to default and ignore the mac users feedback ? personally both are fine to me because i dont use mac, but if it really helps to add these mac specific instructions i'd like to add them even if we remove them later on |
I think it makes sense that if a specific package manager is already mentioned in the build instructions, then other steps which are necessary when using the same should also be included. For example, these instructions would help new users from encountering the same problem I did: # Clone github repo
git clone https://github.com/gcp/leela-zero
cd leela-zero
git submodule update --init --recursive
# Install build depedencies using homebrew
brew install boost cmake qt5
# Use stand alone directory to keep source dir clean
mkdir build && cd build
# include QT5 in the PATH
PATH="$(brew --prefix qt5)/bin:$PATH" cmake ..
cmake --build .
./tests
curl -O https://zero.sjeng.org/best-network
./leelaz --weights best-network I've added |
except that qt5 is needed only if you want to use autogtp @rakenodiax if someone only wants leelaz without wanting to contribute, theres no need to bother him with qt5 ty for the feedback |
here are the mac instructions in my PR btw : |
@wonderingabout Thanks! Moving there |
i'll push a modification, tell me if its ok for you |
It looks good. You might want to use the |
https://github.com/wonderingabout/leela-zero/commit/923bd3f99fdb61025075a4989c00388632675a7a is it ok like that now ? i dont use mac so i'm clueless about how to use |
Wouldn't the PATH have to either be
No worries, I appreciate the responsiveness! |
@rakenodiax adding path way sooner or just sooner is the same, but again some mac users will want only welcome, i appreciate yours too 👍 |
@rakenodiax #2300 addresses this, and i hope fixes it 👍 |
@wonderingabout Looks great! Thanks! 👍 |
Ok, it is June 19 2019 and I am trying to install leela-zero 0.17 on macos 10.14.5. I followed the instructions in the README files, which do not match the revised instructions shown in this thread BTW. Homebrew did not install Qt5 so I installed it manually. (I did not know it was only needed for autogtp, which I did not know what it did until reading this thread). Anyways, I am not a programmer and I don't know Unix well either so this is hard for this Go player on macos to fix. My issue is that Qt5 is not in /usr/local/Cellar since I manually installed it. Here is the error log after I run "cmake .." (inside the build folder as instructed by the README included with 0.17. Benoits-MacBook-Pro:build Benoit$ cmake .. Could not find a package configuration file provided by "Qt5Core" with any
Add the installation prefix of "Qt5Core" to CMAKE_PREFIX_PATH or set CMake Warning at CMakeLists.txt:129 (message): -- Configuring done Thank you SO MUCH for all the contributors to make this project come to light. It is exciting! I still have not been able to achieve a successful install on macos for two years now). Thank you for the help! Benoît |
@bcordoba555 You can try running
And using the find command, located the first file to be in For posterity, I will record my solution to the original problem as well. I too faced the issue of the cmake files for qt5 listing several files that did not exist, including Combining information I found across various issues (including this one), I figured out that there was an error with some environment variable used by the cmake files, which was pointing to
Note that you may need to use |
Hello,
With the
next
branch checked out, I'm following the build instructions for macOS, and CMake is failing to build with the following error:I'm running macOS High Sierra version 10.13.6 (build 17G4015)
The text was updated successfully, but these errors were encountered: