Skip to content

chore: Use stable Rust 1.38.0 and update Linux headers for x86_6…#945

Merged
LucioFranco merged 4 commits intomasterfrom
unknown repository
Oct 1, 2019
Merged

chore: Use stable Rust 1.38.0 and update Linux headers for x86_6…#945
LucioFranco merged 4 commits intomasterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Sep 27, 2019

No description provided.

Signed-off-by: Alexander Rodin <rodin.alexander@gmail.com>
Signed-off-by: Alexander Rodin <rodin.alexander@gmail.com>
@LucioFranco
Copy link
Copy Markdown
Contributor

@a-rodin can we break out the toolchain build part of this and the part that adds the dependencies for vector? We could probably at some point extract our musl toolchain so that others can use it. What do you think?

Signed-off-by: Alexander Rodin <rodin.alexander@gmail.com>
@ghost
Copy link
Copy Markdown
Author

ghost commented Sep 27, 2019

@LucioFranco Actually this builder is already not very Vector-specific. The only part that can be considered external is OpenSSL, which can be extracted easily (but those who want to use OpenSSL available for C libraries would need to do the same patching as we do here in order to compile it).

The idea of making it available for use by others is great. I think it might be worth to decide how is it going to be shared to clarify requirements for that toolchain image. I see three approaches:

  • keep it in Vector's repository, but separated and published to Docker Hub;
  • keep it in a separate repository;
  • try to push it to be an official Rust musl/libc++/clang toolchain image.

@LucioFranco
Copy link
Copy Markdown
Contributor

@a-rodin I created #946 to track this, we can move discussion there.

@LucioFranco
Copy link
Copy Markdown
Contributor

We need to build the image and push it before we merge this PR.

Signed-off-by: Lucio Franco <luciofranco14@gmail.com>
@LucioFranco
Copy link
Copy Markdown
Contributor

@a-rodin Image is being built right now. I moved the patch into the root dir of the builder images since that is where docker loads the context when running build-ci-images.sh.

@LucioFranco
Copy link
Copy Markdown
Contributor

New image has been pushed.

@LucioFranco LucioFranco changed the title chore: Use stable Rust 1.38.0 and update Linux headers for x86_64-unknown-linux-musl builder chore: Use stable Rust 1.38.0 and update Linux headers for x86_6… Oct 1, 2019
@LucioFranco LucioFranco merged commit 07339e1 into vectordotdev:master Oct 1, 2019
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

Successfully merging this pull request may close these issues.

2 participants