Skip to content

Commit

Permalink
bundled: Obey coretext, directwrite features when configuring harfbuz…
Browse files Browse the repository at this point in the history
…z. (#234)

When we build a bundled `harfbuzz`, only enable `HAVE_CORETEXT` and
`HAVE_DIRECTWRITE` when the corresponding features are set.

We don't correct `HAVE_FREETYPE` here as it has other larger issues
that need to be resolved first.
  • Loading branch information
waywardmonkeys committed Aug 28, 2023
1 parent 3552a2e commit f68701a
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions harfbuzz-sys/build.rs
Original file line number Diff line number Diff line change
Expand Up @@ -23,11 +23,11 @@ fn build_harfbuzz() {
cfg.define("HAVE_PTHREAD", "1");
}

if target.contains("apple") {
if target.contains("apple") && cfg!(feature = "coretext") {
cfg.define("HAVE_CORETEXT", "1");
}

if target.contains("windows") {
if target.contains("windows") && cfg!(feature = "directwrite") {
cfg.define("HAVE_DIRECTWRITE", "1");
}

Expand Down

0 comments on commit f68701a

Please sign in to comment.