Test build: Reduce include command length #6968
Merged
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.
Description
Resolves #6577
Tests are built in two passes:
.o
files).o
The second pass involves scanning the build directory creating in the first
pass. This is where the bug lies: the second pass takes an absolute path to
the artifacts create by the first. This adds the directory containing mbed-os
to the each include path, and these extra characters exceed the maximum
command line length on windows.
This patch uses a relate path, eliminating the problem
Pull request type