D5136: Introduce fetch-translations build command

Kevin Funk noreply at phabricator.kde.org
Wed Mar 22 19:14:59 UTC 2017


kfunk added inline comments.

INLINE COMMENTS

> KF5I18nConfig.cmake.in:12
> +
> +execute_process(COMMAND git config --get remote.origin.url
> +    OUTPUT_VARIABLE reponame

Can you delay that git invocation to when the custom target is executed? Otherwise we also pay for this whenever `KF5I18nConfig.cmake` is included

> KF5I18nConfig.cmake.in:17
> +)
> +
> +if(exitCode GREATER 0)

Style: Remove newline

> KF5I18nConfig.cmake.in:24
> +    OUTPUT "${CMAKE_BINARY_DIR}/releaseme"
> +    COMMAND git clone "https://anongit.kde.org/releaseme.git"
> +    COMMENT "Fetching releaseme scripts to download translations..."

`git clone --depth 0 ...`?

> KF5I18nConfig.cmake.in:30
> +    COMMAND cmake -E remove_directory ${CMAKE_BINARY_DIR}/po
> +    COMMAND ruby "${CMAKE_BINARY_DIR}/releaseme/fetchpo.rb" --origin stable --project ${reponame} ${CMAKE_CURRENT_SOURCE_DIR} ${CMAKE_BINARY_DIR}/po
> +    COMMENT "Downloading translations..."

--origin stable being hard-coded doesn't look right

REPOSITORY
  R249 KI18n

REVISION DETAIL
  https://phabricator.kde.org/D5136

To: apol, #frameworks, #build_system, aacid, sitter, kfunk
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170322/7f0242ec/attachment.html>


More information about the Kde-frameworks-devel mailing list