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

Using ctest for launching tests #25

Closed
hpwxf opened this issue Jun 29, 2020 · 3 comments · Fixed by #29
Closed

Using ctest for launching tests #25

hpwxf opened this issue Jun 29, 2020 · 3 comments · Fixed by #29

Comments

@hpwxf
Copy link
Collaborator

hpwxf commented Jun 29, 2020

It could be convenient to run tests using ctest.
To do that, it requires to keep generated library in build directory (and not to copy them into tests directory).
By that way, it would be possible to have in parallel several build directories for several configurations (e.g. Debug, StaticAnalysis, Release...)

Again, if it could be a good idea, I can do it.

@RUrlus
Copy link
Owner

RUrlus commented Jun 29, 2020

Not against it in principle. How does travis handle multiple parallel builds in a single node?
This would involve changing the travis config which is a bit of a mess.
Again, feel free to pick this up, otherwise I'll have a look but that might take a few weeks.

@hpwxf
Copy link
Collaborator Author

hpwxf commented Jun 29, 2020

When I said "parallel build...", it was about "several build directories" with a given configuration for each. This happens in the development process when one wants to compile/test Release and Debug configurations (and in that case, if there are generated files out of build directory they could conflict).

Using "parallel build" in the sense of "compiling several units in parallel", is possible even with a single node (using -j <n> option) but the benefit could be low, especially in Travis.
(while compiling a file, file accesses are typically disk-bound and if there are also other compilation processes, you should maximize CPU usage).

@RUrlus
Copy link
Owner

RUrlus commented Jun 29, 2020

For some reason I had a third option in mind where all configs were build in the same travis job in parallel.
Anyway sounds good, happy to learn how to do it better.

@RUrlus RUrlus linked a pull request Jun 30, 2020 that will close this issue
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.

2 participants