search

ptheywood / actions-playground

Testing / experimenting with github actions
0 stars 0 forks source link

Lint workflow / cmake target #3

Closed ptheywood closed 4 years ago

ptheywood commented 4 years ago

Need to figure out how to structure cmake for a CUDA project, so that linting can be achieved without requiring CUDA.

I.e don't set enable_language(CUDA) until as late as possible? Or provide the list of files separate to CMakeLists.txt so linting can be enabled without?

ptheywood commented 4 years ago

So far via USE_CXX flag this project doesn't need CXX for cpplint buidls

Lists of files are provided before the project is set to use the language, in a USE_CXX check.

This doesnt' work for clang-tidy, which requires the compile commands to be set. Could potentially spoof it by telling cmake that .cu files are actually cpp files, though it would likely trigger issues with missing headers in that case (can't use clang-tidy without nvcc)