Skip to content

Commit

Permalink
Ignore lupdate warnings/failures with C++17 code
Browse files Browse the repository at this point in the history
[skip appveyor]

lupdate is broken with C++17.

Issue #250
  • Loading branch information
rakhimov committed Jan 6, 2018
1 parent aabccc2 commit 2e757db
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
4 changes: 2 additions & 2 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -56,13 +56,13 @@ jobs:
include:
- stage: lint
compiler: clang
before_script: mkdir build
script: .travis/lint.sh
env: CONFIG=Lint # Travis does not provide ENV var for stage names :(
addons:
apt:
packages:
- qttools5-dev-tools
- qt59tools
sources: *Sources
# - stage: test
# os: osx # OS X builds are slow and limited.
# osx_image: xcode9.2 # C++17 not yet fully supported!!!
Expand Down
3 changes: 2 additions & 1 deletion .travis/lint.sh
Original file line number Diff line number Diff line change
Expand Up @@ -52,5 +52,6 @@ lupdate gui -ts gui/translations/scramgui_en.ts 2> ts_warnings.txt || (cat ts_wa
if [[ -s ts_warnings.txt ]]; then
echo "Qt Lupdate warnings:" >&2
cat ts_warnings.txt >&2
exit 1
echo "lupdate C++17 issues"
# exit 1
fi

0 comments on commit 2e757db

Please sign in to comment.