Skip to content
This repository was archived by the owner on Apr 23, 2026. It is now read-only.

Fix the GitHub CI build#390

Merged
bors-servo merged 3 commits intomasterfrom
fix-ci-kernel-repo
Jan 19, 2023
Merged

Fix the GitHub CI build#390
bors-servo merged 3 commits intomasterfrom
fix-ci-kernel-repo

Conversation

@mrobinson
Copy link
Copy Markdown
Member

@mrobinson mrobinson commented Jan 19, 2023

This PR fixes various issues that were preventing the CI from successfully running.

This fixes an issue with missing repository keys.
@mrobinson mrobinson force-pushed the fix-ci-kernel-repo branch 4 times, most recently from 640e811 to 66c473f Compare January 19, 2023 12:18
This also removes the installation of the pre-packaged GStreamer build
in exchange for the one from upstream Ubuntu.
@mrobinson mrobinson changed the title ci: Add the kernel repository using apt-add-repository Fix the GitHub CI build Jan 19, 2023
@mrobinson mrobinson force-pushed the fix-ci-kernel-repo branch 3 times, most recently from bdcbd77 to c875892 Compare January 19, 2023 13:41
This gets things compiling on modern systems and makes sure that the
dependencies are compatible with one another.
@mrobinson
Copy link
Copy Markdown
Member Author

Okay. After a lot of wrestling with cargo and GitHub runners, I think this PR is ready to be merged. It gets the CI working again.

run: cargo build
- name: Examples
run: |
ls examples/*.rs | xargs -I{} basename {} .rs | grep -v params_connect | RUST_BACKTRACE=1 GST_DEBUG=3 xargs -I{} cargo ex {} --all-features
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why aren't we running the examples anymore?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

feel free to r=me to fix the immediate problem though

Copy link
Copy Markdown
Member Author

@mrobinson mrobinson Jan 19, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

At least for me, one of these examples did not finish executing on my local machine. Perhaps I didn't wait long enough for that to happen though. I'll play around a bit locally.

Copy link
Copy Markdown
Member Author

@mrobinson mrobinson Jan 19, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems this works on CI while running the examples, so I have removed that commit.

@mrobinson
Copy link
Copy Markdown
Member Author

@bors-servo r=Manishearth

@bors-servo
Copy link
Copy Markdown
Contributor

📌 Commit e9b29e7 has been approved by Manishearth

@bors-servo
Copy link
Copy Markdown
Contributor

⌛ Testing commit e9b29e7 with merge 7c6fab9...

@mrobinson
Copy link
Copy Markdown
Member Author

Thanks for the review!

@bors-servo
Copy link
Copy Markdown
Contributor

☀️ Test successful - checks-github, status-appveyor
Approved by: Manishearth
Pushing 7c6fab9 to master...

@bors-servo bors-servo merged commit 7c6fab9 into master Jan 19, 2023
@mrobinson mrobinson deleted the fix-ci-kernel-repo branch January 19, 2023 17:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants