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

Re: Newbie: report of first experience with git-rebase.

From: Daniel Barkalow <hidden>
Date: 2016-06-15 22:43:45

On Wed, 31 Oct 2007, Steven Grimm wrote:
J. Bruce Fields wrote:
quoted
I ran into the same confusion as the original poster when starting to
use rebase, so I suspect it's common.
  
I've been using rebase just about every day for close to a year and it *still*
annoys me when it happens. Especially the "Did you forget to git add?" part of
the message. The thought that always goes through my head is, "No, Mr. Rebase,
I did NOT forget to git add. I remembered to git add, then you were too stupid
to do the right thing after that."

Just happened to me this morning, in fact: I had a quick hack in place to work
around a bug, the bug got fixed for real, and I rebased. In the process of
conflict resolution I saw that my workaround wasn't needed any more and
accepted the upstream version of that particular part of the file. Ran git-add
on it, then rebase --continue, and boom, was accused of forgetting to run
git-add.

It is a minor annoyance and nowadays I just sigh a bit and run --skip instead,
but it'd be nice if it didn't happen. I don't like having to care whether or
not I happened to change other files in a particular commit after I resolve
conflicts in one file in favor of the upstream version.
I think it's worth requiring you to say --skip in order to acknowledge 
that you won't have as many commits and you'll lose the commit message. On 
the other hand, the message should probably suggest that you might want to 
skip this commit instead of suggesting that you come up with some other 
change to include in it.

Certainly, if "git diff" returns no changes, "git add" is a bad 
suggestion, and it would be nicer to suggest something possibly correct.

	-Daniel
*This .sig left intentionally blank*
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help