Skip to content

build: Make it clearer which packages are needed, also check for libtool... #88

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 3 commits into from
Closed

build: Make it clearer which packages are needed, also check for libtool... #88

wants to merge 3 commits into from

Conversation

srl295
Copy link
Contributor

@srl295 srl295 commented Feb 26, 2015

  • Clarify which packages are needed
  • check for libtool (the error message was a little obscure)

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 744d832 on srl295:improve_build into * on behdad:master*.

@srl295
Copy link
Contributor Author

srl295 commented Feb 26, 2015

also cleanup .gitignore

@behdad
Copy link
Member

behdad commented Feb 26, 2015

Thanks. Pushed out similar fixes.

@behdad behdad closed this in 1e03d7a Feb 26, 2015
gpgreen pushed a commit to gpgreen/harfbuzz that referenced this pull request Jan 10, 2024
@waywardmonkeys
Use ::std::os::raw instead of libc.
2704705 
This is a breaking change.

Fixes issue harfbuzz#88.
gpgreen pushed a commit to gpgreen/harfbuzz that referenced this pull request Jan 10, 2024
Auto merge of harfbuzz#91 - waywardmonkeys:remove-libc, r=mbrubeck
ff81f10 
[breaking change] Use ::std::os::raw instead of libc.

This is a breaking change.

Fixes issue harfbuzz#88.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-harfbuzz/91)
<!-- Reviewable:end -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@srl295 @coveralls @behdad