Closed Bug 1610583 Opened 2 years ago Closed 2 years ago

moz-phab doesn't correctly detect files without trailing newlines, resulting in patches that cannot be landed.

Categories

(Conduit :: moz-phab, defect, P1)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mgaudet, Assigned: glob)

References

Details

(Keywords: conduit-triaged)

Attachments

(2 files)

I seem to be having trouble landing a patch that removes a file, and I'm not super clear on why. As near as I can tell, the file hasn't been modified since I first landed it; yet the landing fails

applying /tmp/tmpO_JPtY 1 out of 1 hunk ignored -- saving rejects to file js/src/jit-test/tests/bug1580378.js.rej abort: patch command failed: exited with status 256

https://lando.services.mozilla.com/D60461/ is the patch stack that won't land, and https://phabricator.services.mozilla.com/D60452 is the only diff in the stack that touches (deletes!) bug1580378.js

if I look at the diff locally, it looks like it was deleted:

diff --git a/js/src/jit-test/tests/bug1580378.js b/js/src/jit-test/tests/bug1580378.js
deleted file mode 100644
--- a/js/src/jit-test/tests/bug1580378.js
+++ /dev/null

Aryx helpfully pointed out in #lando on slack: bug1580378.js is missing a trailing newline; so in a preceeding patch, I added a newline, then deleted and it landed.

It appears this is not the only case of this: https://lando.services.mozilla.com/D59763/ is another case.

Thanks for the clear bug report - investigating.

:bytesized as a workaround you can ask a sheriff to manually land your patch.

Attached file transplant log β€”

Looks like the diff generated by moz-phab doesn't contain the marker that the deleted file is lacking a trailing newline.

Assignee: nobody → glob
No longer blocks: 1607575
Component: Lando → moz-phab
Keywords: conduit-triaged
Priority: -- → P1
Summary: Can't land a diff that deletes a file that is missing a trailing newline. → moz-phab doesn't correctly detect files without trailing newlines, resulting in patches that cannot be landed.
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
See Also: → 1644874
You need to log in before you can comment on or make changes to this bug.