Skip to content

[0.4.x] libvisual + libvisual-plugins: Cover more compilers for macOS - #262

Merged
hartwork merged 1 commit into
0.4.xfrom
0.4.x-macos
Mar 2, 2023
Merged

hartwork merged 1 commit into
0.4.xfrom
0.4.x-macos

Conversation

@hartwork hartwork added this to the 0.4.2 milestone Feb 7, 2023
@hartwork
hartwork requested a review from kaixiong February 7, 2023 00:01
@hartwork hartwork changed the title [0.4.x] libvisual + libvisual-plugins: Fix support for macOS [0.4.x] libvisual + libvisual-plugins: Fix support for macOS (fixes #246) Feb 7, 2023
@hartwork
hartwork marked this pull request as ready for review February 13, 2023 16:23
@hartwork

Copy link
Copy Markdown
Member Author

@kaixiong rebased onto latest 0.4.x with #214 merged now, ready for review. I can extract smaller pull requests from this and rebase again after their merge, if you see value in that.

@hartwork
hartwork marked this pull request as draft February 13, 2023 18:35
@hartwork

Copy link
Copy Markdown
Member Author

@kaixiong rebased onto latest 0.4.x now, down to 2 commits.

@hartwork
hartwork marked this pull request as ready for review February 14, 2023 16:02
@hartwork

Copy link
Copy Markdown
Member Author

@kaixiong should I extract the last G-Force piece here to a dedicated pull request? Anything else in need of adjustment?

@kaixiong

Copy link
Copy Markdown
Member

@kaixiong should I extract the last G-Force piece here to a dedicated pull request? Anything else in need of adjustment?

@hartwork That would be good, thanks.

@hartwork

Copy link
Copy Markdown
Member Author

@kaixiong should I extract the last G-Force piece here to a dedicated pull request? Anything else in need of adjustment?

@hartwork That would be good, thanks.

@kaixiong here you go: #290

@hartwork hartwork changed the title [0.4.x] libvisual + libvisual-plugins: Fix support for macOS (fixes #246) [0.4.x] libvisual + libvisual-plugins: Cover more compilers for macOS Feb 22, 2023
@hartwork

Copy link
Copy Markdown
Member Author

@kaixiong now rebased off #290 onto latest 0.4.x.

@kaixiong

Copy link
Copy Markdown
Member

@hartwork Digressing a bit but do we need to cover this many compiler versions? I welcome the idea but I feel that the time spent on CI is not scaling well.

@hartwork

Copy link
Copy Markdown
Member Author

@kaixiong do you mean scaling well as in human-perceived CI runtime or as in total machine resource usage?

On other projects (with Python) where I want to be covered against surprises with Python versions in 3.7, 3.8, 3.9, 3.10, 3.11 I often cover the oldest supported and the most recent one but not the intermediates, i.e. 3.7 and 3.11 in that case, and that works okay. However our situation in Libvisual is that we have multiple macOS versions (more dimensions) + developers not using macOS (extra blindness and value) + different past errors than found by Linux CI (also extra value). We can trim this down more but I wonder at what loss and what value.

What do you think?

@hartwork

hartwork commented Feb 22, 2023 •

Copy link
Copy Markdown
Member Author

@kaixiong PS: Three more tiny notes:

  • I have not added/used macos-10.15 anywhere because it's already deprecated (in the eyes of https://github.com/actions/runner-images)
  • (This is already far from a complete coverage of all combinations.)
  • The matrix entries I'd be least reluctant to give up are GCC 10 and GCC 11 (because we'll still have GCC 9 @ macos-11 and GCC 12 @ macos-12).

@hartwork

Copy link
Copy Markdown
Member Author

@kaixiong what do you think?

@kaixiong

kaixiong commented Mar 1, 2023 •

Copy link
Copy Markdown
Member

@hartwork, I think the additional build actions for MacOS are okay.

I was mostly concerned if we're going to do the same for other platforms (OS+compiler combos). That would have a combinatorial/multiplicative effect. If we start adding other kinds of validation (UBSAN, ASAN) in the future, it's going to take even longer.

@hartwork

hartwork commented Mar 2, 2023

Copy link
Copy Markdown
Member Author

@kaixiong cool! 👍

@hartwork
hartwork merged commit a02f20c into 0.4.x Mar 2, 2023
@hartwork
hartwork deleted the 0.4.x-macos branch March 3, 2023 03:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

macOS support / need your help with trying 0.4.x out on macOS (branch "0.4.x-macos")

2 participants