Skip to content
Open
Show file tree
Hide file tree
Changes from 18 commits
Commits
Show all changes
25 commits
Select commit Hold shift + click to select a range
77494be
Adds QtCreator project files to .gitignore
Oct 29, 2019
6eb22f0
Add a lint blacklist so lint checks don't fail for autogenerated files.
Oct 29, 2019
64fcd5b
Proper escaping for when a \ is in the README
Oct 29, 2019
9edc0b0
Removed line at request of Peter
Oct 29, 2019
1294afc
SNAKE_CASE and other consistency changes.
Oct 30, 2019
c33151c
Fixed an embarrasing oversight.
Oct 30, 2019
02ddd33
Spelling fixes...
Oct 30, 2019
f133157
Switched sed back from '#' to '/' at Peters' request.
Oct 31, 2019
2f1b143
Fix spellintian "issue"
Nov 3, 2019
203fc3c
undo spellintian suggested fix of LGPL
Nov 6, 2019
5e90d9c
Switch Master to Ubuntu 18.04 LTS
Nov 10, 2019
0875169
Swicth Master Travis tests to Ubuntu 18.04 also remove all OS X tests…
Nov 10, 2019
f3490d3
This time *all* references to xenial were removed, not just in the co…
Nov 10, 2019
d6b90ea
Undo changes to travis test matrix.
Nov 10, 2019
3668733
Merge branch 'master-resync' into logistics
Nov 10, 2019
07f7d27
Fix the allowed failures OS X list after the merge
peternewman Nov 10, 2019
46dad86
Undid debug changes.
Nov 10, 2019
15d9afd
Merge remote-tracking branch 'origin/logistics' into logistics
Nov 10, 2019
676f39a
Fixed regex oversight, also move to next line to prevent future overs…
Nov 10, 2019
a54ee4d
Modified regex to insert newlines.
Nov 10, 2019
133d3e0
remove unneeded \+
Nov 10, 2019
7c65f2c
Merge branch 'master' into logistics
peternewman Nov 10, 2019
426654c
Explicitely switches script to bash, correctly renders newlines and c…
Nov 10, 2019
d4270e5
Merge remote-tracking branch 'refs/remotes/origin/logistics' into log…
Nov 10, 2019
80908c3
revert
Nov 11, 2019
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -246,3 +246,8 @@ javascript/new-src/node_modules
.cproject
.settings
.vscode/
# QtCreator project files
*.config
*.creator*
*.files
*.includes
33 changes: 19 additions & 14 deletions .travis-ci.sh
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,19 @@ COVERITY_SCAN_BUILD_URL="https://scan.coverity.com/scripts/travisci_build_coveri

PYCHECKER_BLACKLIST="threading,unittest,cmd,optparse,google,google.protobuf,ssl,fftpack,lapack_lite,mtrand"

