Bug 171082 – update-webkit should perform a rebase-style merge when updating.

• Component: Tools / Tests
• Status: NEW

[Kocsen Chung]

update-webkit should perform a rebase-style merge when updating.

[Kocsen Chung]

Created attachment 307648 [details]
Patch

[Kocsen Chung]

<rdar://problem/31737887>

[Daniel Bates]

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.

[Kocsen Chung]

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?

[Michael Catanzaro]

This change seems fine to me. Surely almost all WebKit devs always want --rebase.

[Kocsen Chung]

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`.