1 |
That looks good (by inspection). |
2 |
|
3 |
My only concern is the branch deletion in the non-bare update. |
4 |
|
5 |
First of all, * should be escaped. It seems to work OK with gnu grep, |
6 |
but it would be better to be explicit. (grep(1) says that only ?+{|() |
7 |
loose their meta-meaning in basic regexps.) |
8 |
|
9 |
Second, why delete the other branches at all? I'd rather have a |
10 |
complete clone. Just the checkout ought to be enough. |
11 |
|
12 |
Otherwise, cool. |
13 |
|
14 |
-JimC |
15 |
-- |
16 |
James Cloos <cloos@×××××××.com> OpenPGP: 1024D/ED7DAEA6 |