D5713: Fix quoting of preprocessor-definition.

2017-05-05 Thread Adriaan de Groot
This revision was automatically updated to reflect the committed changes. Closed by commit R241:def270105286: Fix quoting of preprocessor-definition. (authored by adridg). REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D5713?vs=14149&id=14159 REVISION DETAIL htt

D5713: Fix quoting of preprocessor-definition.

2017-05-05 Thread Ben Cooksley
bcooksley accepted this revision. bcooksley added a comment. This revision is now accepted and ready to land. From a theoretical perspective this looks fine to me. REPOSITORY R241 KIO BRANCH master REVISION DETAIL https://phabricator.kde.org/D5713 To: adridg, dfaure, bcooksley Cc: #fr

D5713: Fix quoting of preprocessor-definition.

2017-05-05 Thread Adriaan de Groot
adridg created this revision. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REVISION SUMMARY If the LIBEXECDIR contains spaces (as it does specifically on the CI systems) then this needs one extra layer of quoting to protect it from