-
-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
Ease building with custom includedir. #7316
Conversation
Cast @nalimilan |
Cc: @staticfloat |
@crayxt Could you give us a little more info as to what the idea behind this PR is? I think I agree with most of these changes, (except perhaps for the |
@staticfloat I apologize for not providing good explanation from the beginning. As for llvm-config change, it addresses the similar issue of hardcoded path, this time for binary files. If I change |
Makes much sense, but I think @staticfloat was curious about the situation in which you need it. It's always interesting to understand changes. BTW, if the llvm-config change is about binaries, you should make the commit message more general. |
Ah, I see; These are all good changes. It sounds like your |
@staticfloat Pull request updated. I couldn't find a better way other than using ifeq switch. |
I don't think your
I think it's because of these lines which unfortunately don't work on OSX because |
Do you really need to use |
@tkelman I think you are right, I will check |
@staticfloat Pull request updated. I suppose there is no reason to wait for Travis results, as it uses system libs. Sadly, I don't have OSX to check this. |
@$(call PRINT_PERL, $(CPP) -dM $(shell $(PCRE_CONFIG) --prefix)/include/pcre.h | perl -nle '/^\s*#define\s+PCRE_(\w*)\s*\(?($(PCRE_CONST))\)?\s*$$/ and print "const $$1 = uint32($$2)"' | sort > $@) | ||
else | ||
@$(call PRINT_PERL, $(CPP) -dM $(build_includedir)/pcre.h | perl -nle '/^\s*#define\s+PCRE_(\w*)\s*\(?($(PCRE_CONST))\)?\s*$$/ and print "const $$1 = uint32($$2)"' | sort > $@) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How delightfully ironic that we use these convoluted regexes to patch PCRE.
Oddly enough, I couldn't find the magic invocations to |
Ease building with custom includedir.
If you specify build_includedir to be inside of$(build_prefix)/usr and not in $ (build_prefix)/include then include files of gmp and LLVM are not installed to build_includedir but to $(build_prefix)/include.
The only thing missing is specifying pcre.h header file for pcre_h.jl. I am still thinking on it.