D5890: Pass `--key-positions=*` to gperf instead of `-k *` because on Windows this symbol seems to have a special meaning.

2017-05-18 Thread Gleb Popov
This revision was automatically updated to reflect the committed changes. Closed by commit R313:07a8a4e6985c: Pass `--key-positions=*` to gperf instead of `-k *` because on Windows this… (authored by arrowdodger). REPOSITORY R313 KHtml CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D5

D5890: Pass `--key-positions=*` to gperf instead of `-k *` because on Windows this symbol seems to have a special meaning.

2017-05-18 Thread Gleb Popov
arrowdodger added a comment. In https://phabricator.kde.org/D5890#110586, @bcooksley wrote: > This looks good to me. Would this also resolve https://build-sandbox.kde.org/view/Frameworks/job/Frameworks%20kcodecs%20kf5-qt5%20WindowsQt5.7/6/console ? > Invalid position value

D5890: Pass `--key-positions=*` to gperf instead of `-k *` because on Windows this symbol seems to have a special meaning.

2017-05-18 Thread Ben Cooksley
bcooksley accepted this revision. bcooksley added a comment. This revision is now accepted and ready to land. This looks good to me. Would this also resolve https://build-sandbox.kde.org/view/Frameworks/job/Frameworks%20kcodecs%20kf5-qt5%20WindowsQt5.7/6/console ? REPOSITORY R313 KHtml BR

D5890: Pass `--key-positions=*` to gperf instead of `-k *` because on Windows this symbol seems to have a special meaning.

2017-05-16 Thread Gleb Popov
arrowdodger created this revision. Restricted Application added a project: Frameworks. REVISION SUMMARY Pass `--key-positions=*` to gperf instead of `-k *` because on Windows this symbol seems to have a special meaning. TEST PLAN Successfully compiled with MSVC. REPOSITORY R313 KHtml BRA