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

Short-list just the Harfbuzz sources we need #63

Merged
merged 1 commit into from
Sep 11, 2020

Conversation

alerque
Copy link
Member

@alerque alerque commented Sep 11, 2020

The GNU Make, Cmake, Python scripts, and other random stuff in
Harfbuzz's source directory are not relevant to the cython build which
only uses the amalgam file (harfbuzz.cc) and compiles it using a c++
compiler directly. This includes the other sources and header files, but
doesn't need any of the build machinery.

See discussion on #62.

No need to release for this as 0.13.2 isn't broken. This is just a tidy set of sources for next time.

The GNU Make, Cmake, Python scripts, and other random stuff in
Harfbuzz's source directory are not relevant to the cython build which
only uses the amalgam file (harfbuzz.cc) and compiles it using a c++
compiler directly. This includes the other sources and header files, but
doesn't need any of the build machinery.
@justvanrossum justvanrossum merged commit 9820b8e into harfbuzz:master Sep 11, 2020
@alerque alerque deleted the tidier-dist branch September 11, 2020 09:52
Copy link
Member

@anthrotype anthrotype left a comment

Choose a reason for hiding this comment

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

thanks for cleaning this up!

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.

3 participants