Skip to content

CI: Update GitHub Actions workflow to use GCC 11 - #628

Closed
EwoutH wants to merge 2 commits into
AOMediaCodec:mainfrom
EwoutH:patch-34
Closed

EwoutH wants to merge 2 commits into
AOMediaCodec:mainfrom
EwoutH:patch-34

Conversation

@EwoutH

@EwoutH EwoutH commented May 18, 2021

Copy link
Copy Markdown
Contributor

Updates the GitHub Actions workflow to use GCC 11 (instead of GCC 10) on the Ubuntu and macOS runs.

GCC 11 was added to the Ubuntu 20.04 environment earlier this month.

Updates the GitHub Actions workflow to use GCC 11 (instead of GCC 10) on the Ubuntu and macOS runs.
@EwoutH

EwoutH commented Jun 3, 2021

Copy link
Copy Markdown
Contributor Author

Rebased

@EwoutH

EwoutH commented Oct 27, 2022

Copy link
Copy Markdown
Contributor Author

@y-guyon @wantehchang I'm doing some cleanup of old, open PRs. If you would like to get these changes in, please leave a reply or review.

In two weeks I will close this PR if no reaction received.

Comment thread .github/workflows/ci.yml
- name: Set GCC & G++ 11 compiler (on Linux)
if: runner.os == 'Linux'
run: echo "CC=gcc-10" >> $GITHUB_ENV && echo "CXX=g++-10" >> $GITHUB_ENV
run: echo "CC=gcc-11" >> $GITHUB_ENV && echo "CXX=g++-11" >> $GITHUB_ENV

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

unrelated, but could this simply be an env: entry? Or is there something special about the use of $GITHUB_ENV here?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Looking at the git blame, I modified it in PR #595 about one and a half years ago. I think this has to do with this issue:

Or it has to do with retention between steps.

But I don't remember exactly anymore. We could try if it works with a regular env:, if you want.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

For this change we can leave it as is. I was just curious when comparing other workflow files.

@jzern
jzern requested a review from y-guyon October 27, 2022 23:47

@y-guyon y-guyon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should we also migrate other GitHub workflows such as ci-unix-shared.yml?

@vrabaud

vrabaud commented Dec 14, 2023

Copy link
Copy Markdown
Contributor

GCC 13 is out. Should we update all the CI? Or maybe use that action? https://github.com/marketplace/actions/install-gcc

@vrabaud

vrabaud commented Dec 18, 2023

Copy link
Copy Markdown
Contributor

Closing in favor of #1886

@vrabaud vrabaud closed this Dec 18, 2023
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.

4 participants