Improve tde-create-translated-desktop for generated source files #68
Merged
SlavekB
merged 3 commits from feat/create-translate-desktop-from-generated-source
into master
3 years ago
Loading…
Reference in new issue
There is no content yet.
Delete Branch 'feat/create-translate-desktop-from-generated-source'
Deleting a branch is permanent. It CANNOT be undone. Continue?
This PR solves several things:
Newer versions of CMake may display a warning when
find_program
is used for a script that does not have the executable bit set. This relates to incorporatedintltool-merge
, see commit53c7e28984
.When used on a generated desktop file – a file located in
CMAKE_BINARY_DIR
– this caused the working files to be generated in a confusing folder structure. Although it did not affect functionality, it is better if they are generated in the right place.To use
intltool-merge
, desktop file preprocessing was performed there already in the configuration phase. This made it impossible to use macro for generated desktop files. Respectively, such desktop files also had to be generated already in the configuration phase. This causes warning messages when usingninja-build
. Therefore, it is better that everything can take place naturally in the build phase.LGTM
1686d0e64e
into master 3 years agoReviewers
1686d0e64e
.