Repository navigation
Pnpm-accepted patches without EOF annotations fail under aube #1514
Aube fails with Both pnpm versions apply the patch and preserve the absent trailing The repro uses a two-line local package with no registry dependencies. I encountered this while patching react-native-keyboard-controller@1.21.13. Could aube match pnpm's tolerance for the missing annotation? |
Replies: 2 comments
|
Confirmed and opened a fix in #1515. Aube was passing the patch directly to diffy, which compares newline bytes exactly: the final context line in the patch ends with a newline, while the source file's final line does not. The fix retries with an EOF annotation on matching final context, preserving the absent trailing newline. Explicit newline edits keep their existing behavior, and content mismatches still fail. Your reproduction now passes both the canonical and missing-marker variants with the patched aube 2.2.13 build against pnpm 11.10.0 and 12.3.4 on macOS arm64, including its exact output-byte assertions. All 127 linker tests and workspace Clippy checks pass. Added regression coverage for CRLF, multiple hunks and shifted offsets, explicit newline changes, and content mismatches. The fix is in the PR and has not been released yet; adding the missing annotations remains the workaround for the released version. This comment was generated by Codex. |
|
Retested Shipping fix: #1515 missing-eof-marker, included in 2.2.14. |
Confirmed and opened a fix in #1515.
Aube was passing the patch directly to diffy, which compares newline bytes exactly: the final context line in the patch ends with a newline, while the source file's final line does not. The fix retries with an EOF annotation on matching final context, preserving the absent trailing newline. Explicit newline edits keep their existing behavior, and content mismatches still fail.
Your reproduction now passes both the canonical and missing-marker variants with the patched aube 2.2.13 build against pnpm 11.10.0 and 12.3.4 on macOS arm64, including its exact output-byte assertions. All 127 linker tests and workspace Clippy checks pass. Added regression cover…