Skip to content

thirdparty[harfbuzz]: upgrade harfbuzz 2.8.2 => 14.2.1 - #374

Closed
teleshoes wants to merge 1 commit into
buggins:masterfrom
teleshoes:thirdparty_upgrade_harfbuzz
Closed

thirdparty[harfbuzz]: upgrade harfbuzz 2.8.2 => 14.2.1#374
teleshoes wants to merge 1 commit into
buggins:masterfrom
teleshoes:thirdparty_upgrade_harfbuzz

Conversation

@teleshoes

Copy link
Copy Markdown
Contributor

updated dependency to fix build error in audiobook_in_tts
modern C++ prevents compiling any harfbuzz version < 9.0.0 for me

@buggins

buggins commented Jul 20, 2026

Copy link
Copy Markdown
Owner

I''ve already updated all libraries under another PR #375

@buggins

buggins commented Jul 20, 2026

Copy link
Copy Markdown
Owner

Your PR is probably obsolete.

@teleshoes teleshoes closed this Jul 20, 2026
@teleshoes teleshoes reopened this Jul 20, 2026
@teleshoes
teleshoes force-pushed the thirdparty_upgrade_harfbuzz branch from d683485 to a12c0b0 Compare July 20, 2026 13:51
@teleshoes

Copy link
Copy Markdown
Contributor Author

actually, i get this while trying to build android on the latest:

  CMake Error at thirdparty_libs/harfbuzz/CMakeLists.txt:64 (add_library):
    Cannot find source file:
  
      /home/teleshoes/Code/coolreader/thirdparty/harfbuzz-14.2.1/src/hb-ot-shape-complex-arabic.cc
  
    Tried extensions .c .C .c++ .cc .cpp .cxx .cu .mpp .m .M .mm .ixx .cppm .h
    .hh .h++ .hm .hpp .hxx .in .txx .f .F .for .f77 .f90 .f95 .f03 .hip .ispc
  
  
  CMake Error at thirdparty_libs/harfbuzz/CMakeLists.txt:64 (add_library):
    No SOURCES given to target: local_harfbuzz

many source files moved in harfbuzz.

i rebased on your upgrade and included only the source files moving, which fixes the compilation issues i'm having

@teleshoes teleshoes closed this Jul 20, 2026
@teleshoes
teleshoes deleted the thirdparty_upgrade_harfbuzz branch July 20, 2026 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants