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

[libgpiod] Show build issues with container #68

Closed
wants to merge 2 commits into from

Conversation

vireshk
Copy link
Collaborator

@vireshk vireshk commented Dec 3, 2021

Hi @andreeaflorescu

The containers used for builds currently don't have necessary tools to
compile buildgen stuff, which compiles few C files and create bindings.

This pull request is about highlighting those issues here.

We need these tools in the containers: libclang-dev clang musl-tools.

We also need to configure the container to pass the include path for the Linux libraries,
else we get more errors.

THIS PULL REQUEST SHOULDN'T BE MERGED.

@vireshk
Copy link
Collaborator Author

vireshk commented Dec 3, 2021

Earlier it used to fail while trying to build libgpiod, now it is not even able to fetch it. Not sure why. Are we restricting the dependencies to be from known paths only?

@andreeaflorescu
Copy link
Member

Earlier it used to fail while trying to build libgpiod, now it is not even able to fetch it. Not sure why. Are we restricting the dependencies to be from known paths only?

We are not restricting dependencies in any way. Not sure why this is happening.

@vireshk
Copy link
Collaborator Author

vireshk commented Dec 8, 2021

We are not restricting dependencies in any way. Not sure why this is happening.

It was failing ealier because of edition 2021 in libgpiod, after making that 2018, it is working fine. Only the build issues are left.

@andreeaflorescu
Copy link
Member

FYI @vireshk the container is not automatically published. Once it gets published, we need to also update rust-vmm-ci to use the new container version.

@andreeaflorescu
Copy link
Member

@vireshk I published rust-vmm/dev:v14, can you update the container version in rust-vmm-ci?

@vireshk
Copy link
Collaborator Author

vireshk commented Dec 22, 2021

@vireshk I published rust-vmm/dev:v14, can you update the container version in rust-vmm-ci?

Sure.

rust-vmm/rust-vmm-ci#98

The containers used for builds currently don't have necessary tools to
compile buildgen stuff, which compiles few C files and create bindings.

This pull request is about highlighting those issues here.

We need these tools in the containers: libclang-dev clang musl-tools.

Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
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