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

TEST: Debug the wheel build #108

Merged
merged 7 commits into from
Jan 22, 2023
Merged

TEST: Debug the wheel build #108

merged 7 commits into from
Jan 22, 2023

Conversation

jwiggins
Copy link
Member

@jwiggins jwiggins commented Jan 2, 2023

Final step to cover #106

@jwiggins jwiggins force-pushed the test/release-process branch 2 times, most recently from f67f56b to c26921c Compare January 2, 2023 14:41
@jwiggins jwiggins force-pushed the test/release-process branch from c26921c to 3558605 Compare January 2, 2023 14:43
@jwiggins
Copy link
Member Author

jwiggins commented Jan 2, 2023

LOL. I've never built wheels with Harfbuzz

@aurivus-ph
Copy link

I've played around with this myself, and I've got the pipeline to succeed. See here: aurivus-ph#1

I also temporarily disabled building the sdist in it.

The issues were:

  • harfbuzz doesn't have a configure script, so autogen.sh has to be run first
  • aclocal is installed twice, and the installation that is used misses the pkgconfig macros
  • the actions/setup-python@v4 action mangles the pkgconfig environment variable, so it has to be run after building the wheels

@jwiggins jwiggins force-pushed the test/release-process branch from 3d61acb to 7f845b5 Compare January 21, 2023 12:27
@jwiggins
Copy link
Member Author

Thanks for the assistance @aurivus-ph!

@jwiggins jwiggins merged commit 35bb09c into main Jan 22, 2023
@jwiggins jwiggins deleted the test/release-process branch January 22, 2023 14:31
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