|
0
|
1 |
#!/bin/sh
|
|
|
2 |
#
|
|
|
3 |
# Copyright (c) 2006 Junio C Hamano
|
|
|
4 |
#
|
|
|
5 |
|
|
|
6 |
publish=next
|
|
|
7 |
basebranch="$1"
|
|
|
8 |
if test "$#" = 2
|
|
|
9 |
then
|
|
|
10 |
topic="refs/heads/$2"
|
|
|
11 |
else
|
|
|
12 |
topic=`git symbolic-ref HEAD`
|
|
|
13 |
fi
|
|
|
14 |
|
|
|
15 |
case "$basebranch,$topic" in
|
|
|
16 |
master,refs/heads/??/*)
|
|
|
17 |
;;
|
|
|
18 |
*)
|
|
|
19 |
exit 0 ;# we do not interrupt others.
|
|
|
20 |
;;
|
|
|
21 |
esac
|
|
|
22 |
|
|
|
23 |
# Now we are dealing with a topic branch being rebased
|
|
|
24 |
# on top of master. Is it OK to rebase it?
|
|
|
25 |
|
|
|
26 |
# Is topic fully merged to master?
|
|
|
27 |
not_in_master=`git-rev-list --pretty=oneline ^master "$topic"`
|
|
|
28 |
if test -z "$not_in_master"
|
|
|
29 |
then
|
|
|
30 |
echo >&2 "$topic is fully merged to master; better remove it."
|
|
|
31 |
exit 1 ;# we could allow it, but there is no point.
|
|
|
32 |
fi
|
|
|
33 |
|
|
|
34 |
# Is topic ever merged to next? If so you should not be rebasing it.
|
|
|
35 |
only_next_1=`git-rev-list ^master "^$topic" ${publish} | sort`
|
|
|
36 |
only_next_2=`git-rev-list ^master ${publish} | sort`
|
|
|
37 |
if test "$only_next_1" = "$only_next_2"
|
|
|
38 |
then
|
|
|
39 |
not_in_topic=`git-rev-list "^$topic" master`
|
|
|
40 |
if test -z "$not_in_topic"
|
|
|
41 |
then
|
|
|
42 |
echo >&2 "$topic is already up-to-date with master"
|
|
|
43 |
exit 1 ;# we could allow it, but there is no point.
|
|
|
44 |
else
|
|
|
45 |
exit 0
|
|
|
46 |
fi
|
|
|
47 |
else
|
|
|
48 |
not_in_next=`git-rev-list --pretty=oneline ^${publish} "$topic"`
|
|
|
49 |
perl -e '
|
|
|
50 |
my $topic = $ARGV[0];
|
|
|
51 |
my $msg = "* $topic has commits already merged to public branch:\n";
|
|
|
52 |
my (%not_in_next) = map {
|
|
|
53 |
/^([0-9a-f]+) /;
|
|
|
54 |
($1 => 1);
|
|
|
55 |
} split(/\n/, $ARGV[1]);
|
|
|
56 |
for my $elem (map {
|
|
|
57 |
/^([0-9a-f]+) (.*)$/;
|
|
|
58 |
[$1 => $2];
|
|
|
59 |
} split(/\n/, $ARGV[2])) {
|
|
|
60 |
if (!exists $not_in_next{$elem->[0]}) {
|
|
|
61 |
if ($msg) {
|
|
|
62 |
print STDERR $msg;
|
|
|
63 |
undef $msg;
|
|
|
64 |
}
|
|
|
65 |
print STDERR " $elem->[1]\n";
|
|
|
66 |
}
|
|
|
67 |
}
|
|
|
68 |
' "$topic" "$not_in_next" "$not_in_master"
|
|
|
69 |
exit 1
|
|
|
70 |
fi
|
|
|
71 |
|
|
|
72 |
exit 0
|
|
|
73 |
|
|
|
74 |
################################################################
|
|
|
75 |
|
|
|
76 |
This sample hook safeguards topic branches that have been
|
|
|
77 |
published from being rewound.
|
|
|
78 |
|
|
|
79 |
The workflow assumed here is:
|
|
|
80 |
|
|
|
81 |
* Once a topic branch forks from "master", "master" is never
|
|
|
82 |
merged into it again (either directly or indirectly).
|
|
|
83 |
|
|
|
84 |
* Once a topic branch is fully cooked and merged into "master",
|
|
|
85 |
it is deleted. If you need to build on top of it to correct
|
|
|
86 |
earlier mistakes, a new topic branch is created by forking at
|
|
|
87 |
the tip of the "master". This is not strictly necessary, but
|
|
|
88 |
it makes it easier to keep your history simple.
|
|
|
89 |
|
|
|
90 |
* Whenever you need to test or publish your changes to topic
|
|
|
91 |
branches, merge them into "next" branch.
|
|
|
92 |
|
|
|
93 |
The script, being an example, hardcodes the publish branch name
|
|
|
94 |
to be "next", but it is trivial to make it configurable via
|
|
|
95 |
$GIT_DIR/config mechanism.
|
|
|
96 |
|
|
|
97 |
With this workflow, you would want to know:
|
|
|
98 |
|
|
|
99 |
(1) ... if a topic branch has ever been merged to "next". Young
|
|
|
100 |
topic branches can have stupid mistakes you would rather
|
|
|
101 |
clean up before publishing, and things that have not been
|
|
|
102 |
merged into other branches can be easily rebased without
|
|
|
103 |
affecting other people. But once it is published, you would
|
|
|
104 |
not want to rewind it.
|
|
|
105 |
|
|
|
106 |
(2) ... if a topic branch has been fully merged to "master".
|
|
|
107 |
Then you can delete it. More importantly, you should not
|
|
|
108 |
build on top of it -- other people may already want to
|
|
|
109 |
change things related to the topic as patches against your
|
|
|
110 |
"master", so if you need further changes, it is better to
|
|
|
111 |
fork the topic (perhaps with the same name) afresh from the
|
|
|
112 |
tip of "master".
|
|
|
113 |
|
|
|
114 |
Let's look at this example:
|
|
|
115 |
|
|
|
116 |
o---o---o---o---o---o---o---o---o---o "next"
|
|
|
117 |
/ / / /
|
|
|
118 |
/ a---a---b A / /
|
|
|
119 |
/ / / /
|
|
|
120 |
/ / c---c---c---c B /
|
|
|
121 |
/ / / \ /
|
|
|
122 |
/ / / b---b C \ /
|
|
|
123 |
/ / / / \ /
|
|
|
124 |
---o---o---o---o---o---o---o---o---o---o---o "master"
|
|
|
125 |
|
|
|
126 |
|
|
|
127 |
A, B and C are topic branches.
|
|
|
128 |
|
|
|
129 |
* A has one fix since it was merged up to "next".
|
|
|
130 |
|
|
|
131 |
* B has finished. It has been fully merged up to "master" and "next",
|
|
|
132 |
and is ready to be deleted.
|
|
|
133 |
|
|
|
134 |
* C has not merged to "next" at all.
|
|
|
135 |
|
|
|
136 |
We would want to allow C to be rebased, refuse A, and encourage
|
|
|
137 |
B to be deleted.
|
|
|
138 |
|
|
|
139 |
To compute (1):
|
|
|
140 |
|
|
|
141 |
git-rev-list ^master ^topic next
|
|
|
142 |
git-rev-list ^master next
|
|
|
143 |
|
|
|
144 |
if these match, topic has not merged in next at all.
|
|
|
145 |
|
|
|
146 |
To compute (2):
|
|
|
147 |
|
|
|
148 |
git-rev-list master..topic
|
|
|
149 |
|
|
|
150 |
if this is empty, it is fully merged to "master".
|