D5143: Introduce fetch-translations build command
Harald Sitter
noreply at phabricator.kde.org
Wed Apr 5 13:33:30 UTC 2017
sitter added a comment.
needs a version bump in the docs now 👅
beyond that LGTM
INLINE COMMENTS
> KDECMakeSettings.cmake:86
> +#
> +# Since 5.33.0
>
5.34 now
> KDECMakeSettings.cmake:323
> + COMMAND ruby "${CMAKE_BINARY_DIR}/releaseme/fetchpo.rb" --origin ${KDE_L10N_BRANCH} --project "${reponame}" "${CMAKE_CURRENT_SOURCE_DIR}" "${CMAKE_BINARY_DIR}/po"
> + BYPRODUCTS "${CMAKE_BINARY_DIR}/po"
> + DEPENDS "${CMAKE_BINARY_DIR}/releaseme"
This was only introduced in cmake 3.2. I am not sure we care or if this even would cause problems beyond a warning, but technically frameworks are 3.0 compatible (or so they claim anyway ;)). Seeing as the feature is opt-in it probably doesn't matter eitherway.
REPOSITORY
R240 Extra CMake Modules
REVISION DETAIL
https://phabricator.kde.org/D5143
To: apol, #frameworks, #build_system, kfunk, ltoscano, aacid, ilic
Cc: sitter
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20170405/d4778d6b/attachment.html>
More information about the Kde-buildsystem
mailing list