Open
Bug 514802
Opened 15 years ago
Updated 11 years ago
Interdiff is missing some changes between patches
Categories
(Bugzilla :: Attachments & Requests, defect)
Bugzilla
Attachments & Requests
Tracking
()
NEW
People
(Reporter: jruderman, Unassigned)
Details
https://bugzilla.mozilla.org/attachment.cgi?oldid=395372&action=interdiff&newid=398801&headers=1 is missing some of the changes between the two patches. For example, the two patches make different changes to nsFrameList.h, and the interdiff doesn't show anything for nsFrameList.h.
Interdiff is significantly less useful when you can't rely on it not to lie :(
bz pointed this out and asked me to file.
Comment 1•15 years ago
|
||
Comment 2•15 years ago
|
||
Note that this is most likely a bug in "interdiff" itself, which is what we're using in the backend (we actually call the binary).
Comment 3•15 years ago
|
||
Max, could it be that when interdiff fails bugzilla silently ignores the error?
Comment 4•15 years ago
|
||
That's a possibility; I can look into it. interdiff is also somewhat unreliable in various strange circumstances. I'll check it out using the two patches that Jesse posted, though.
Comment 5•15 years ago
|
||
If you need some other examples where the bugzilla interdiff just silently leaves hunks out, just let me know.
Comment 6•15 years ago
|
||
Indeed, bz is correct, at least from my brief investigation. Running interdiff manually returns a valid interdiff, but it also throws:
1 out of 3 hunks FAILED -- saving rejects to file /tmp/interdiff-1.UDLniZ.rej
interdiff: Error applying patch1 to reconstructed file
My guess is that in this situation, there's something in Bugzilla or PatchReader that causes it to fail.
Target Milestone: --- → Bugzilla 3.6
Comment 7•14 years ago
|
||
Bugzilla 3.6 is now restricted to security fixes only, and this bug got no traction for several months. We will retarget this bug once it has a patch ready for checkin.
Target Milestone: Bugzilla 3.6 → ---
You need to log in
before you can comment on or make changes to this bug.
Description
•