Skip to content

Fix cmake in Release mode#2655

Merged
danmar merged 1 commit into
cppcheck-opensource:masterfrom
Ken-Patrick:cmake_tools
May 22, 2020
Merged

Fix cmake in Release mode#2655
danmar merged 1 commit into
cppcheck-opensource:masterfrom
Ken-Patrick:cmake_tools

Conversation

@Ken-Patrick
Copy link
Copy Markdown
Contributor

In tools/, it did not understand that some files are generated by
matchcompiler.py.

CMake Error at tools/CMakeLists.txt:7 (add_executable):
  Cannot find source file:

    </some/path>/cppcheck/build/lib/build/mc_pathmatch.cpp

  Tried extensions .c .C .c++ .cc .cpp .cxx .cu .m .M .mm .h .hh .h++ .hm
  .hpp .hxx .in .txx

CMake Error at tools/CMakeLists.txt:7 (add_executable):
  No SOURCES given to target: dmake

In tools/, it did not understand that some files are generated by
matchcompiler.py.

```
CMake Error at tools/CMakeLists.txt:7 (add_executable):
  Cannot find source file:

    </some/path>/cppcheck/build/lib/build/mc_pathmatch.cpp

  Tried extensions .c .C .c++ .cc .cpp .cxx .cu .m .M .mm .h .hh .h++ .hm
  .hpp .hxx .in .txx

CMake Error at tools/CMakeLists.txt:7 (add_executable):
  No SOURCES given to target: dmake

```
@danmar
Copy link
Copy Markdown
Collaborator

danmar commented May 21, 2020

to me (CMake noob) this looks fine to merge.. any opinions from anybody else before I merge it?

@firewave
Copy link
Copy Markdown
Collaborator

Looks good. I saw that error a few times, but always when switching around branches and could never reproduce it.
That code should be put into a function though and not be duplicated. Will take a look at that when all my other CMake-related things are finally merged.

@danmar danmar merged commit d127985 into cppcheck-opensource:master May 22, 2020
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 this pull request may close these issues.

3 participants