Move common parts to the top-level CMakeLists.txt
ClosedPublic

Authored by heikobecker on Apr 19 2017, 5:06 PM.

Details

Summary

Besides de-duplication being a nice thing this also solves
translations getting installed into an undesired location. It
happens because at the time "ki18n_install(po)" (added on tarball
generation in order to install translations) is processed,
KDEInstallDirs is not yet included and LOCALE_INSTALL_DIR is not
defined, which leads to ki18n_install setting it to a default value.

Diff Detail

Repository
R451 KDE Development Utils
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
heikobecker created this revision.Apr 19 2017, 5:06 PM
ltoscano accepted this revision as: ltoscano.Apr 25 2017, 3:54 PM

Yes, thanks. I simply mirrored the old behavior when proting, but this makes more sense.

This revision is now accepted and ready to land.Apr 25 2017, 3:54 PM
This revision was automatically updated to reflect the committed changes.