Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Correct the name of the GitHub Actions jobs and make sure to test all static/shared combinations #216

Merged
merged 2 commits into from
Aug 18, 2023

Conversation

waywardmonkeys
Copy link
Collaborator

This peels 2 commits off of #211 since I think these are needed no matter what else happens.

The shared job should not disable pkg-config usage and should
be the one that does a Harfbuzz build.

The static job should disable pkg-config and should not do
its own Harfbuzz build outside of the `harfbuzz-sys` build
scripts.

This is a precursor to making sure things are actually set up
correctly and clearly doing the appropriate thing.
This is an experiment to see if we're able to pick it up from
pkg-config.
@waywardmonkeys waywardmonkeys changed the title Ci vs static CI vs static Aug 18, 2023
Copy link
Member

@mrobinson mrobinson left a comment

Choose a reason for hiding this comment

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

Makes sense.

@mrobinson mrobinson added this pull request to the merge queue Aug 18, 2023
@mrobinson mrobinson removed this pull request from the merge queue due to a manual request Aug 18, 2023
@mrobinson mrobinson changed the title CI vs static Correct the name of the GitHub Actions jobs and make sure to test all static/shared combinations Aug 18, 2023
@mrobinson mrobinson added this pull request to the merge queue Aug 18, 2023
Merged via the queue into servo:master with commit 9b57880 Aug 18, 2023
6 checks passed
@waywardmonkeys waywardmonkeys deleted the ci-vs-static branch August 18, 2023 16:44
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