-
Notifications
You must be signed in to change notification settings - Fork 143
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
Makefile refactors for global install and new sysroot #801
Conversation
Docker Images PR (which should work after this): #785 |
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.
This looks good. I had one minor suggestion, but it looks like that variable isn't really used anywhere, so I'm going to approve.
One other thing to note: You don't need to add -I include/kllvm
if you add kllvm/
before your includes to the runtime
directory. So it would be kllvm/runtime/header.h
. It's up to you though, you're free to keep the code with the change you just made as well.
This PR makes several changes in preparation for (i) updating the K submodule to the new sysroot, and (ii) using K Docker Images for CI (with a global K install).
Makefile
andkevm
. In order of priority, first it's (i) what the user sets environment variable to, (ii) the submodule if the submodule is present, and (iii) a global install of K found withwhich kompile
.git submodule update ...
are removed from theMakefile
, because the Makefile should not depend on the VCS that is used to store it. This also makes it easier to have a custom commit of K in the submodule without having to commit it (for example).LINK_PROCPS
is removed, in favor of aSTANDALONE_KOMPILE_OPTS
flag which holds all the kompile options for the standalone build (and adds-lprocps
for Linux systems).cmake/client
file is updated to include the paths needed for a global install of K.