azul: Harfbuzz fails to build due to wrong Visual Studio version
Description
rustup 1.18.3 (435397f48 2019-05-22)
Version / OS
- azul version: azul = { git = “https://github.com/maps4print/azul” }
Steps to Reproduce
Trying to build
Additional Information
Getting this error:
error: failed to run custom build command for harfbuzz-sys v0.3.0 (https://github.com/maps4print/azul-dependencies?rev=bf2933b9aac43a7003278862772250398e4fa457#bf2933b9)
Caused by:
process didn’t exit successfully: D:\RustProjects\azul_test\azulapp\target\debug\build\harfbuzz-sys-94af335e9d1d05e2\build-script-build (exit code: 101)
— stdout
cargo:rerun-if-env-changed=HARFBUZZ_SYS_NO_PKG_CONFIG
— stderr thread ‘main’ panicked at ’
couldn’t determine visual studio generator
if VisualStudio is installed, however, consider running the appropriate vcvars script before building this crate
', C:\Users\user.cargo\git\checkouts\azul-dependencies-70bb1f94316762f9\bf2933b\cmake-0.1.35\src\lib.rs:687:25
note: Run with RUST_BACKTRACE=1 environment variable to display a backtrace.
warning: build failed, waiting for other jobs to finish… error: build failed
About this issue
- Original URL
- State: closed
- Created 5 years ago
- Comments: 15 (6 by maintainers)
Commits related to this issue
- Fix FreeType using the wrong integer size on Windows (fixes #203) — committed to fschutt/azul by fschutt 5 years ago
- Merge pull request #236 from NNemec/fix-VisualStudio2019 fix VS2019 build (#203) — committed to fschutt/azul by fschutt 4 years ago
Has this been fixed yet? I am also getting this issue.
It seems that
azul-dependenciesrefers to an older version that the dependencies fix forharfbuzz-sys 0.3.0has not been merged yet. This old rev is still includingcmake-0.1.35, which should becmake-0.1.40as mentioned at this PR.@fschutt can you fix this for us? Can’t wait for first try with azul but have to keep waiting ^^…
There is a temp way to work around. link
@metalwhale Use the branch
unvendor_dependencies, it’s fixed on there. But scrolling doesn’t work on that branch, that’s why it’s not merged into master yet.