Bug 31458

Summary:

bugzilla-tool should post git binary diff

Product:

WebKit

Reporter:

Shinichiro Hamaji <hamaji>

Component:

Tools / Tests

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

abarth

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Bug Depends on:

31457

Bug Blocks:

Attachments:

Description	Flags
Patch v1	none
Patch v2	none

Shinichiro Hamaji

Reported 2009-11-12 23:29:34 PST

As we are getting ready for handling git binary patches, let's add --binary option into Git.create_patch() in scm.py.

Attachments
Patch v1 (2.29 KB, patch) 2009-11-12 23:30 PST, Shinichiro Hamaji	no flags	Details Formatted Diff Diff
Patch v2 (3.67 KB, patch) 2009-11-13 00:05 PST, Shinichiro Hamaji	no flags	Details Formatted Diff Diff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Shinichiro Hamaji

Comment 1 2009-11-12 23:30:45 PST

Created attachment 43139 [details] Patch v1

Shinichiro Hamaji

Comment 2 2009-11-12 23:31:38 PST

Note that the unittest I added will fail due to Bug 31457 .

Shinichiro Hamaji

Comment 3 2009-11-12 23:37:07 PST

Comment on attachment 43139 [details] Patch v1 Ah, I should have modified create_patch_from_local_commit and create_patch_since_local_commit as well. Clearing the review bit.

Shinichiro Hamaji

Comment 4 2009-11-13 00:05:50 PST

Created attachment 43141 [details] Patch v2

Shinichiro Hamaji

Comment 5 2009-11-13 00:07:02 PST

(In reply to comment #4) > Created an attachment (id=43141) [details] > Patch v2 I added --binary flag for these two commands as well.

Eric Seidel (no email)

Comment 6 2009-11-13 13:14:54 PST

Comment on attachment 43141 [details] Patch v2 LGTM.

Adam Barth

Comment 7 2009-11-15 15:32:43 PST

Comment on attachment 43141 [details] Patch v2 Clearing flags on attachment: 43141 Committed r51005: <http://trac.webkit.org/changeset/51005>

Adam Barth

Comment 8 2009-11-15 15:32:48 PST

All reviewed patches have been landed. Closing bug.

Shinichiro Hamaji

Comment 9 2009-11-15 21:40:26 PST

(In reply to comment #8) > All reviewed patches have been landed. Closing bug. Thanks for landing this patch. However, this patch will make unittest (GitTest.test_create_binary_patch) fail until Bug 31457 will be fixed. I guess we don't need to rollback this change because unittests aren't running in buildbot. I'll ask someone to review Bug 31457. I think I should have set commit-queue- for this patch to mention this fact, sorry.

Note You need to log in before you can comment on or make changes to this bug.