Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

EnumToString.tests.cpp fails to compile #1613

Closed
WentsingNee opened this issue Apr 26, 2019 · 2 comments
Closed

EnumToString.tests.cpp fails to compile #1613

WentsingNee opened this issue Apr 26, 2019 · 2 comments

Comments

@WentsingNee
Copy link

cmake info

cmake info

[ 16%] Building CXX object projects/CMakeFiles/SelfTest.dir/SelfTest/UsageTests/Generators.tests.cpp.o
/home/peter/git/lib/Catch2/projects/SelfTest/UsageTests/EnumToString.tests.cpp:93:47: error: extra ‘;’ [-Werror=pedantic]
                      Bikeshed::Colours::Blue );
                                               ^
cc1plus: all warnings being treated as errors
projects/CMakeFiles/SelfTest.dir/build.make:422: recipe for target 'projects/CMakeFiles/SelfTest.dir/SelfTest/UsageTests/EnumToString.tests.cpp.o' failed
make[2]: *** [projects/CMakeFiles/SelfTest.dir/SelfTest/UsageTests/EnumToString.tests.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
CMakeFiles/Makefile2:981: recipe for target 'projects/CMakeFiles/SelfTest.dir/all' failed
make[1]: *** [projects/CMakeFiles/SelfTest.dir/all] Error 2
Makefile:162: recipe for target 'all' failed
make: *** [all] Error 2

Platform information:

  • OS: Linux Mint 19.1 Cinnamon
  • Compiler+version: GCC v8.2.0
  • commit: 6267b06
@mlimber
Copy link
Contributor

mlimber commented Apr 26, 2019

Travis failed for #1614 failed because of this. :-(

@philsquared
Copy link
Collaborator

sorry about that - it was my misreading of the branch results. Should all be fixed now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants