Re: [PATCH v2 2/3] push: Add support for pre-push hooks

2013-01-14 Thread Junio C Hamano
Junio C Hamano writes: > At least the attached patch is necessary. Sorry, but the last hunk (see below) is not. It breaks the hook. > In the longer term, we may want to discuss what should happen when > the hook exited without even reading what we fed. My gut feeling is > that we can still tr

Re: [PATCH v2 2/3] push: Add support for pre-push hooks

2013-01-14 Thread Junio C Hamano
Aaron Schrab writes: > t/t5571-pre-push-hook.sh | 129 > + > diff --git a/t/t5571-pre-push-hook.sh b/t/t5571-pre-push-hook.sh > new file mode 100755 > index 000..d68fed7 > --- /dev/null > +++ b/t/t5571-pre-push-hook.sh > @@ -0,0 +1,129 @@ > +#!/b

Re: [PATCH v2 2/3] push: Add support for pre-push hooks

2013-01-14 Thread Junio C Hamano
Aaron Schrab writes: > Add support for a pre-push hook which can be used to determine if the > set of refs to be pushed is suitable for the target repository. The > hook is run with two arguments specifying the name and location of the > destination repository. > > Information about what is to b

[PATCH v2 2/3] push: Add support for pre-push hooks

2013-01-12 Thread Aaron Schrab
Add support for a pre-push hook which can be used to determine if the set of refs to be pushed is suitable for the target repository. The hook is run with two arguments specifying the name and location of the destination repository. Information about what is to be pushed is provided by sending li