* [PATCH] subtree : fix split output containing unrelated history
@ 2021-02-04 18:25 Taylor Jones via GitGitGadget
0 siblings, 0 replies; only message in thread
From: Taylor Jones via GitGitGadget @ 2021-02-04 18:25 UTC (permalink / raw)
To: git; +Cc: Taylor Jones, tjones629
From: tjones629 <tjones629@gmail.com>
When a subtree is deleted and re-added at the same location with the
--squash arguement the history created by "git subtree split" will
contain commits that do not affect the files in the given prefix
resulting in an incorrect history for the subtree.
This commit fixes the issue by removing lines that create cache files for
commits that did not contain the given prefix after new parent commits have
been created. The history created by "git subtree split" with this change
no longer contains unrelated commits or commits that occured before the
subtree was removed and re-added.
Signed-off-by: Taylor Jones <tjones629@gmail.com>
---
Bugfix for subtree split function
This patch fixes a bug in the subtree split command. The bug can be
reproduced by adding a squashed subtree to a repository then removing it
and squash adding it again with the same prefix. This will result in the
subtree split history containing every commit from the main repository
instead of just the commits that affect the subtree.
Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-867%2Ftjones629%2Fbugfix-subtree_removal-v1
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-867/tjones629/bugfix-subtree_removal-v1
Pull-Request: https://github.com/gitgitgadget/git/pull/867
contrib/subtree/git-subtree.sh | 4 ----
1 file changed, 4 deletions(-)
diff --git a/contrib/subtree/git-subtree.sh b/contrib/subtree/git-subtree.sh
index 868e18b9a1ab..f1b8a3b2e74c 100755
--- a/contrib/subtree/git-subtree.sh
+++ b/contrib/subtree/git-subtree.sh
@@ -667,10 +667,6 @@ process_split_commit () {
if test -z "$tree"
then
set_notree "$rev"
- if test -n "$newparents"
- then
- cache_set "$rev" "$rev"
- fi
return
fi
base-commit: 71ca53e8125e36efbda17293c50027d31681a41f
--
gitgitgadget
^ permalink raw reply related [flat|nested] only message in thread
only message in thread, other threads:[~2021-02-04 18:30 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-02-04 18:25 [PATCH] subtree : fix split output containing unrelated history Taylor Jones via GitGitGadget
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).