mbox series

[0/3] un-breaking osx-gcc ci job

Message ID 20240509162219.GA1707955@coredump.intra.peff.net (mailing list archive)
Headers show
Series un-breaking osx-gcc ci job | expand

Message

Jeff King May 9, 2024, 4:22 p.m. UTC
The osx-gcc job seems to break reliably for me, which started in the
last week or so (since the last time I actually triggered CI). The third
patch fixes it, but I noticed while investigating that the job is not
even running gcc at all! That's fixed in the second patch. And the first
one is just a cleanup I found along the way.

I'm not sure that this job carries a huge amount of value over the
osx-clang one, so we might consider just ditching it. But in the
meantime, this should get things passing again.

  [1/3]: ci: drop mention of BREW_INSTALL_PACKAGES variable
  [2/3]: ci: avoid bare "gcc" for osx-gcc job
  [3/3]: ci: stop installing "gcc-13" for osx-gcc

 .github/workflows/main.yml | 3 +--
 ci/install-dependencies.sh | 2 --
 2 files changed, 1 insertion(+), 4 deletions(-)

-Peff

Comments

Junio C Hamano May 9, 2024, 4:52 p.m. UTC | #1
Jeff King <peff@peff.net> writes:

> The osx-gcc job seems to break reliably for me, which started in the
> last week or so (since the last time I actually triggered CI). The third
> patch fixes it, but I noticed while investigating that the job is not
> even running gcc at all! That's fixed in the second patch. And the first
> one is just a cleanup I found along the way.
>
> I'm not sure that this job carries a huge amount of value over the
> osx-clang one, so we might consider just ditching it. But in the
> meantime, this should get things passing again.

Thanks.

>   [1/3]: ci: drop mention of BREW_INSTALL_PACKAGES variable
>   [2/3]: ci: avoid bare "gcc" for osx-gcc job
>   [3/3]: ci: stop installing "gcc-13" for osx-gcc
>
>  .github/workflows/main.yml | 3 +--
>  ci/install-dependencies.sh | 2 --
>  2 files changed, 1 insertion(+), 4 deletions(-)
>
> -Peff