Fix parallel building tests and pcheck in packages #736
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
When building in test directory with multiple jobs, the build sometimes failed. There was the following error when linking pmemdetect tool on one of its object files: *.o: file not recognized: File truncated The problem was that each and every test depends on pmemdetect tool and there is a rule in src/test/Makefile.inc which builds the pmemdetect from the test directory. The tests and tools were building in parallel and thus somethimes the pmemdetect was build twice (at least) and make could not synchronize this because the rules were in different makefiles. This patch fixes the dependencies in src/test/Makefile which fixes the issue.
... because make pcheck with multiple jobs may fail due to "No space on device error". We need to figure it out how to resolve this problem. Until then run tests in sequentially.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This change is