Add -DBUILD_TESTING=OFF to disable tests

PLSysSec / rlbox

RLBox sandboxing framework

https://rlbox.dev

MIT License

285 stars 21 forks source link

Closed bansan85 closed 2 years ago

bansan85 commented 3 years ago

and Catch2 dependency.

Fix #28

There is no need to update CI. Tests are always enabled by default.

I also add a minor documentation about make install. I don't think it's mandatory to add a message about the empty make when tests are disabled.

bansan85 commented 2 years ago

Polite ping ❤️ 😃

shravanrn commented 2 years ago

Polite ping ❤️ 😃

Terribly sorry about the long delay. I will definitely review the PR this week :)

shravanrn commented 2 years ago

@bansan85 I pushed a rebased commit implementing this in the tree

bansan85 commented 2 years ago

Thanks for rebased my commit :) I'm in a rush in my personal time. I didn't write a single line of code for a week :/