Re: [PATCH/RFC 06/10] Unify the lenght of $SHORT* and the commits in the TODO list

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Junio,

Junio C Hamano schrieb am Fri 11. Apr, 17:00 (-0700):
> Jörg Sommer <joerg@xxxxxxxxxxxx> writes:
> 
> > ---
> >  git-rebase--interactive.sh |    6 +++---
> >  1 files changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/git-rebase--interactive.sh b/git-rebase--interactive.sh
> > index 1698c3e..060b40f 100755
> > --- a/git-rebase--interactive.sh
> > +++ b/git-rebase--interactive.sh
> > @@ -600,9 +600,9 @@ do
> >  			MERGES_OPTION=--no-merges
> >  		fi
> >  
> > -		SHORTUPSTREAM=$(git rev-parse --short $UPSTREAM)
> > -		SHORTHEAD=$(git rev-parse --short $HEAD)
> > -		SHORTONTO=$(git rev-parse --short $ONTO)
> > +		SHORTUPSTREAM=$(git rev-parse --short=7 $UPSTREAM)
> > +		SHORTHEAD=$(git rev-parse --short=7 $HEAD)
> > +		SHORTONTO=$(git rev-parse --short=7 $ONTO)
> 
> Lacking is a better justification as to why this is a good change

“This makes it easier to test for equality of a commit in the TODO list
and one of SHORTUPSTREAM, SHORTHEAD or SHORTONTO.”

> and 7 is a good number.

I don't know why 7 is a good number. It was there from the beginning:

% git show 1b1dce4b:git-rebase--interactive.sh G -C1 \=7
                git rev-list --no-merges --pretty=oneline --abbrev-commit \
                        --abbrev=7 --reverse $UPSTREAM..$HEAD | \
                        sed "s/^/pick /" >> "$TODO"

Bye, Jörg.
-- 
Damit das Mögliche entsteht, muß immer wieder das Unmögliche versucht
werden.                                       (Hermann Hesse)

Attachment: signature.asc
Description: Digital signature http://en.wikipedia.org/wiki/OpenPGP


[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux