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

Build and install additional packages for rutabaga_gfx #110

Merged
merged 1 commit into from
Aug 9, 2024

Conversation

dorindabassey
Copy link

Summary of the PR

The vhost-device-gpu crate linked here
rust-vmm/vhost-device#668
contains rutabaga_gfx and needs some packages and
virglrenderer greater that 1.0.0 which is not available
in the used ubuntu image, so install the neccessary
packages that are available in ubuntu's repositories and
for unavailable packages build and install it manually.

Requirements

Before submitting your PR, please make sure you addressed the following
requirements:

  • All commits in this PR have Signed-Off-By trailers (with
    git commit -s), and the commit message has max 60 characters for the
    summary and max 75 characters for each description line.
  • All added/changed functionality has a corresponding unit/integration
    test.
  • All added/changed public-facing functionality has entries in the "Upcoming
    Release" section of CHANGELOG.md (if no such section exists, please create one).
  • Any newly added unsafe code is properly documented.

stsquad
stsquad previously approved these changes Aug 8, 2024
@stefano-garzarella
Copy link
Member

IIUC this contains also the patch of #109 , please can you rebase on main? I'd like to merge that patch with the right PR.

The vhost-device-gpu crate linked here
rust-vmm/vhost-device#668
contains rutabaga_gfx and needs some packages and
virglrenderer greater that 1.0.0 which is not available
in the used ubuntu image, so install the neccessary
packages that are available in ubuntu's repositories and
for unavailable packages build and install it manually.

Signed-off-by: Dorinda Bassey <dbassey@redhat.com>
@dorindabassey
Copy link
Author

IIUC this contains also the patch of #109 , please can you rebase on main? I'd like to merge that patch with the right PR.

ok done. Thanks!

@stsquad stsquad merged commit 380309d into rust-vmm:main Aug 9, 2024
2 checks passed
dorindabassey pushed a commit to dorindabassey/rust-vmm-ci that referenced this pull request Aug 9, 2024
This version includes vhost-device-gpu dependencies
which is required for vhost-device-gpu crate
more info about the crate can be found here:
rust-vmm/vhost-device#668
link to the PR that adds these dependencies
where the tag is generated can be found here:
rust-vmm/rust-vmm-container#110

Signed-off-by: Dorinda Bassey <dbassey@redhat.com>
ShadowCurse pushed a commit to rust-vmm/rust-vmm-ci that referenced this pull request Aug 9, 2024
This version includes vhost-device-gpu dependencies
which is required for vhost-device-gpu crate
more info about the crate can be found here:
rust-vmm/vhost-device#668
link to the PR that adds these dependencies
where the tag is generated can be found here:
rust-vmm/rust-vmm-container#110

Signed-off-by: Dorinda Bassey <dbassey@redhat.com>
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.

3 participants