Skip to content

Commit

Permalink
Properly find lconvert by multiple names
Browse files Browse the repository at this point in the history
It needs a NAMES parameter there. It is now also set to REQUIRED so that
it's clear what the error is.
  • Loading branch information
askmeaboutlo0m committed Aug 7, 2023
1 parent e024889 commit 834d666
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion cmake/DrawpileTranslations.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,7 @@ Combines a set of translation files into bundles.
function(bundle_translations out_files)
cmake_parse_arguments(PARSE_ARGV 1 ARG "" "NAME;OUTPUT_LOCATION" "QT;TARGETS")

find_program(lconvert "lconvert" "lconvert-qt${QT_VERSION_MAJOR}")
find_program(lconvert NAMES "lconvert" "lconvert-qt${QT_VERSION_MAJOR}" REQUIRED)

if(NOT ARG_NAME)
message(FATAL_ERROR "Missing required NAME for translation bundle")
Expand Down

0 comments on commit 834d666

Please sign in to comment.