SPELLINGBLACKLIST=$(cat <<-BLACKLIST
LINT_BLACKLIST=$(cat <<-EO_LINT_BL
-wholename "./common/protocol/Ola.pb.*" -or \
-wholename "./common/rpc/Rpc.pb.*" -or \
-wholename "./common/rpc/TestService.pb.*" -or \
-wholename "./common/rdm/Pids.pb.*" -or \
-wholename "./config.h" -or \
-wholename "./plugins/*/messages/*ConfigMessages.pb.*" -or \
-wholename "./tools/ola_trigger/config.tab.*" -or \
-wholename "./tools/ola_trigger/lex.yy.cpp"
EO_LINT_BL
)

SPELLING_BLACKLIST=$(cat <<-EO_SPELL_BL
-wholename "./.codespellignore" -or \
-wholename "./.git/*" -or \
-wholename "./aclocal.m4" -or \
Expand Down Expand Up @@ -43,7 +55,7 @@ SPELLINGBLACKLIST=$(cat <<-BLACKLIST
-wholename "./plugins/artnet/messages/ArtNetConfigMessages.pb.*" -or \
-wholename "./tools/ola_trigger/config.tab.*" -or \
-wholename "./tools/ola_trigger/lex.yy.cpp"
BLACKLIST
EO_SPELL_BL
)

if [[ $TASK = 'lint' ]]; then
Expand All @@ -69,19 +81,12 @@ if [[ $TASK = 'lint' ]]; then
echo "Found $nolints generic NOLINTs"
fi;
# then fetch and run the main cpplint tool
lintfiles=$(eval "find ./ \( -name "*.h" -or -name "*.cpp" \) -and ! \( $LINT_BLACKLIST \) | xargs")
wget -O cpplint.py $CPP_LINT_URL;
chmod u+x cpplint.py;
./cpplint.py \
--filter=-legal/copyright,-readability/streams,-runtime/arrays \
$(find ./ \( -name "*.h" -or -name "*.cpp" \) -and ! \( \
-wholename "./common/protocol/Ola.pb.*" -or \
-wholename "./common/rpc/Rpc.pb.*" -or \
-wholename "./common/rpc/TestService.pb.*" -or \
-wholename "./common/rdm/Pids.pb.*" -or \
-wholename "./config.h" -or \
-wholename "./plugins/*/messages/*ConfigMessages.pb.*" -or \
-wholename "./tools/ola_trigger/config.tab.*" -or \
-wholename "./tools/ola_trigger/lex.yy.cpp" \) | xargs)
$lintfiles
if [[ $? -ne 0 ]]; then
exit 1;
fi;
Expand Down Expand Up @@ -116,7 +121,7 @@ elif [[ $TASK = 'spellintian' ]]; then
make builtfiles;
travis_fold end "make_builtfiles"
spellingfiles=$(eval "find ./ -type f -and ! \( \
$SPELLINGBLACKLIST \
$SPELLING_BLACKLIST \
\) | xargs")
# count the number of spellintian errors, ignoring duplicate words
spellingerrors=$(zrun spellintian $spellingfiles 2>&1 | grep -v "\(duplicate word\)" | wc -l)
Expand All @@ -142,7 +147,7 @@ elif [[ $TASK = 'spellintian-duplicates' ]]; then
make builtfiles;
travis_fold end "make_builtfiles"
spellingfiles=$(eval "find ./ -type f -and ! \( \
$SPELLINGBLACKLIST \
$SPELLING_BLACKLIST \
\) | xargs")
# count the number of spellintian errors
spellingerrors=$(zrun spellintian $spellingfiles 2>&1 | wc -l)
Expand All @@ -168,7 +173,7 @@ elif [[ $TASK = 'codespell' ]]; then
make builtfiles;
travis_fold end "make_builtfiles"
spellingfiles=$(eval "find ./ -type f -and ! \( \
$SPELLINGBLACKLIST \
$SPELLING_BLACKLIST \
\) | xargs")
# count the number of codespell errors
spellingerrors=$(zrun codespell --check-filenames --check-hidden --quiet 2 --regex "[a-zA-Z0-9][\\-'a-zA-Z0-9]+[a-zA-Z0-9]" --exclude-file .codespellignore $spellingfiles 2>&1 | wc -l)
Expand Down
18 changes: 18 additions & 0 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -256,19 +256,37 @@ matrix:
- TASK='compile'
- CPPUNIT='1.14'
- LIBFTDI='0'
- os: osx
osx_image: xcode9.3
compiler: clang
env:
- TASK='compile'
- CPPUNIT='1.13'
- PROTOBUF='3.6'
- LIBFTDI='1'
- os: osx
osx_image: xcode9.3
compiler: gcc
env:
- TASK='compile'
- CPPUNIT='1.13'
- PROTOBUF='3.6'
- LIBFTDI='1'
- os: osx
osx_image: xcode9.3
compiler: clang
env:
- TASK='compile'
- CPPUNIT='1.14'
- PROTOBUF='3.6'
- LIBFTDI='1'
- os: osx
osx_image: xcode9.3
compiler: gcc
env:
- TASK='compile'
- CPPUNIT='1.14'
- PROTOBUF='3.6'
- LIBFTDI='1'
- os: linux
dist: xenial
Expand Down
4 changes: 2 additions & 2 deletions plugins/convert_README_to_header.sh
Original file line number Diff line number Diff line change
Expand Up @@ -30,8 +30,8 @@ outfilename=`basename $outfile`;
# See http://stackoverflow.com/a/16576291
# On Mac OS's sed, \n is not recognized as a newline character, but
# \[actual newline] works
desc=`sed -e ':a' -e 'N' -e '$!ba' -e 's/\"/\\\"/g' -e 's/\n/\\\\n"\\
"/g' "$path/README.md"`;
desc=`sed -e ':a' -e 'N' -e '$!ba' -e 's/\\\/\\\\\\\/g' -e 's/\"/\\\"/g' -e 's/\n/\\\\n"\\
"#g' "$path/README.md"`;

identifier=`echo "PLUGINS_${plugin}_${outfilename%.h}_H_" | tr '[:lower:]' '[:upper:]'`

Expand Down