[PATCH] push: respect --no-thin

2013-08-10 Thread Nguyễn Thái Ngọc Duy
Over the time the default value for --thin has been switched between on and off. As of now it's always on, even if --no-thin is given. Correct the code to respect --no-thin. Signed-off-by: Nguyễn Thái Ngọc Duy pclo...@gmail.com --- builtin/push.c | 5 ++--- 1 file changed, 2 insertions(+), 3

Re: [PATCH] push: respect --no-thin

2013-08-10 Thread Jeff King
On Sat, Aug 10, 2013 at 05:10:07PM +0700, Nguyen Thai Ngoc Duy wrote: Over the time the default value for --thin has been switched between on and off. As of now it's always on, even if --no-thin is given. Correct the code to respect --no-thin. Signed-off-by: Nguyễn Thái Ngọc Duy

Re: [PATCH] push: respect --no-thin

2013-08-10 Thread Duy Nguyen
On Sat, Aug 10, 2013 at 06:31:23AM -0400, Jeff King wrote: It might be nice to have a test, but I am not sure of a good way to check whether a push was thin. I guess we can grep the output of GIT_TRACE, though it feels a bit hacky. In theory, if we have a way to tell receive-pack not to pass