-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Add autoconf to build dependencies #9128
Conversation
`autoheader` from `autoconf` was necessary for the build on my system.
Is this due to harfbuzz failing to compile? |
As decided in the meeting today, we're going to accept this while continuing to think about better solutions than requiring autoconf. |
@bors-servo: r+ |
📌 Commit a244f78 has been approved by |
Add autoconf to build dependencies `autoheader` from `autoconf` was necessary for the build on my system. <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9128) <!-- Reviewable:end -->
💔 Test failed - linux-rel |
⚡ Previous build results for android, gonk, linux-dev, mac-dev-ref-unit, mac-rel-css, mac-rel-wpt are reusable. Rebuilding only linux-rel... |
☀️ Test successful - android, gonk, linux-dev, linux-rel, mac-dev-ref-unit, mac-rel-css, mac-rel-wpt |
This includes the fix for servo/rust-harfbuzz#54, and reverts servo#9128 which was a temporary workaround for that issue.
This includes the fix for servo/rust-harfbuzz#54, and reverts servo#9128 which was a temporary workaround for that issue.
autoheader
fromautoconf
was necessary for the build on my system.