Skip to content

Conversation

@steve-downey
Copy link
Member

Improve the CI workflow, building with clang 17, 18, and 19, against libc++, and with GCC 13 and 14 using Ubuntu 24.04 as the base OS.

steve-downey and others added 4 commits June 3, 2024 16:14
Use really the latest Ubuntu and retrieve Clang from LLVM apt.
Build with libc++, add 17 and 19 to CI
Workaround for gtest issues with libc++ from clang-17.

Run as root
@@ -3,7 +3,7 @@ include_guard(GLOBAL)
set(CMAKE_CXX_STANDARD 23)
Copy link
Member

Choose a reason for hiding this comment

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

remind me -- is there something here that requires 23, or could it be 20?

Copy link
Member Author

Choose a reason for hiding this comment

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

I'll check. I believe at one point I was looking at deducing this but then didn't.

@@ -1,76 +1,121 @@
name: ci
name: Test
Copy link
Member

Choose a reason for hiding this comment

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

I'm no expert on these github workflows now - @bretbrownjr any comments? Specifically I'm wondering about non-Linux platforms. Also, @steve-downey did this come from our template or do we need to update?

Copy link
Member

Choose a reason for hiding this comment

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

@steve-downey , is there a way to run these flows on local environment? e.g. git API to span a Docker for every job, but on my local setup. I am just curious if I am doing a change, if I can run all jobs on my MAC OS / Ubuntu.

Copy link
Member Author

Choose a reason for hiding this comment

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

I have not found a way to simulate any of this locally. I've ended up enabling actions on PRs and repeated force-pushes to a branch to trigger a run.

Copy link
Member Author

Choose a reason for hiding this comment

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

I took a bunch of it from bemanproject/exemplar#6. I also have some possible updates to deal with Node version warnings over here: https://github.com/steve-downey/lambda-fun/pull/2/files

@steve-downey
Copy link
Member Author

Merging so as to base further work on working CI

@steve-downey steve-downey merged commit 32b1d41 into bemanproject:main Jun 11, 2024
steve-downey added a commit that referenced this pull request Nov 4, 2024
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