Skip to content

Conversation

@mhegazy
Copy link
Collaborator

@mhegazy mhegazy commented Sep 12, 2025

Now that we are using the config action everywhere there is a lot of flags that were manually passed that are not needed any more.

  • Remove unused inputs from the action
  • Remove unused flags from the matrix definition
  • Remove Platform in favor of a single property os
  • Remove compiler_target in favor of triple

Downstream job: https://github.com/thebrowsercompany/swift-build/actions/runs/17679457004

@mhegazy
Copy link
Collaborator Author

mhegazy commented Sep 12, 2025

my next step is remove custom matrixes from jobs in swift-toolchain.yml so we are always using host/target matrix when applicable, and then simplify Android handeling in the script.

Copy link
Owner

@compnerd compnerd left a comment

Choose a reason for hiding this comment

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

Nice! I like red! I wonder if the matrix is now simple enough to inline instead of having the two layer thing.

@mhegazy mhegazy merged commit 016eba7 into compnerd:main Sep 12, 2025
@mhegazy mhegazy deleted the upstream-cleanup branch September 12, 2025 19:15
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.

2 participants