Skip to content

Commit

Permalink
CMake: The compile definition is ENABLE_NLS, not XZ_NLS
Browse files Browse the repository at this point in the history
The CMake variables were renamed and accidentally also
the compile definition was renamed. As a result, translation
support wasn't actually enabled in the executables.

Fixes: 29f77c7
  • Loading branch information
Larhzu committed Jul 2, 2024
1 parent 45d08ab commit a0df067
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -1953,7 +1953,7 @@ if(XZ_TOOL_LZMAINFO AND HAVE_DECODERS)
target_link_libraries(lzmainfo PRIVATE Intl::Intl)

target_compile_definitions(lzmainfo PRIVATE
XZ_NLS
ENABLE_NLS
PACKAGE="${TRANSLATION_DOMAIN}"
LOCALEDIR="${LOCALEDIR_DEFINITION}"
)
Expand Down Expand Up @@ -2111,7 +2111,7 @@ this many MiB of RAM if xz cannot determine the amount at runtime")
target_link_libraries(xz PRIVATE Intl::Intl)

target_compile_definitions(xz PRIVATE
XZ_NLS
ENABLE_NLS
PACKAGE="${TRANSLATION_DOMAIN}"
LOCALEDIR="${LOCALEDIR_DEFINITION}"
)
Expand Down

0 comments on commit a0df067

Please sign in to comment.