Dear diary, on Sun, Jun 11, 2006 at 02:04:55PM CEST, I got a letter where Yann Dirson <ydirson@xxxxxxxxxx> said that... > Signed-off-by: Yann Dirson <ydirson@xxxxxxxxxx> > --- > > cg-admin-rewritehist | 4 ++++ > 1 files changed, 4 insertions(+), 0 deletions(-) > > diff --git a/cg-admin-rewritehist b/cg-admin-rewritehist > index 861c7f6..fe3f210 100755 > --- a/cg-admin-rewritehist > +++ b/cg-admin-rewritehist > @@ -199,6 +199,10 @@ done > git-rev-list --topo-order HEAD $startrev | tac >../revs > commits=$(cat ../revs | wc -l) > > +if [ $commits -eq 0 ]; then > + die "Found nothing to rewrite" > +fi > + > i=0 > while read commit; do > i=$((i+1)) Thanks, applied. Dear diary, on Sun, Jun 11, 2006 at 02:04:57PM CEST, I got a letter where Yann Dirson <ydirson@xxxxxxxxxx> said that... > Signed-off-by: Yann Dirson <ydirson@xxxxxxxxxx> > --- > > cg-admin-rewritehist | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/cg-admin-rewritehist b/cg-admin-rewritehist > index fe3f210..7cbdb30 100755 > --- a/cg-admin-rewritehist > +++ b/cg-admin-rewritehist > @@ -154,6 +154,8 @@ while optparse; do > if optparse -d=; then > tempdir="$OPTARG" > elif optparse -r=; then > + git-rev-parse "$OPTARG" >/dev/null || die "Unknown revision '$OPTARG'" > + git-rev-parse "$OPTARG^" >/dev/null || die "Revision '$OPTARG' does not have parents, check what you really want" > startrev="^$OPTARG^ $OPTARG $startrev" > startrevparents="$OPTARG $startrevparents" > elif optparse --env-filter=; then Thanks, I've adapted it to the current codebase. Dear diary, on Sun, Jun 11, 2006 at 02:05:00PM CEST, I got a letter where Yann Dirson <ydirson@xxxxxxxxxx> said that... > This is a fix for 95621e54cedef1c4a270af5570a72fc1331b5fcb. > > Signed-off-by: Yann Dirson <ydirson@xxxxxxxxxx> > --- > > cg-admin-rewritehist | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/cg-admin-rewritehist b/cg-admin-rewritehist > index 7cbdb30..6dd8b92 100755 > --- a/cg-admin-rewritehist > +++ b/cg-admin-rewritehist > @@ -157,7 +157,7 @@ while optparse; do > git-rev-parse "$OPTARG" >/dev/null || die "Unknown revision '$OPTARG'" > git-rev-parse "$OPTARG^" >/dev/null || die "Revision '$OPTARG' does not have parents, check what you really want" > startrev="^$OPTARG^ $OPTARG $startrev" > - startrevparents="$OPTARG $startrevparents" > + startrevparents="$OPTARG^ $startrevparents" > elif optparse --env-filter=; then > filter_env="$OPTARG" > elif optparse --tree-filter=; then Thanks; I've already applied a patch similar in spirit from someone else. -- Petr "Pasky" Baudis Stuff: http://pasky.or.cz/ A person is just about as big as the things that make them angry. - : send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html