Merge lp:~dobey/cmake-extras/simpler-potfile into lp:cmake-extras

Proposed by dobey
Status: Merged
Approved by: dobey
Approved revision: 83
Merged at revision: 67
Proposed branch: lp:~dobey/cmake-extras/simpler-potfile
Merge into: lp:cmake-extras
Prerequisite: lp:~dobey/cmake-extras/it-append-file-globs
Diff against target: 24 lines (+2/-4)
1 file modified
src/Intltool/IntltoolConfig.cmake (+2/-4)
To merge this branch: bzr merge lp:~dobey/cmake-extras/simpler-potfile
Reviewer Review Type Date Requested Status
Pete Woods Approve
Charles Kerr (community) Approve
Review via email: mp+318030@code.launchpad.net

Commit message

Don't use the gettext unique target magic for pot files.

To post a comment you must log in.
Revision history for this message
Charles Kerr (charlesk) :
review: Approve
Revision history for this message
Pete Woods (pete-woods) wrote :

I guess we're making the assumption that valid filename characters are also valid target names. But even assuming that's true (which it likely is for most stuff we care about) it'd be nice to put it in quotes.

review: Needs Fixing
Revision history for this message
Pete Woods (pete-woods) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/Intltool/IntltoolConfig.cmake'
--- src/Intltool/IntltoolConfig.cmake 2017-02-28 17:32:22 +0000
+++ src/Intltool/IntltoolConfig.cmake 2017-02-28 17:32:22 +0000
@@ -297,18 +297,16 @@
297 ${_PO_DIRECTORY}297 ${_PO_DIRECTORY}
298 )298 )
299 299
300 _GETTEXT_GET_UNIQUE_TARGET_NAME(${_POT_FILE} _UNIQUE_TARGET_NAME)
301
302 if(_ARG_ALL)300 if(_ARG_ALL)
303 add_custom_target(301 add_custom_target(
304 ${_UNIQUE_TARGET_NAME}302 "${_POT_FILE}"
305 ALL303 ALL
306 DEPENDS304 DEPENDS
307 "${_PO_DIRECTORY}/${_POT_FILE}"305 "${_PO_DIRECTORY}/${_POT_FILE}"
308 )306 )
309 else()307 else()
310 add_custom_target(308 add_custom_target(
311 ${_UNIQUE_TARGET_NAME}309 "${_POT_FILE}"
312 DEPENDS310 DEPENDS
313 "${_PO_DIRECTORY}/${_POT_FILE}"311 "${_PO_DIRECTORY}/${_POT_FILE}"
314 )312 )

Subscribers

People subscribed via source and target branches

to all changes: