-
Notifications
You must be signed in to change notification settings - Fork 75
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
player error #76
Comments
It works for me, but I have an old version of Xcode (7.0.1). I'll try it with a more recent version and see if I get the same failure. |
I just successfully built it on my virtual machine with the latest Xcode:
|
never mind, that machine doesn't have Xcode. I'll install it now... |
I've tested every permutation I can think of, and I can't reproduce this error. I'll punt by cc'ing @gerkey |
I haven't tried to reproduce it, but based on the fact that the linker errors are for the swig-generated bindings to
You could try that manually, then try it as a modification to the brew formula. (Of course, I'm assuming here that you don't need the Python or Ruby bindings. If you do need them, then we'll need to figure out what's causing them to fail to link.) |
More drastically, you could also uninstall |
yes!! without |
I'm now able to reproduce the problem by brew installing swig and then trying to brew install player. Turns out that there's already an issue for this upstream: playerproject/player#3. |
Is there a cmake argument we could pass to disable the swig bindings? |
@scpeters I didn't see a specific argument to disable swig, but adding these cmake args would disable both the swig-generated bindings that are causing trouble:
|
Ok, I added those cmake args to the player formula in 19922e1 |
I am not able to install player 3.0.2 on Yosemite. I get an error on building libplayerc++.dylib
My gist is here
I also tried the HEAD version:
brew install player --HEAD
but a patch error occurs.Is it a configuration problem?
Thanks,
Luigi
The text was updated successfully, but these errors were encountered: