Re: [PATCH v5 00/36] Massive improvents to rebase and cherry-pick
On Mon, Jun 10, 2013 at 8:09 PM, Phil Hord wrote: > On Mon, Jun 10, 2013 at 7:43 PM, Felipe Contreras > wrote: >> On Mon, Jun 10, 2013 at 5:55 PM, Phil Hord wrote: >>> On Sun, Jun 9, 2013 at 3:37 PM, Felipe Contreras >>> wrote: On Sun, Jun 9, 2013 at 2:24 PM, Felipe Contreras wrote: > Same as before, but: Also, remove the patches from Martin von Zweigbergk, because apparently some people have trouble understanding that they were not part of this series. >>> >>> Please try not to sound disgruntled. This attitude is toxic. You have >>> turned this change into a complaint: that "some people have trouble >>> understanding" which shows a genuine lack of understanding and >>> compassion on your part. Instead you can phrase your change notes >>> more helpfully if you make changes only when you yourself actually >>> believe the change should be made. If you cannot do this, perhaps you >>> can pretend. >> >> That would be dishonest. Moreover, there wasn't a good reason to >> remove these patches, I made it clear I added those patches only to >> make sure the real patches of this series worked correctly. Also, I >> clarified that to Thomas Rast[1], only to receive a totally >> unconstructive comment[2]. >> >> Why don't you ask Thomas Rast to be more constructive[2]? >> >> Then Johan Herland uses that as an example of a constructive >> comment[3]. Why don't you correct Johan Herland? > > I do not see what their comments have to do with your attitude. My attitude is fine. I sent a lot of patches, and I made clear that some of them were meant only to test the rest. And I clarified that twice. There's nothing wrong with that. > Aren't your own man with cogent self-will and personal responsibility? > Why should I also have to consider these other emails which I have > not bothered to read yet? Don't be that girlfriend that brings the times you haven't picked up the towel properly when talking about something completely and totally different. When talking about the attitude in *this* patch series, limit yourself to *this* patch series. >> No, you pick the easy target: me. > > You seem to have mistaken me for someone else. Moreover, you seem to > have mistaken you for someone else. You are the least easy target I > know of on this list. > Everyone else seems open to community standards. And yet you try to correct me, who did nothing wrong. And ignore the transgressions of the other people, whom I already demonstrated actually *did* do something wrong. How convenient of you to not mention my arguments *at all*. >> I already dd more than my fair share by carrying these 36 patches >> through several iterations, yet you ask *more* of me. Why don't you >> ask more of the people that just hit reply on their MUA? >> >> Thomas' task was easy; he simply had to say "Oh, these aren't meant to >> be applied, got it." >> >> [1] http://article.gmane.org/gmane.comp.version-control.git/227039 >> [2] http://article.gmane.org/gmane.comp.version-control.git/227040 >> [3] http://article.gmane.org/gmane.comp.version-control.git/227102 > > I did not comment on their posts because they did not catch my eye. > Rebase and cherry-pick improvements are interesting to me, so I read > your post. I will try not to make this mistake again. Yes, because my patches are so obviously wrong. If you were a truly productive member of this community, you would ignore all the bullshit, take the patches, fix whatever is technically wrong with them (nothing), and resend them. But no, that would be way too productive. -- Felipe Contreras -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v5 00/36] Massive improvents to rebase and cherry-pick
On Mon, Jun 10, 2013 at 7:43 PM, Felipe Contreras wrote: > On Mon, Jun 10, 2013 at 5:55 PM, Phil Hord wrote: >> On Sun, Jun 9, 2013 at 3:37 PM, Felipe Contreras >> wrote: >>> >>> On Sun, Jun 9, 2013 at 2:24 PM, Felipe Contreras >>> wrote: >>> > Same as before, but: >>> >>> Also, remove the patches from Martin von Zweigbergk, because >>> apparently some people have trouble understanding that they were not >>> part of this series. >> >> Please try not to sound disgruntled. This attitude is toxic. You have >> turned this change into a complaint: that "some people have trouble >> understanding" which shows a genuine lack of understanding and >> compassion on your part. Instead you can phrase your change notes >> more helpfully if you make changes only when you yourself actually >> believe the change should be made. If you cannot do this, perhaps you >> can pretend. > > That would be dishonest. Moreover, there wasn't a good reason to > remove these patches, I made it clear I added those patches only to > make sure the real patches of this series worked correctly. Also, I > clarified that to Thomas Rast[1], only to receive a totally > unconstructive comment[2]. > > Why don't you ask Thomas Rast to be more constructive[2]? > > Then Johan Herland uses that as an example of a constructive > comment[3]. Why don't you correct Johan Herland? I do not see what their comments have to do with your attitude. Aren't your own man with cogent self-will and personal responsibility? Why should I also have to consider these other emails which I have not bothered to read yet? > No, you pick the easy target: me. You seem to have mistaken me for someone else. Moreover, you seem to have mistaken you for someone else. You are the least easy target I know of on this list. Everyone else seems open to community standards. > I already dd more than my fair share by carrying these 36 patches > through several iterations, yet you ask *more* of me. Why don't you > ask more of the people that just hit reply on their MUA? > > Thomas' task was easy; he simply had to say "Oh, these aren't meant to > be applied, got it." > > [1] http://article.gmane.org/gmane.comp.version-control.git/227039 > [2] http://article.gmane.org/gmane.comp.version-control.git/227040 > [3] http://article.gmane.org/gmane.comp.version-control.git/227102 I did not comment on their posts because they did not catch my eye. Rebase and cherry-pick improvements are interesting to me, so I read your post. I will try not to make this mistake again. Phil -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v5 00/36] Massive improvents to rebase and cherry-pick
On Mon, Jun 10, 2013 at 5:55 PM, Phil Hord wrote: > On Sun, Jun 9, 2013 at 3:37 PM, Felipe Contreras > wrote: >> >> On Sun, Jun 9, 2013 at 2:24 PM, Felipe Contreras >> wrote: >> > Same as before, but: >> >> Also, remove the patches from Martin von Zweigbergk, because >> apparently some people have trouble understanding that they were not >> part of this series. > > Please try not to sound disgruntled. This attitude is toxic. You have > turned this change into a complaint: that "some people have trouble > understanding" which shows a genuine lack of understanding and > compassion on your part. Instead you can phrase your change notes > more helpfully if you make changes only when you yourself actually > believe the change should be made. If you cannot do this, perhaps you > can pretend. That would be dishonest. Moreover, there wasn't a good reason to remove these patches, I made it clear I added those patches only to make sure the real patches of this series worked correctly. Also, I clarified that to Thomas Rast[1], only to receive a totally unconstructive comment[2]. Why don't you ask Thomas Rast to be more constructive[2]? Then Johan Herland uses that as an example of a constructive comment[3]. Why don't you correct Johan Herland? No, you pick the easy target: me. I already dd more than my fair share by carrying these 36 patches through several iterations, yet you ask *more* of me. Why don't you ask more of the people that just hit reply on their MUA? Thomas' task was easy; he simply had to say "Oh, these aren't meant to be applied, got it." [1] http://article.gmane.org/gmane.comp.version-control.git/227039 [2] http://article.gmane.org/gmane.comp.version-control.git/227040 [3] http://article.gmane.org/gmane.comp.version-control.git/227102 -- Felipe Contreras -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v5 00/36] Massive improvents to rebase and cherry-pick
On Sun, Jun 9, 2013 at 3:37 PM, Felipe Contreras wrote: > > On Sun, Jun 9, 2013 at 2:24 PM, Felipe Contreras > wrote: > > Same as before, but: > > Also, remove the patches from Martin von Zweigbergk, because > apparently some people have trouble understanding that they were not > part of this series. Please try not to sound disgruntled. This attitude is toxic. You have turned this change into a complaint: that "some people have trouble understanding" which shows a genuine lack of understanding and compassion on your part. Instead you can phrase your change notes more helpfully if you make changes only when you yourself actually believe the change should be made. If you cannot do this, perhaps you can pretend. Also, remove the patches from Martin von Zweigbergk, which are not a part of this series. Or even this: Also, remove the patches from Martin von Zweigbergk to avoid confusing reviewers. Thanks, Phil -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v5 00/36] Massive improvents to rebase and cherry-pick
On Sun, Jun 9, 2013 at 2:24 PM, Felipe Contreras wrote: > Same as before, but: Also, remove the patches from Martin von Zweigbergk, because apparently some people have trouble understanding that they were not part of this series. -- Felipe Contreras -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH v5 00/36] Massive improvents to rebase and cherry-pick
Same as before, but: diff --git a/Documentation/config.txt b/Documentation/config.txt index 7fd4035..d72d2d2 100644 --- a/Documentation/config.txt +++ b/Documentation/config.txt @@ -1679,11 +1679,10 @@ GIT_NOTES_REF) is also implicitly added to the list of refs to be displayed. notes.rewrite.:: - When rewriting commits with (currently `amend` or - `rebase`) and this variable is set to `true`, Git - automatically copies your notes from the original to the - rewritten commit. Defaults to `true`, but see - "notes.rewriteRef" below. + When rewriting commits with (currently `amend`, `rebase`, or + `cherry-pick`) and this variable is set to `true`, Git automatically + copies your notes from the original to the rewritten commit. Defaults + to `true`, but see "notes.rewriteRef" below. notes.rewriteMode:: When copying notes during a rewrite (see the diff --git a/Documentation/githooks.txt b/Documentation/githooks.txt index d48bf4d..8cfa13b 100644 --- a/Documentation/githooks.txt +++ b/Documentation/githooks.txt @@ -352,10 +352,10 @@ post-rewrite This hook is invoked by commands that rewrite commits (`git commit ---amend`, 'git-rebase'; currently 'git-filter-branch' does 'not' call -it!). Its first argument denotes the command it was invoked by: -currently one of `amend` or `rebase`. Further command-dependent -arguments may be passed in the future. +--amend`, `git rebase`, `git cherry-pick`; currently `git filter-branch` does +'not' call it!). Its first argument denotes the command it was invoked by +(e.g. `rebase`). Further command-dependent arguments may be passed in the +future. The hook receives a list of the rewritten commits on stdin, in the format diff --git a/builtin/sequencer.c b/builtin/sequencer.c index ddd369f..f805883 100644 --- a/builtin/sequencer.c +++ b/builtin/sequencer.c @@ -33,8 +33,8 @@ static void finish(struct replay_opts *opts) if (!*name) return; - run_rewrite_hook(&rewritten, name); copy_rewrite_notes(&rewritten, name); + run_rewrite_hook(&rewritten, name); } static void remove_sequencer_state(void) Felipe Contreras (36): build: generate and clean test scripts build: do not install git-remote-testgit build: trivial cleanup build: add builtin lib log-tree: remove dependency from sequencer Move sequencer to builtin unpack-trees: plug a memory leak read-cache: plug a few leaks sequencer: remove useless indentation sequencer: trivial fix cherry-pick: don't barf when there's nothing to do cherry-pick: add --skip-empty option revert/cherry-pick: add --quiet option revert/cherry-pick: add --skip option builtin: add rewrite helper cherry-pick: store rewritten commits cherry-pick: don't store skipped commit builtin: move run_rewrite_hook() to rewrite.c builtin: add copy_rewrite_notes() cherry-pick: copy notes and run hooks cherry-pick: add --action-name option cherry-pick: remember rerere-autoupdate rebase: split the cherry-pick stuff rebase: cherry-pick: fix mode storage rebase: cherry-pick: fix sequence continuation rebase: cherry-pick: fix abort of cherry mode rebase: cherry-pick: fix command invocations rebase: cherry-pick: fix status messages rebase: cherry-pick: automatically commit stage rebase: cherry-pick: set correct action-name rebase: trivial cleanup rebase: use 'cherrypick' mode instead of 'am' rebase: cherry-pick: fix for shell prompt rebase: cherry-pick: add merge options rebase: remove merge mode rebase: cherry-pick: add copyright .gitignore | 1 + Documentation/config.txt | 9 +- Documentation/git-cherry-pick.txt | 10 +- Documentation/git-revert.txt | 7 +- Documentation/githooks.txt | 8 +- Documentation/sequencer.txt| 3 + Makefile | 31 ++-- builtin/commit.c | 46 +- builtin/revert.c | 17 +++ builtin/rewrite.c | 124 builtin/rewrite.h | 20 +++ sequencer.c => builtin/sequencer.c | 263 + sequencer.h => builtin/sequencer.h | 12 +- contrib/completion/git-prompt.sh | 4 +- git-rebase--am.sh | 12 +- git-rebase--cherrypick.sh | 72 + git-rebase--interactive.sh | 4 +- git-rebase--merge.sh | 151 --- git-rebase.sh | 16 +- log-tree.c | 161 +++- log-tree.h | 3 + read-cache.c | 4 + t/t3406-rebase-message.sh | 15 -- t/t3407-rebase-abort.sh| 2 +- t/t3420-rebase-autostash.sh| 2 +- t/t3508-cherry-pick-many-commits.sh| 1