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

Update osquery-toolchain to the 1.1.0 version #6315

Merged

Conversation

Smjert
Copy link
Member

@Smjert Smjert commented Mar 19, 2020

This new toolchain contains a newer LLVM version (9.0.1),
a fix for the scan-build scripts and it keeps the LLVM static libraries,
necessary to implement the new BPF framework and tables.

@Smjert Smjert added build Linux do not merge Do not merge PR as it's pending on some discussion or external factor. Reviewer should have context. labels Mar 19, 2020
@Smjert
Copy link
Member Author

Smjert commented Mar 19, 2020

Forgot to push the container, I'll wait for it to finish and rerun the failed jobs.

I marked it do not merge because it requires a last pass/commit before being merged.

theopolis
theopolis previously approved these changes Mar 20, 2020
This new toolchain contains a newer LLVM version (9.0.1),
a fix for the scan-build scripts and it keeps the LLVM static libraries,
necessary to implement the new BPF framework and tables.
@Smjert Smjert force-pushed the stefano/build/update-osquery-toolchain branch from 930ccc1 to ec385e7 Compare March 23, 2020 19:19
@Smjert Smjert removed the do not merge Do not merge PR as it's pending on some discussion or external factor. Reviewer should have context. label Mar 23, 2020
@Smjert
Copy link
Member Author

Smjert commented Mar 23, 2020

Now the PR is ready, with the correct Docker image name.

@Smjert Smjert merged commit 7ff9cf5 into osquery:master Mar 24, 2020
@Smjert Smjert deleted the stefano/build/update-osquery-toolchain branch April 2, 2020 22:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants