Skip to content

use previous ubuntu version for example validation#15882

Merged
alice-i-cecile merged 2 commits intobevyengine:mainfrom
mockersf:use-ubuntu-22-for-example-validation
Oct 13, 2024
Merged

use previous ubuntu version for example validation#15882
alice-i-cecile merged 2 commits intobevyengine:mainfrom
mockersf:use-ubuntu-22-for-example-validation

Conversation

@mockersf
Copy link
Copy Markdown
Member

@mockersf mockersf commented Oct 13, 2024

Objective

  • Example validation job fails in CI
  • This happened after GitHub updated the latest version of ubuntu from the 22.04 to the 24.04
  • The package libegl1-mesa is not available on ubuntu 24.04

Solution

  • Keep using ubuntu 22.04
  • This is a temp fix and we should fix the update

Testing

  • if it can get merged then it works 🤷

@mockersf mockersf added A-Build-System Related to build systems or continuous integration P-Critical This must be fixed immediately or contributors or users will be severely impacted labels Oct 13, 2024
@alice-i-cecile alice-i-cecile added the O-Linux Specific to the Linux desktop operating system label Oct 13, 2024
alice-i-cecile
alice-i-cecile approved these changes Oct 13, 2024
View reviewed changes
Copy link
Copy Markdown
Member

@alice-i-cecile alice-i-cecile left a comment

Choose a reason for hiding this comment

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

I love CI.

BD103
BD103 approved these changes Oct 13, 2024
View reviewed changes
Copy link
Copy Markdown
Member

@BD103 BD103 left a comment

Choose a reason for hiding this comment

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

CI spelled backwards is pronounced "ick".

@BenjaminBrienen BenjaminBrienen added D-Trivial Nice and easy! A great choice to get started with Bevy S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it labels Oct 13, 2024
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Oct 13, 2024
Merged via the queue into bevyengine:main with commit 3b14ebe Oct 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alice-i-cecile alice-i-cecile alice-i-cecile approved these changes

+1 more reviewer

@BD103 BD103 BD103 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

A-Build-System Related to build systems or continuous integration D-Trivial Nice and easy! A great choice to get started with Bevy O-Linux Specific to the Linux desktop operating system P-Critical This must be fixed immediately or contributors or users will be severely impacted S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@mockersf @alice-i-cecile @BD103 @BenjaminBrienen