Thread (3 messages) flat view 3 messages, 3 authors, 2016-06-15

Re: [PATCH 3/3] rebase: refuse to rebase with -s ours

From: Johannes Schindelin <hidden>
Date: 2016-06-15 22:47:43

Hi,

On Mon, 16 Nov 2009, Junio C Hamano wrote:
Johannes Schindelin [off-list ref] writes:
quoted
On Sun, 15 Nov 2009, Thomas Rast wrote:
quoted
Using the "ours" strategy with rebase just discards all changes, 
turning <branch> into <upstream> (or <newbase> if given).  This is 
unlikely to be what the user wants, so simply refuse to do it.
quoted
Besides, I find it rather arbitrary that the "ours" strategy is 
refused, but none of the user-provided merge strategies.  IOW 
disallowing "ours" may very well foster unreasonable expectations.
I cannot read this quite clearly.
I meant the following: if "rebase -s ours" refuses to run, but my boss has 
written this cunning merge strategy "superduper" which is equally unlikely 
to yield a sensible result, "rebase -s superduper" should still refuse to 
run, no?

Now, this scenario might be too rare to take care of, but maybe it shows 
that we have a design flaw here?

Ciao,
Dscho

P.S.: Please note that I do not make a case against Thomas' patch series.  
As gitzilla once said "I cannot provide alternative patches, so that's 
that".
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help