From: Lucien Kong <Lucien.Kong@ensimag.imag.fr>
To: git@vger.kernel.org
Cc: Lucien Kong <Lucien.Kong@ensimag.imag.fr>,
Valentin Duperray <Valentin.Duperray@ensimag.imag.fr>,
Franck Jonas <Franck.Jonas@ensimag.imag.fr>,
Thomas Nguy <Thomas.Nguy@ensimag.imag.fr>,
Huynh Khoi Nguyen Nguyen
<Huynh-Khoi-Nguyen.Nguyen@ensimag.imag.fr>,
Matthieu Moy <Matthieu.Moy@grenoble-inp.fr>
Subject: [PATCHv3 1/2] Warnings before rebasing -i published history
Date: Mon, 11 Jun 2012 23:56:20 +0200 [thread overview]
Message-ID: <1339451781-29324-1-git-send-email-Lucien.Kong@ensimag.imag.fr> (raw)
In-Reply-To: <1339409091-28150-1-git-send-email-Lucien.Kong@ensimag.imag.fr>
"git rebase -i" can be very dangerous if used on an already published
history. This code detects that one is rewriting a commit that is an
ancestor of a remote-tracking branch, and warns the user through the
editor. This feature is controlled by a new config key
rebase.checkremoterefs.
Here is an example of the behaviour of this feature:
pick <id> commit pushed_one
# Commits above this line appear in origin/master
pick <id> commit pushed_two
pick <id> commit pushed_three
# Commits above this line appear in origin/next
pick <id> commit local
Signed-off-by: Lucien Kong <Lucien.Kong@ensimag.imag.fr>
Signed-off-by: Valentin Duperray <Valentin.Duperray@ensimag.imag.fr>
Signed-off-by: Franck Jonas <Franck.Jonas@ensimag.imag.fr>
Signed-off-by: Thomas Nguy <Thomas.Nguy@ensimag.imag.fr>
Signed-off-by: Huynh Khoi Nguyen Nguyen <Huynh-Khoi-Nguyen.Nguyen@ensimag.imag.fr>
Signed-off-by: Matthieu Moy <Matthieu.Moy@grenoble-inp.fr>
---
The remote points are now only shown once. The behavior of
git branch --contains is the same than in v2 (not optimized).
Documentation/config.txt | 5 +++++
Documentation/git-rebase.txt | 5 +++++
git-rebase--interactive.sh | 32 ++++++++++++++++++++++++++++++++
git-rebase.sh | 2 ++
t/lib-rebase.sh | 11 +++++++++++
t/t3404-rebase-interactive.sh | 27 +++++++++++++++++++++++++++
6 files changed, 82 insertions(+), 0 deletions(-)
diff --git a/Documentation/config.txt b/Documentation/config.txt
index 915cb5a..e47f6e0 100644
--- a/Documentation/config.txt
+++ b/Documentation/config.txt
@@ -1753,6 +1753,11 @@ rebase.stat::
rebase.autosquash::
If set to true enable '--autosquash' option by default.
+rebase.checkremoterefs::
+ If it is set to 'true', git rebase -i will show after each
+ commit ancestor of a remote-tracking branch the name of these
+ branches through the editor.
+
receive.autogc::
By default, git-receive-pack will run "git-gc --auto" after
receiving data from git-push and updating refs. You can stop
diff --git a/Documentation/git-rebase.txt b/Documentation/git-rebase.txt
index 147fa1a..b68a80b 100644
--- a/Documentation/git-rebase.txt
+++ b/Documentation/git-rebase.txt
@@ -208,6 +208,11 @@ rebase.stat::
rebase.autosquash::
If set to true enable '--autosquash' option by default.
+rebase.checkremoterefs::
+ If it is set to 'true', git rebase -i will show after each
+ commit ancestor of a remote-tracking branch the name of these
+ branches through the editor.
+
OPTIONS
-------
<newbase>::
diff --git a/git-rebase--interactive.sh b/git-rebase--interactive.sh
index 0c19b7c..7e4ae16 100644
--- a/git-rebase--interactive.sh
+++ b/git-rebase--interactive.sh
@@ -684,6 +684,37 @@ rearrange_squash () {
rm -f "$1.sq" "$1.rearranged"
}
+# Add the name of the branches after each pick, fixup or squash commit that
+# is an ancestor of a remote-tracking branch.
+add_remoterefs () {
+ to_print=
+ branches_name=
+ while read -r command sha1 message
+ do
+ git branch -r --contains "$sha1" >"$1.branch"
+ if test -s "$1.branch"
+ then
+ while read -r branch
+ do
+ branches_name="$branches_name $branch"
+ done <"$1.branch"
+ fi
+
+ if test "$to_print" != "$branches_name"
+ then
+ if test -n "$to_print"
+ then
+ printf '%s\n' "# Commits above this line appear in:$to_print"
+ fi
+ to_print=$branches_name
+ fi
+ branches_name=
+ printf '%s\n' "$command $sha1 $message"
+ done >"$1.published" <"$1"
+ cat "$1.published" >"$1"
+ rm -f "$1.published" "$1.branch"
+}
+
case "$action" in
continue)
# do we have anything to commit?
@@ -857,6 +888,7 @@ fi
test -s "$todo" || echo noop >> "$todo"
test -n "$autosquash" && rearrange_squash "$todo"
+test -n "$checkremoterefs" && add_remoterefs "$todo"
cat >> "$todo" << EOF
# Rebase $shortrevisions onto $shortonto
diff --git a/git-rebase.sh b/git-rebase.sh
index e616737..f8675b5 100755
--- a/git-rebase.sh
+++ b/git-rebase.sh
@@ -99,7 +99,9 @@ action=
preserve_merges=
autosquash=
keep_empty=
+checkremoterefs=
test "$(git config --bool rebase.autosquash)" = "true" && autosquash=t
+test "$(git config --bool rebase.checkremoterefs)" = "true" && checkremoterefs=t
read_basic_state () {
head_name=$(cat "$state_dir"/head-name) &&
diff --git a/t/lib-rebase.sh b/t/lib-rebase.sh
index 6ccf797..4906867 100644
--- a/t/lib-rebase.sh
+++ b/t/lib-rebase.sh
@@ -65,3 +65,14 @@ EOF
test_set_editor "$(pwd)/fake-editor.sh"
chmod a+x fake-editor.sh
}
+
+
+set_copy_editor () {
+ echo "#!$SHELL_PATH" >editor.sh
+ cat >> editor.sh <<\EOF
+cat "$1" >"$TODO_DUMP"
+EOF
+
+ test_set_editor "$(pwd)/editor.sh"
+ chmod a+x editor.sh
+}
diff --git a/t/t3404-rebase-interactive.sh b/t/t3404-rebase-interactive.sh
index 025c1c6..72934a5 100755
--- a/t/t3404-rebase-interactive.sh
+++ b/t/t3404-rebase-interactive.sh
@@ -755,4 +755,31 @@ test_expect_success 'rebase-i history with funny messages' '
test_cmp expect actual
'
+test_expect_success 'warn before rewriting published history' '
+ test_when_finished "rm -rf git.git git" &&
+ git init git.git &&
+ git clone git &&
+ (
+ cd git &&
+ git config rebase.checkremoterefs true &&
+ test_commit one_commit main.txt one_commit &&
+ test_commit two_commit main.txt two_commit &&
+ test_commit three_commit main.txt three_commit &&
+ git push --all &&
+ test_commit four_commit main.txt four_commit &&
+ set_copy_editor &&
+ TODO_DUMP=actual EDITOR=./editor.sh git rebase -i HEAD~3 &&
+ tmp=$(cat actual | sed -n 1,5p) &&
+ echo "$tmp" >actual &&
+ two_sha1=$(git rev-parse --short HEAD~2) &&
+ three_sha1=$(git rev-parse --short HEAD~1) &&
+ four_sha1=$(git rev-parse --short HEAD) &&
+ echo "pick $two_sha1 two_commit" >expected &&
+ echo "pick $three_sha1 three_commit" >>expected &&
+ echo "# Commits above this line appear in: origin/master" >>expected &&
+ echo "pick $four_sha1 four_commit" >>expected &&
+ test_cmp expected actual
+ )
+'
+
test_done
--
1.7.8
next prev parent reply other threads:[~2012-06-11 21:56 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-06-07 21:20 [PATCH] Warnings before rebasing -i published history Lucien Kong
2012-06-07 22:04 ` Matthieu Moy
2012-06-08 14:03 ` konglu
2012-06-08 14:25 ` Matthieu Moy
2012-06-08 14:57 ` Junio C Hamano
2012-06-07 22:49 ` Junio C Hamano
2012-06-08 7:32 ` konglu
2012-06-08 8:52 ` Matthieu Moy
2012-06-08 9:18 ` Tomas Carnecky
2012-06-08 9:23 ` Matthieu Moy
2012-06-08 14:55 ` Junio C Hamano
2012-06-11 10:04 ` [PATCHv2] " Lucien Kong
2012-06-11 10:55 ` Matthieu Moy
2012-06-11 11:36 ` konglu
2012-06-11 11:39 ` Matthieu Moy
2012-06-11 11:46 ` branch --contains is unbearably slow [Re: [PATCHv2] Warnings before rebasing -i published history] Thomas Rast
2012-06-11 16:16 ` Junio C Hamano
2012-06-11 22:04 ` Thomas Rast
2012-06-11 22:08 ` Thomas Rast
2012-06-11 23:04 ` Junio C Hamano
2012-06-11 21:56 ` Lucien Kong [this message]
2012-06-11 21:56 ` [PATCHv3 2/2] Warnings before amending published history Lucien Kong
2012-06-12 7:34 ` Matthieu Moy
2012-06-12 15:22 ` Junio C Hamano
2012-06-12 7:45 ` [PATCHv3 1/2] Warnings before rebasing -i " Nguy Thomas
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=1339451781-29324-1-git-send-email-Lucien.Kong@ensimag.imag.fr \
--to=lucien.kong@ensimag.imag.fr \
--cc=Franck.Jonas@ensimag.imag.fr \
--cc=Huynh-Khoi-Nguyen.Nguyen@ensimag.imag.fr \
--cc=Matthieu.Moy@grenoble-inp.fr \
--cc=Thomas.Nguy@ensimag.imag.fr \
--cc=Valentin.Duperray@ensimag.imag.fr \
--cc=git@vger.kernel.org \
/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).