From: Felipe Contreras <felipe.contreras@gmail.com>
To: git@vger.kernel.org
Cc: Junio C Hamano <gitster@pobox.com>,
Martin von Zweigbergk <martinvonz@gmail.com>,
Felipe Contreras <felipe.contreras@gmail.com>
Subject: Re: [PATCH 0/5] rebase: improve the keep-empty
Date: Tue, 28 May 2013 08:33:22 -0500 [thread overview]
Message-ID: <CAMP44s2KX+Rsd+WL-=PUeF4jt9J2sp1QbkyaTL0VU+8aggaWYw@mail.gmail.com> (raw)
In-Reply-To: <1369747757-10192-1-git-send-email-felipe.contreras@gmail.com>
On Tue, May 28, 2013 at 8:29 AM, Felipe Contreras
<felipe.contreras@gmail.com> wrote:
> Hi,
>
> I've been analyzing 'git rebase' and found that the --keep-empty option
> triggers a very very different behavior. Here's a bunch of patches that make it
> behave like the 'am' does does for the most part.
>
> There's only a few minor changes, after which it might be possible to replace
> the whole 'am' mode to use cherr-pick instead.
>
> Felipe Contreras (5):
> rebase: split the cherry-pick stuff
> rebase: fix 'cherry' mode storage
> rebase: fix sequence continuation
> rebase: fix abort of cherry mode
> rebase: fix cherry-pick invocations
To see the issue, simply enable keep_mode by default, and run the tests:
Test Summary Report
-------------------
./t3401-rebase-partial.sh (Wstat: 256
Tests: 7 Failed: 4)
Failed tests: 3, 5-7
Non-zero exit status: 1
./t3403-rebase-skip.sh (Wstat: 256
Tests: 10 Failed: 3)
Failed tests: 4-5, 9
Non-zero exit status: 1
./t3406-rebase-message.sh (Wstat: 256
Tests: 6 Failed: 3)
Failed tests: 3-5
Non-zero exit status: 1
./t3400-rebase.sh (Wstat: 256
Tests: 27 Failed: 10)
Failed tests: 13-17, 22-25, 27
Non-zero exit status: 1
./t3407-rebase-abort.sh (Wstat: 256
Tests: 11 Failed: 5)
Failed tests: 2-6
Non-zero exit status: 1
./t3412-rebase-root.sh (Wstat: 256
Tests: 31 Failed: 17)
Failed tests: 4, 6-8, 10-12, 14-17, 23, 25-26, 28-29
31
Non-zero exit status: 1
./t3417-rebase-whitespace-fix.sh (Wstat: 256
Tests: 4 Failed: 4)
Failed tests: 1-4
Non-zero exit status: 1
./t3419-rebase-patch-id.sh (Wstat: 256
Tests: 9 Failed: 1)
Failed test: 4
Non-zero exit status: 1
./t3418-rebase-continue.sh (Wstat: 256
Tests: 6 Failed: 2)
Failed tests: 3, 5
Non-zero exit status: 1
./t5407-post-rewrite-hook.sh (Wstat: 256
Tests: 13 Failed: 4)
Failed tests: 4-6, 8
Non-zero exit status: 1
./t5520-pull.sh (Wstat: 256
Tests: 21 Failed: 8)
Failed tests: 12-15, 18-21
Non-zero exit status: 1
./t7512-status-help.sh (Wstat: 256
Tests: 35 Failed: 3)
Failed tests: 5-6, 28
Non-zero exit status: 1
./t9106-git-svn-commit-diff-clobber.sh (Wstat: 256
Tests: 10 Failed: 3)
Failed tests: 6, 9-10
Non-zero exit status: 1
./t3404-rebase-interactive.sh (Wstat: 256
Tests: 71 Failed: 2)
Failed tests: 50, 69
Non-zero exit status: 1
./t9137-git-svn-dcommit-clobber-series.sh (Wstat: 256
Tests: 5 Failed: 1)
Failed test: 4
Non-zero exit status: 1
./t9140-git-svn-reset.sh (Wstat: 256
Tests: 6 Failed: 1)
Failed test: 6
Non-zero exit status: 1
./t9903-bash-prompt.sh (Wstat: 256
Tests: 51 Failed: 20)
Failed tests: 27-34, 36-39, 41-42, 45-49, 51
Non-zero exit status: 1
./t9164-git-svn-dcommit-concurrent.sh (Wstat: 256
Tests: 8 Failed: 4)
Failed tests: 4-6, 8
Non-zero exit status: 1
Files=629, Tests=10036, 308 wallclock secs ( 7.60 usr 1.39 sys +
556.35 cusr 382.46 csys = 947.80 CPU)
Result: FAIL
--
Felipe Contreras
next prev parent reply other threads:[~2013-05-28 13:33 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-28 13:29 [PATCH 0/5] rebase: improve the keep-empty Felipe Contreras
2013-05-28 13:29 ` [PATCH 1/5] rebase: split the cherry-pick stuff Felipe Contreras
2013-05-28 22:24 ` Junio C Hamano
2013-05-28 13:29 ` [PATCH 2/5] rebase: fix 'cherry' mode storage Felipe Contreras
2013-05-28 22:29 ` Junio C Hamano
2013-05-29 3:03 ` Felipe Contreras
2013-05-28 13:29 ` [PATCH 3/5] rebase: fix sequence continuation Felipe Contreras
2013-05-28 22:31 ` Junio C Hamano
2013-05-29 5:53 ` Felipe Contreras
2013-05-28 13:29 ` [PATCH 4/5] rebase: fix abort of cherry mode Felipe Contreras
2013-05-28 13:29 ` [PATCH 5/5] rebase: fix cherry-pick invocations Felipe Contreras
2013-05-28 22:46 ` Junio C Hamano
2013-05-28 22:51 ` Junio C Hamano
2013-05-29 5:57 ` Felipe Contreras
2013-05-29 5:55 ` Felipe Contreras
2013-05-28 13:33 ` Felipe Contreras [this message]
2013-05-28 17:17 ` [PATCH 0/5] rebase: improve the keep-empty Martin von Zweigbergk
2013-05-29 3:01 ` Felipe Contreras
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: http://vger.kernel.org/majordomo-info.html
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAMP44s2KX+Rsd+WL-=PUeF4jt9J2sp1QbkyaTL0VU+8aggaWYw@mail.gmail.com' \
--to=felipe.contreras@gmail.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=martinvonz@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).