[TopGit PATCH] tg-push: don't accept non-tgish branches to push

[TopGit PATCH] tg-push: don't accept non-tgish branches to push

[Bert Wesarg]

tg push dies if you give it a non-tgish branch:

$ tg push -r push-test master
fatal: Not a valid object name refs/top-bases/master
fatal: ambiguous argument '^{tree}': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions
fatal: Not a valid object name master:.topdeps

Fix this.

Signed-off-by: Bert Wesarg <bert.wesarg@googlemail.com>

---
 tg-push.sh |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/tg-push.sh b/tg-push.sh
index c813927..1d7a8a7 100644
--- a/tg-push.sh
+++ b/tg-push.sh
@@ -42,7 +42,8 @@ if [ -z "$branches" ]; then
 fi
 
 for name in $branches; do
-	ref_exists "$name" || die "detached HEAD? Can't push $name"
+	ref_exists "refs/top-bases/$name" ||
+		die "not a TopGit-controlled branch: $name"
 done
 
 _listfile="$(mktemp -t tg-push-listfile.XXXXXX)"
-- 
tg: (8ec0303..) bw/push-fixes-2 (depends on: bw/push-fixes)

Re: [TopGit PATCH] tg-push: don't accept non-tgish branches to push

[Uwe Kleine-König]

Hi Bert,

On Tue, May 26, 2009 at 11:46:22PM +0200, Bert Wesarg wrote:
> tg push dies if you give it a non-tgish branch:
> 
> $ tg push -r push-test master
> fatal: Not a valid object name refs/top-bases/master
> fatal: ambiguous argument '^{tree}': unknown revision or path not in the working tree.
> Use '--' to separate paths from revisions
> fatal: Not a valid object name master:.topdeps
> 
> Fix this.
Why not just make it work instead of dying?

Best regards
Uwe

-- 
Pengutronix e.K.                              | Uwe Kleine-König            |
Industrial Linux Solutions                    | http://www.pengutronix.de/  |