Refine GitHub Actions CI workflow to support a matrix of compiler versions#334
Conversation
|
OMG, after a crazy number of commits, I finally got the tests to pass inside the ubuntu:14.04 container! First, I had to remove localhost from the ::1 address in test->push_port(libtest::get_free_port());
test->push_port(libtest::get_free_port());to use a fixed port number: test->push_port( 19298 );
test->push_port( 19300 );And that worked! All tests passed. (I just picked the port numbers 19298 and 19300 at random.) If anyone has any ideas on how to get the original code to work inside the container without this hack, let me know. Otherwise, I'm happy with it as-is. Now to get clang builds working! |
465e7d5 to
a4ca74d
Compare
|
Between 6 months ago and now, all the Ubuntu 18.04 configurations stopped working. :sigh: I posted a message about it in the GitHub Actions community. I'm hoping someone there can help me fix it. https://github.com/orgs/community/discussions/38963 P.S. Don't worry. I'll squash all these commits at some point. 😄 |
|
Well, I added Then I tried to add an Ubuntu 22.04 build, and the bootstrap failed. 😞 https://github.com/gearman/gearmand/actions/runs/3454365941/jobs/5765614111 I guess we don't need to bootstrap on 22.04. As long as the release tarball compiles on 22.04, we're good to go in the short-term. Should I save adding the Ubuntu 22.04 build for later? |
d938195 to
9abb54d
Compare
|
Changed the gcc-11 build to run on Ubuntu 20.04, and that worked around the 22.04 bootstrap problem. But then the @SpamapS: Two options: (1) I remove the gcc-11 build from this PR and we merge it. Then you rebase PR #350 on master and add the gcc-11 build to the CI workflow in your PR. (2) You merge PR #350 and then I rebase this PR on top. Less work, but it assumes PR #350 won't break any builds. Which option do you prefer? |
|
Lets just drop the gcc-11 and merge this. It's really fantastic and will help us move faster with future maintenance. |
9abb54d to
a0291cc
Compare
|
Rebased on master to pick up the gcc-11 fixes in PR #350. Squashed commits. All builds pass. The |
|
All |
|
Awesome and agreed on pushing 22.04 to later. |
This PR refines the GitHub Actions CI workflow to support a matrix of compiler versions. This is needed so that we can "retire" Travis CI.
I'd like to rename the
.github/workflow/c-cpp.ymlfile toci.yml, but I haven't done that yet. Let me know what you think about that.