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
Use -Dinstall-tests=true in more places #202
Conversation
This is for systemd/systemd#14338. Now all TEST-* tests need builds with -Dinstall-tests=true.
Thanks! One concern though: I dropped |
Oh, so this is an exact revert of your commit. I think that your commit looks wrong:
We can't meaningfully run tests (or other binaries in fact) from temporary install prefix. |
Ah, I see. Just to make things clear - I thought that |
Yes. #14338 makes it required to have |
Hm, in that case I'm not sure how to proceed here. We can't run /cc @evverx |
I will try to partially revert the patch to make TEST-01-BASIC do manual install again. |
This is useful for sanitizer builds under CI, see systemd/systemd-centos-ci#202 (comment).
This is useful for sanitizer builds under CI, see systemd/systemd-centos-ci#202 (comment).
This is useful for sanitizer builds under CI, see systemd/systemd-centos-ci#202 (comment).
This is useful for sanitizer builds under CI, see systemd/systemd-centos-ci#202 (comment).
This is useful for sanitizer builds under CI, see systemd/systemd-centos-ci#202 (comment).
Closing as this was addressed in systemd/systemd#14338 and we can run (Also, I just noticed I already did the exact same change in 73d4c60, sigh) |
This is for systemd/systemd#14338.
Now all TEST-* tests need builds with -Dinstall-tests=true.