[Bug] - Incorrectly handled absolute paths#1060
Conversation
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f5f061e084
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
another go at the effort started in #1057
Note
Improves COPY handling to support absolute and escaped-relative paths by normalizing sources and attaching a per-instruction
contextPathused end-to-end.normalizeCopySourcePath/normalize_copy_source_path; COPY now stores normalized src andcontextPathinstruction.contextPathforcalculateFilesHash,readDockerignore, anduploadFilecwd, POSIX paths) ingetAllFilesInPathand tar path constructionInstructiontypes with optionalcontextPathWritten by Cursor Bugbot for commit 60cc754. This will update automatically on new commits. Configure here.