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

OS X with homebrew llvm -- pip install fails #76

Closed
defjaf opened this issue Jul 3, 2015 · 3 comments
Closed

OS X with homebrew llvm -- pip install fails #76

defjaf opened this issue Jul 3, 2015 · 3 comments

Comments

@defjaf
Copy link

defjaf commented Jul 3, 2015

When I try to install the latest llvmlite 0.6.0 with homebrew llvm 3.6.1, after setting LLVM_CONFIG=/usr/local/Cellar/llvm/3.6.1/bin/llvm-config I get the following error:

    initfini.cpp:9:10: fatal error: 'llvm/Config/config.h' file not found
    #include "llvm/Config/config.h"

It appears that /usr/local/Cellar/llvm/3.6.1/include/llvm/Config/llvm-config.h exists, but not config.h. This can be fixed with a softlink, but it is probably an error either in the homebrew installation or the initfini.cpp file. I have also reported a version of this error to the homebrew repo.

@pitrou
Copy link
Contributor

pitrou commented Jul 4, 2015

As I answered on the other bug, this is probably because homebrew builds LLVM using cmake, while llvmlite assumes Unix builds use the autotools (./configure) method.
We could probably do better here. Out of curiosity, can you paste the contents of your llvm-config.h somewhere?

@tamird
Copy link

tamird commented Jul 5, 2015

llvm-mirror/llvm@90c2949#diff-b67911656ef5d18c4ae36cb6741b7965R30 suggests that config.h is never intended to be used outside the LLVM build system. llvm-config.h seems to be safe, though.

@pitrou
Copy link
Contributor

pitrou commented Jul 5, 2015

Yes, we can probably switch to always use llvm-config.h now that we use LLVM 3.6. config.h was required on autotools builds with LLVM 3.5.

@pitrou pitrou closed this as completed in e14abc0 Jul 6, 2015
pitrou added a commit that referenced this issue Jul 6, 2015
Fix #76: always use llvm-config.h, not config.h
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants