Re: Rebase/line wrap question

[Alison Schofield <amsfield22@gmail.com>]

On Wed, Sep 28, 2016 at 12:25:08PM -0700, Elizabeth Ferdman wrote:
> Hey Alison,
> 
> I received some feedback from Greg that I want
> to fix and I have a bunch of questions. The plus side is that maybe
> some of the answers could go into the tutorial.
> 
> First he said "please wrap your changelog comments at 72 columns."
> 
> Is this something I add to my .vimrc? I've been using "git commit -m".
> Is that causing the issue? I thought git commit messages were supposed
> to be less than 50 chars anyway. I'm confused.

"commit message" is that one line message in your Subject line and it
needs to fit in 80chars.  The tutorial tells you about using git with
git log --pretty=oneline --abbrev-commit' to see style used for that
particular file/subsystem.

"changelog" is the next section in the body of the patch. I saw Greg's
message yesterday, and I went off and looked at that a bit.  I think
I always dodge that just by luck and the way I visually like to line
things up.  I googled a bit and see lots of little widgets for
linewrapping text to a certain column size.   Don't know how to
integrate any of those with git commit editing.  Maybe someone else
here does.

w.r.t. using "git commit -m"  - just curious why you are not doing
"git commint -s -v"  per the tutorial?
> 
> The other issue was that my commit didn't apply to his tree. He says
> please rebase and resend. I think I need to do:
> 
assuming you are in your staging tree
> $ git fetch origin
> $ git rebase origin/staging-testing

When this happens git will try to place any commits it finds
in your tree on top of all the stuff it grabbed from greg.
If it doesn't apply nicely, it will give you message to repair.

> Then reformat the patch?

Caveat - if the rebase failed and you had to tweak your patch,
then you may end up needing to git commit --amend -v to update
your changelog.

> $ git format-patch -o ~/mypatches/ HEAD^
> $ mutt -H <file>
> 
> So I don't need to uncommit/recommit?
No, that git rebase does that for you.

Another aside: Whenever Greg sends out a message that he's caught
up on patches, that's certainly a time to rebase.  Also, it could
be part of your daily routine, to make sure you on "top" each
day before you start working on your patches.

Hope that gets you further along,
alisons

> 
> Thanks,
> Liz
>