# Remote subtree split --annotate 4 messages from 2016-01-02 to 2016-01-04. Participants: David Greene, Sebastian Schuberth, greened@obbligato.org. Thread: https://gitlist.dev/t/41109 ## David Greene, 2016-01-02 20:36 Subject: Remote subtree split --annotate Message-ID: <1451766984-4648-1-git-send-email-greened@obbligato.org> URL: https://gitlist.dev/e/1451766984-4648-1-git-send-email-greened%40obbligato.org ``` This is a fairly simple patch to remove subtree's split --annotate option. The intent of the option is to mark commits split from a subdirectory but git has other tools more suitable for editing commit messages. Given --annotate an --unannotate feature seems obvious and in fact was requested at one point. Discussion ensued and it became clear that the definition of --unannotate is not obvious given the multitude of ways one might want to rewrite a commit message. Since it is simple to run other git commit message rewriting tools on commits split from a subdirectory, it seems best to just get git-subtree out of the commit rewriting business altogether. ``` ## David Greene, 2016-01-02 20:36 Subject: [PATCH] contrib/subtree: Remove --annotate Message-ID: <1451766984-4648-2-git-send-email-greened@obbligato.org> URL: https://gitlist.dev/e/1451766984-4648-2-git-send-email-greened%40obbligato.org In-Reply-To: <1451766984-4648-1-git-send-email-greened@obbligato.org> ``` From: "David A. Greene" Remove --annotate. This obviates the need for an --unannotate command, which is both an obvious addition and difficult to define due to the numerous ways one might want to specify how to edit commit messages. git has other tools more suited to rewriting commit messages and it's easy enough to use them after a subtree split. Signed-off-by: David A. Greene --- contrib/subtree/git-subtree.sh | 6 +---- contrib/subtree/t/t7900-subtree.sh | 50 +++++++++++++++++++------------------- 2 files changed, 26 insertions(+), 30 deletions(-) diff --git a/contrib/subtree/git-subtree.sh b/contrib/subtree/git-subtree.sh index edf36f8..699c954 100755 --- a/contrib/subtree/git-subtree.sh +++ b/contrib/subtree/git-subtree.sh @@ -21,7 +21,6 @@ d show debug messages P,prefix= the name of the subdir to split out m,message= use the given message as the commit message for the merge commit options for 'split' -annotate= add a prefix to commit message of new commits b,branch= create a new branch from the split subtree ignore-joins ignore prior --rejoin commits onto= try connecting new tree to an existing one @@ -43,7 +42,6 @@ command= onto= rejoin= ignore_joins= -annotate= squash= message= prefix= @@ -87,8 +85,6 @@ while [ $# -gt 0 ]; do case "$opt" in -q) quiet=1 ;; -d) debug=1 ;; - --annotate) annotate="$1"; shift ;; - --no-annotate) annotate= ;; -b) branch="$1"; shift ;; -P) prefix="${1%/}"; shift ;; -m) message="$1"; shift ;; @@ -319,7 +315,7 @@ copy_commit() GIT_COMMITTER_NAME \ GIT_COMMITTER_EMAIL \ GIT_COMMITTER_DATE - (printf "%s" "$annotate"; cat ) | + (echo -n ""; cat ) | git commit-tree "$2" $3 # reads the rest of stdin ) || die "Can't copy commit $1" } diff --git a/contrib/subtree/t/t7900-subtree.sh b/contrib/subtree/t/t7900-subtree.sh index 751aee3..521c401 100755 --- a/contrib/subtree/t/t7900-subtree.sh +++ b/contrib/subtree/t/t7900-subtree.sh @@ -340,8 +340,8 @@ test_expect_success 'split sub dir/ with --rejoin' ' cd "$subtree_test_count" && git fetch ./"sub proj" master && git subtree merge --prefix="sub dir" FETCH_HEAD && - split_hash=$(git subtree split --prefix="sub dir" --annotate="*") && - git subtree split --prefix="sub dir" --annotate="*" --rejoin && + split_hash=$(git subtree split --prefix="sub dir") && + git subtree split --prefix="sub dir" --rejoin && check_equal "$(last_commit_message)" "Split '\''sub dir/'\'' into commit '\''$split_hash'\''" ) ' @@ -365,7 +365,7 @@ test_expect_success 'split sub dir/ with --rejoin and --message' ' cd "$subtree_test_count" && git fetch ./"sub proj" master && git subtree merge --prefix="sub dir" FETCH_HEAD && - git subtree split --prefix="sub dir" --message="Split & rejoin" --annotate="*" --rejoin && + git subtree split --prefix="sub dir" --message="Split & rejoin" --rejoin && check_equal "$(last_commit_message)" "Split & rejoin" ) ' @@ -389,8 +389,8 @@ test_expect_success 'split "sub dir"/ with --branch' ' cd "$subtree_test_count" && git fetch ./"sub proj" master && git subtree merge --prefix="sub dir" FETCH_HEAD && - split_hash=$(git subtree split --prefix="sub dir" --annotate="*") && - git subtree split --prefix="sub dir" --annotate="*" --branch subproj-br && + split_hash=$(git subtree split --prefix="sub dir") && + git subtree split --prefix="sub dir" --branch subproj-br && check_equal "$(git rev-parse subproj-br)" "$split_hash" ) ' @@ -414,8 +414,8 @@ test_expect_success 'check hash of split' ' cd "$subtree_test_count" && git fetch ./"sub proj" master && git subtree merge --prefix="sub dir" FETCH_HEAD && - split_hash=$(git subtree split --prefix="sub dir" --annotate="*") && - git subtree split --prefix="sub dir" --annotate="*" --branch subproj-br && + split_hash=$(git subtree split --prefix="sub dir") && + git subtree split --prefix="sub dir" --branch subproj-br && check_equal "$(git rev-parse subproj-br)" "$split_hash" && # Check hash of split new_hash=$(git rev-parse subproj-br^2) && @@ -500,7 +500,7 @@ test_expect_success 'make sure exactly the right set of files ends up in the sub cd "$subtree_test_count" && git fetch ./"sub proj" master && git subtree merge --prefix="sub dir" FETCH_HEAD && - git subtree split --prefix="sub dir" --annotate="*" --branch subproj-br --rejoin + git subtree split --prefix="sub dir" --branch subproj-br --rejoin ) && test_create_commit "$subtree_test_count/sub proj" sub3 && test_create_commit "$subtree_test_count" "sub dir"/main-sub3 && @@ -512,12 +512,12 @@ test_expect_success 'make sure exactly the right set of files ends up in the sub test_create_commit "$subtree_test_count/sub proj" sub4 && ( cd "$subtree_test_count" && - git subtree split --prefix="sub dir" --annotate="*" --branch subproj-br --rejoin + git subtree split --prefix="sub dir" --branch subproj-br --rejoin ) && test_create_commit "$subtree_test_count" "sub dir"/main-sub4 && ( cd "$subtree_test_count" && - git subtree split --prefix="sub dir" --annotate="*" --branch subproj-br --rejoin + git subtree split --prefix="sub dir" --branch subproj-br --rejoin ) && ( cd "$subtree_test_count/sub proj" && @@ -566,7 +566,7 @@ test_expect_success 'make sure the subproj *only* contains commits that affect t cd "$subtree_test_count" && git fetch ./"sub proj" master && git subtree merge --prefix="sub dir" FETCH_HEAD && - git subtree split --prefix="sub dir" --annotate="*" --branch subproj-br --rejoin + git subtree split --prefix="sub dir" --branch subproj-br --rejoin ) && test_create_commit "$subtree_test_count/sub proj" sub3 && test_create_commit "$subtree_test_count" "sub dir"/main-sub3 && @@ -578,12 +578,12 @@ test_expect_success 'make sure the subproj *only* contains commits that affect t test_create_commit "$subtree_test_count/sub proj" sub4 && ( cd "$subtree_test_count" && - git subtree split --prefix="sub dir" --annotate="*" --branch subproj-br --rejoin + git subtree split --prefix="sub dir" --branch subproj-br --rejoin ) && test_create_commit "$subtree_test_count" "sub dir"/main-sub4 && ( cd "$subtree_test_count" && - git subtree split --prefix="sub dir" --annotate="*" --branch subproj-br --rejoin + git subtree split --prefix="sub dir" --branch subproj-br --rejoin ) && ( cd "$subtree_test_count/sub proj" && @@ -631,7 +631,7 @@ test_expect_success 'make sure exactly the right set of files ends up in the mai cd "$subtree_test_count" && git fetch ./"sub proj" master && git subtree merge --prefix="sub dir" FETCH_HEAD && - git subtree split --prefix="sub dir" --annotate="*" --branch subproj-br --rejoin + git subtree split --prefix="sub dir" --branch subproj-br --rejoin ) && test_create_commit "$subtree_test_count/sub proj" sub3 && test_create_commit "$subtree_test_count" "sub dir"/main-sub3 && @@ -643,12 +643,12 @@ test_expect_success 'make sure exactly the right set of files ends up in the mai test_create_commit "$subtree_test_count/sub proj" sub4 && ( cd "$subtree_test_count" && - git subtree split --prefix="sub dir" --annotate="*" --branch subproj-br --rejoin + git subtree split --prefix="sub dir" --branch subproj-br --rejoin ) && test_create_commit "$subtree_test_count" "sub dir"/main-sub4 && ( cd "$subtree_test_count" && - git subtree split --prefix="sub dir" --annotate="*" --branch subproj-br --rejoin + git subtree split --prefix="sub dir" --branch subproj-br --rejoin ) && ( cd "$subtree_test_count/sub proj" && @@ -704,7 +704,7 @@ test_expect_success 'make sure each filename changed exactly once in the entire cd "$subtree_test_count" && git fetch ./"sub proj" master && git subtree merge --prefix="sub dir" FETCH_HEAD && - git subtree split --prefix="sub dir" --annotate="*" --branch subproj-br --rejoin + git subtree split --prefix="sub dir" --branch subproj-br --rejoin ) && test_create_commit "$subtree_test_count/sub proj" sub3 && test_create_commit "$subtree_test_count" "sub dir"/main-sub3 && @@ -716,12 +716,12 @@ test_expect_success 'make sure each filename changed exactly once in the entire test_create_commit "$subtree_test_count/sub proj" sub4 && ( cd "$subtree_test_count" && - git subtree split --prefix="sub dir" --annotate="*" --branch subproj-br --rejoin + git subtree split --prefix="sub dir" --branch subproj-br --rejoin ) && test_create_commit "$subtree_test_count" "sub dir"/main-sub4 && ( cd "$subtree_test_count" && - git subtree split --prefix="sub dir" --annotate="*" --branch subproj-br --rejoin + git subtree split --prefix="sub dir" --branch subproj-br --rejoin ) && ( cd "$subtree_test_count/sub proj" && @@ -785,7 +785,7 @@ test_expect_success 'make sure the --rejoin commits never make it into subproj' cd "$subtree_test_count" && git fetch ./"sub proj" master && git subtree merge --prefix="sub dir" FETCH_HEAD && - git subtree split --prefix="sub dir" --annotate="*" --branch subproj-br --rejoin + git subtree split --prefix="sub dir" --branch subproj-br --rejoin ) && test_create_commit "$subtree_test_count/sub proj" sub3 && test_create_commit "$subtree_test_count" "sub dir"/main-sub3 && @@ -797,12 +797,12 @@ test_expect_success 'make sure the --rejoin commits never make it into subproj' test_create_commit "$subtree_test_count/sub proj" sub4 && ( cd "$subtree_test_count" && - git subtree split --prefix="sub dir" --annotate="*" --branch subproj-br --rejoin + git subtree split --prefix="sub dir" --branch subproj-br --rejoin ) && test_create_commit "$subtree_test_count" "sub dir"/main-sub4 && ( cd "$subtree_test_count" && - git subtree split --prefix="sub dir" --annotate="*" --branch subproj-br --rejoin + git subtree split --prefix="sub dir" --branch subproj-br --rejoin ) && ( cd "$subtree_test_count/sub proj" && @@ -835,7 +835,7 @@ test_expect_success 'make sure no "git subtree" tagged commits make it into subp cd "$subtree_test_count" && git fetch ./"sub proj" master && git subtree merge --prefix="sub dir" FETCH_HEAD && - git subtree split --prefix="sub dir" --annotate="*" --branch subproj-br --rejoin + git subtree split --prefix="sub dir" --branch subproj-br --rejoin ) && test_create_commit "$subtree_test_count/sub proj" sub3 && test_create_commit "$subtree_test_count" "sub dir"/main-sub3 && @@ -847,12 +847,12 @@ test_expect_success 'make sure no "git subtree" tagged commits make it into subp test_create_commit "$subtree_test_count/sub proj" sub4 && ( cd "$subtree_test_count" && - git subtree split --prefix="sub dir" --annotate="*" --branch subproj-br --rejoin + git subtree split --prefix="sub dir" --branch subproj-br --rejoin ) && test_create_commit "$subtree_test_count" "sub dir"/main-sub4 && ( cd "$subtree_test_count" && - git subtree split --prefix="sub dir" --annotate="*" --branch subproj-br --rejoin + git subtree split --prefix="sub dir" --branch subproj-br --rejoin ) && ( cd "$subtree_test_count/sub proj" && -- 2.6.1 ``` ## Sebastian Schuberth, 2016-01-03 09:06 Subject: Re: [PATCH] contrib/subtree: Remove --annotate Message-ID: URL: https://gitlist.dev/e/CAHGBnuMtmUx_NwETPv%2BxU23PG8vmWm6jWpoRXc9%2B2qXok5Ak4w%40mail.gmail.com In-Reply-To: <1451766984-4648-2-git-send-email-greened@obbligato.org> ``` On Sat, Jan 2, 2016 at 9:36 PM, David Greene wrote: > commit messages. git has other tools more suited to rewriting > commit messages and it's easy enough to use them after a subtree > split. For completeness, it probably would be a good idea to name examples for such more suitable tools as part of the commit message. -- Sebastian Schuberth ``` ## greened@obbligato.org, 2016-01-04 15:53 Subject: Re: [PATCH] contrib/subtree: Remove --annotate Message-ID: URL: https://gitlist.dev/e/nng1t9xxrwy.fsf%40lnx-dag.us.cray.com In-Reply-To: ``` Sebastian Schuberth writes: > On Sat, Jan 2, 2016 at 9:36 PM, David Greene wrote: > >> commit messages. git has other tools more suited to rewriting >> commit messages and it's easy enough to use them after a subtree >> split. > > For completeness, it probably would be a good idea to name examples > for such more suitable tools as part of the commit message. Ok, that makes sense. -David ```