Bug 171082

Summary: update-webkit should perform a rebase-style merge when updating.
Product: WebKit Reporter: Kocsen Chung <kocsen_chung>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: NEW ---    
Severity: Normal CC: aakash_jain, ap, bburg, buildbot, clopez, dbates, ddkilzer, kocsen_chung, lforschler, mcatanzaro
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch dbates: review-

Description Kocsen Chung 2017-04-20 15:39:28 PDT
update-webkit should perform a rebase-style merge when updating.
Comment 1 Kocsen Chung 2017-04-20 15:40:15 PDT
Created attachment 307648 [details]
Patch
Comment 2 Kocsen Chung 2017-04-20 16:17:07 PDT
<rdar://problem/31737887>
Comment 3 Daniel Bates 2017-04-20 19:17:59 PDT
Comment on attachment 307648 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=307648&action=review

> Tools/ChangeLog:10
> +        to mimic `git svn rebase` behavior in the case of a git-svn repo.

Why do we need to force a particular workflow on people's local branches? We should email webkit-dev to see if anyone would object to forcing a rebase workflow for their local branches.
Comment 4 Kocsen Chung 2017-04-20 20:47:40 PDT
I don't see this as forcing a specific workflow - I see it more as more better mirroring the workflow WebKit currently uses.

Those on git-svn have no option to update their checkout other than using git-svn **rebase** which is achieved in pure git with the `--rebase` flag.

== 

Having said that that is not the intent of this change. It is more to alleviate the problem from the radar in Comment 2.
That particular problem could be easily remediated without any changes to the WebKit codebase.

What are your thoughts?
Comment 5 Michael Catanzaro 2017-04-20 21:19:19 PDT
This change seems fine to me. Surely almost all WebKit devs always want --rebase.
Comment 6 Kocsen Chung 2017-04-21 15:07:22 PDT
Upon second thought, it's important to realize that --rebase is a destructive action as it rewrites history. Although we can infer that this is what most people want we do not have the data.
This means we should probably start a discussion amongst webkit-dev to see if this would impact other workflows that we are unaware of. Only then can we be confident to even change this.

Keeping in mind that the current workflow allows those who would want to opt-in to `--rebase` workflow to set it in their git configuration as so with: `git config pull.rebase true`.