I'm not sure how it's possible that pofiles is defined but tsfiles
isn't, but this does fix the issue reported by Luigi:
CMake Error at /usr/lib64/cmake/KF5I18n/KF5I18NMacros.cmake:129
(add_dependencies):
Cannot add target-level dependencies to non-existent target "tsfiles".
Built svn+ssh://svn@svn.kde.org/home/kde/trunk/l10n-kf5/fr by doing
./scripts/autogen.sh fr; mkdir build; cd build; cmake ../fr
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
locale fr or uk build now, but without the script files beeing installed.
Looks like ./scripts/autogen.sh needs to be adapted as well
No, scripts/autogen.sh doesn't need to be adapted, what Aleix needs to do is fix the regression he caused properly
It seems to work, just one question though:
| cmake/KF5I18NMacros.cmake | ||
|---|---|---|
| 178 | Before this line, in the original version before the changes, there was this line: string(REPLACE "@" "_" pmapc_target ${pmapc_target})Could it be still relevant? | |
| cmake/KF5I18NMacros.cmake | ||
|---|---|---|
| 178 | You are right, I must have copied a version that didn't have it yet. Fine by me, included. | |
The issue initially reported is fixed. I tried with fr and other 3 languages which should be representative enough (de, uk, sr@ijekavian; the latter was complaining before the last revision of the review).