depfile,dyndep: reconcile relative and absolute paths with build manifest #1925
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
Previously this was covered only as part of more complex tests.
Convert a working directory path to forward slashes, with a trailing slash.
Compute it from `getcwd`. Save it with a trailing slash for later use.
A depfile may contain relative and/or absolute paths based on compiler behavior and the include directory flags given. It is not always possible to make these exactly match corresponding paths in the build manifest. If Ninja does not find an exact match for a depfile path, try converting it to the opposite kind of path (absolute or relative) and look again. If neither matches, add a node with the path as-is. Issue: ninja-build#1251
A dyndep file may contain relative and/or absolute paths generated by external tooling just as a depfile might. Apply the same lookup rules we use for depfile paths to dyndep file paths too. Fixes: ninja-build#1251
|
This version differs from #1924 in that it does not add the |
|
Closing for now. I have some ideas on a more general solution. |
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.
A depfile (or dyndep file) may contain relative and/or absolute paths based on compiler
behavior and the include directory flags given. It is not always
possible to make these exactly match corresponding paths in the build
manifest. If Ninja does not find an exact match for a depfile path, try
converting it to the opposite kind of path (absolute or relative) and
look again. If neither matches, add a node with the path as-is.
Fixes: #1251