-
-
Notifications
You must be signed in to change notification settings - Fork 197
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
Build fails when adding YoJson as a dependency using esy Linux Manjaro #666
Comments
@srsholmes Can you give the harfbuzz version. As mentioned here we need native harfbuzz > 1.7.7 Can you add the content of |
Thanks very much for your quick reply, and thanks for your work on revery, i'm really excited to use the project. |
Thanks for the repro I get the same build err it is strange because I can build fine revery 🤔. Can you build revery from source ? Also note revery is in 0.29.0 now. |
Ok this commit build 640885e (HEAD^) What changed between 2.8.0 and 2.8.2 https://github.com/revery-ui/reason-fontkit/tree/9b9a9bdffcaf0e953fd4362953404b342f11ea1a + https://github.com/revery-ui/reason-fontkit/tree/9b9a9bdffcaf0e953fd4362953404b342f11ea1a The first one shouldn't break (because it keep the flags if folder exist) |
One fix I can suggest is to run |
Seem to be fixed with this esy-packages/esy-harfbuzz#12 |
I was able to compile the laster HEAD commit with |
I have tested on arch linux and got the same error. I haven't the issue so I assume this will fix on manjaro can you test(If it success I will PR). |
I am not able to currently build 0.29 (master) from source, with the same error described above. I was able to build revery 0.29 from source with the I was also able to build my test repo and my project repo using the Thanks for your help and quick responses. I'll wait for the PR and then try and update revery in my project and let you know how it goes. |
@bryphe has done the PR so just wait the new release and enjoy revery. When the new version is released can you confirm it is ok so we can close this issue :) |
Will do, thanks |
@srsholmes It is merged and released just do |
The solution above fixes the issue. But I have another one:
Ubuntu 20.04 Gnome/Cinnamon |
@koekaverna if you happened to use the revery-quick-start like I did, turns out I had to update I figured this out after a full night of updating random packages to try to fix this issue with adding |
Link to github repo: https://github.com/srsholmes/revery-yo-json-build-fail
Steps to reproduce:
Clone revery quick start repo
I also tried to add the same 'resolutions' key int he package.json as the one in onivim 2, but no luck unfortunately.
"@opam/yojson": "github:onivim/yojson:yojson.opam#f480aef",
build error
Expected Result:
A successful build
Additional Information:
No source code was changed, and the lib was not added to any dune files. I have successfully managed to add the dependency in the esy hello-reason project.
Search terms used:
Linux
Build failed
libpangoft2
undefined reference to
hb_ot_layout_script_select_language' undefined reference to
hb_ot_var_named_instance_get_design_coords'undefined reference to
hb_ot_tags_from_script_and_language' undefined reference to
hb_ot_var_get_axis_infos'undefined reference to `hb_buffer_set_invisible_glyph'
The text was updated successfully, but these errors were encountered: