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

Create a new harfbuzz-sys crate #53

Merged
merged 5 commits into from Sep 29, 2015
Merged

Conversation

@mbrubeck
Copy link
Contributor

mbrubeck commented Sep 25, 2015

  • Upgrade to harfbuzz 1.0.3
  • Follow -sys crate conventions
  • New FFI bindings built completely with rust-bindgen
  • Can build HarfBuzz from source or link to it via pkg-config
  • Make ICU an optional feature with the new icu-sys crate

r? @larsberg

Review on Reviewable

mbrubeck added 4 commits Sep 16, 2015
* Follow -sys crate conventions
* New FFI bindings built completely with bindgen
* Can build HarfBuzz from source or link to it via pkg-config
* Make ICU an optional feature with the icu-sys crate
Because ICU source is too big to be a Cargo package, and we can replace it
with the pure rust unicode-script crate instead.
@mbrubeck mbrubeck force-pushed the mbrubeck:harfbuzz-sys branch from 222c721 to 3570938 Sep 28, 2015
@mbrubeck mbrubeck force-pushed the mbrubeck:harfbuzz-sys branch 3 times, most recently from d9a6a10 to be0955b Sep 28, 2015
@larsbergstrom
Copy link
Contributor

larsbergstrom commented Sep 29, 2015

@bors-servo
Copy link
Contributor

bors-servo commented Sep 29, 2015

📌 Commit be0955b has been approved by larsbergstrom

@bors-servo
Copy link
Contributor

bors-servo commented Sep 29, 2015

Testing commit be0955b with merge 5155dd2...

bors-servo pushed a commit that referenced this pull request Sep 29, 2015
Create a new harfbuzz-sys crate

* Upgrade to harfbuzz 1.0.3
* Follow `-sys` crate conventions
* New FFI bindings built completely with rust-bindgen
* Can build HarfBuzz from source or link to it via pkg-config
* Make ICU an optional feature with the new `icu-sys` crate

r? @larsberg

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/rust-harfbuzz/53)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Sep 29, 2015

☀️ Test successful - travis

@bors-servo bors-servo merged commit be0955b into servo:master Sep 29, 2015
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
bors-servo pushed a commit to servo/servo that referenced this pull request Sep 29, 2015
Use Harfbuzz 1.0 and unicode-script for text shaping

Depends on servo/rust-harfbuzz#53 and introduces a dependency on the new servo/unicode-script crate.  r? @pcwalton

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/7786)
<!-- Reviewable:end -->
bors-servo pushed a commit to servo/servo that referenced this pull request Sep 29, 2015
Use Harfbuzz 1.0 and unicode-script for text shaping

Depends on servo/rust-harfbuzz#53 and introduces a dependency on the new servo/unicode-script crate.  r? @pcwalton

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/7786)
<!-- Reviewable:end -->
jrmuizel pushed a commit to jrmuizel/gecko-cinnabar that referenced this pull request Jun 12, 2017
…ping (from mbrubeck:harfbuzz-sys); r=pcwalton

Depends on servo/rust-harfbuzz#53 and introduces a dependency on the new servo/unicode-script crate.  r? @pcwalton

Source-Repo: https://github.com/servo/servo
Source-Revision: a144d086d911375f8d5674ac90ff55a9dcaf0cda
gecko-dev-updater pushed a commit to marco-c/gecko-dev-comments-removed that referenced this pull request Sep 30, 2019
…ping (from mbrubeck:harfbuzz-sys); r=pcwalton

Depends on servo/rust-harfbuzz#53 and introduces a dependency on the new servo/unicode-script crate.  r? pcwalton

Source-Repo: https://github.com/servo/servo
Source-Revision: a144d086d911375f8d5674ac90ff55a9dcaf0cda

UltraBlame original commit: f66ad38ebe0b849d5d647707f20b8c2990d6b696
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified that referenced this pull request Oct 1, 2019
…ping (from mbrubeck:harfbuzz-sys); r=pcwalton

Depends on servo/rust-harfbuzz#53 and introduces a dependency on the new servo/unicode-script crate.  r? pcwalton

Source-Repo: https://github.com/servo/servo
Source-Revision: a144d086d911375f8d5674ac90ff55a9dcaf0cda

UltraBlame original commit: f66ad38ebe0b849d5d647707f20b8c2990d6b696
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified-and-comments-removed that referenced this pull request Oct 1, 2019
…ping (from mbrubeck:harfbuzz-sys); r=pcwalton

Depends on servo/rust-harfbuzz#53 and introduces a dependency on the new servo/unicode-script crate.  r? pcwalton

Source-Repo: https://github.com/servo/servo
Source-Revision: a144d086d911375f8d5674ac90ff55a9dcaf0cda

UltraBlame original commit: f66ad38ebe0b849d5d647707f20b8c2990d6b696
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.