Summary: | update-webkit should perform a rebase-style merge when updating. | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Kocsen Chung <kocsen_chung> | ||||
Component: | Tools / Tests | Assignee: | 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
Kocsen Chung
2017-04-20 15:39:28 PDT
Created attachment 307648 [details]
Patch
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. 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? This change seems fine to me. Surely almost all WebKit devs always want --rebase. 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`. |