Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upAdded harfbuzz-devel to Fedora's dependencies in README #15498
Conversation
highfive
commented
Feb 10, 2017
|
Thanks for the pull request, and welcome! The Servo team is excited to review your changes, and you should hear from @asajeffrey (or someone else) soon. |
|
This change looks very sensible! Is |
|
@asajeffrey that I don't know. I just had a fresh setup of Fedora 25 (Gnome), and doing |
|
If |
|
We always build harfbuzz from scratch on Windows, and we seem to use the system harfbuzz if it's present on other platforms, and build it from scratch if it's not. |
|
@kvark can you send the log for the build failure you had without |
|
|
Hmm, you appear not to have the harfbuzz lib in your load path. What does |
|
@asajeffrey ls shows several
TL;DR: "gtk-doc >= 1.15 is not found" |
|
That looks like aclocal-1.14 is missing; the gtk-doc thing is just a warning about |
|
|
Right, but it wants |
|
It looks like this issue is caused by the fact that we are vendoring the generated Installing |
|
@asajeffrey so I guess this PR should be closed, and you guys can open a separate issue to fix the vendored harfbuzz version. Thanks for looking into it! |
|
cc @mbrubeck |
|
@asajeffrey that issue is closed, which seems wrong to me |
|
Yes, we may need to reopen it. |
kvark commentedFeb 10, 2017
•
edited by larsbergstrom
./mach build -ddoes not report any errors./mach test-tidydoes not report any errorsThis change is