Re: [RFC/PATCH 2/3] pull: trivial cleanups

2013-05-16 Thread Ramkumar Ramachandra
Felipe Contreras wrote: git-pull.sh | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) This one obviously looks good. I'm not sure why you sent it along with the other two patches though. On a related note, I'm interested in fixing pull. What I have on paper so far: -

Re: [RFC/PATCH 2/3] pull: trivial cleanups

2013-05-16 Thread Felipe Contreras
On Thu, May 16, 2013 at 3:29 AM, Ramkumar Ramachandra artag...@gmail.com wrote: Felipe Contreras wrote: git-pull.sh | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) This one obviously looks good. I'm not sure why you sent it along with the other two patches though. Because this

Re: [RFC/PATCH 2/3] pull: trivial cleanups

2013-05-16 Thread Ramkumar Ramachandra
Felipe Contreras wrote: But my only concern is that there's no way to do something like: % git fetch backup 'refs/tags/*:refs/tags/*' 'refs/heads/*:refs/heads/*' [remote backup] fetch = refs/tags/*:refs/tags/* fetch = refs/heads/*:refs/heads/* What am I missing? -- To unsubscribe

Re: [RFC/PATCH 2/3] pull: trivial cleanups

2013-05-16 Thread Felipe Contreras
On Thu, May 16, 2013 at 4:34 AM, Ramkumar Ramachandra artag...@gmail.com wrote: Felipe Contreras wrote: But my only concern is that there's no way to do something like: % git fetch backup 'refs/tags/*:refs/tags/*' 'refs/heads/*:refs/heads/*' [remote backup] fetch =

Re: [RFC/PATCH 2/3] pull: trivial cleanups

2013-05-16 Thread Ramkumar Ramachandra
Felipe Contreras wrote: % git fetch origin 'refs/tags/*:refs/tags/*' 'refs/heads/*:refs/heads/*' fatal: Refusing to fetch into current branch refs/heads/fc/fast-export/cleanup of non-bare repository fatal: The remote end hung up unexpectedly That's because your HEAD is pointing to something

Re: [RFC/PATCH 2/3] pull: trivial cleanups

2013-05-16 Thread Felipe Contreras
On Thu, May 16, 2013 at 4:54 AM, Ramkumar Ramachandra artag...@gmail.com wrote: Felipe Contreras wrote: % git fetch origin 'refs/tags/*:refs/tags/*' 'refs/heads/*:refs/heads/*' fatal: Refusing to fetch into current branch refs/heads/fc/fast-export/cleanup of non-bare repository fatal: The

Re: [RFC/PATCH 2/3] pull: trivial cleanups

2013-05-16 Thread Ramkumar Ramachandra
Felipe Contreras wrote: It doesn't matter if you think it's a bad practice or not, 'git push --mirror' works, and it's possible to update a branch even if HEAD is point to it. There should be a possibility to do the same with 'git fetch'. So, introduce a configuration variable to turn off

Re: [RFC/PATCH 2/3] pull: trivial cleanups

2013-05-16 Thread Felipe Contreras
On Thu, May 16, 2013 at 5:18 AM, Ramkumar Ramachandra artag...@gmail.com wrote: Felipe Contreras wrote: It doesn't matter if you think it's a bad practice or not, 'git push --mirror' works, and it's possible to update a branch even if HEAD is point to it. There should be a possibility to do

Re: [RFC/PATCH 2/3] pull: trivial cleanups

2013-05-16 Thread Antoine Pelisse
On Thu, May 16, 2013 at 11:36 AM, Felipe Contreras felipe.contre...@gmail.com wrote: Actually trying that command: % git fetch origin 'refs/tags/*:refs/tags/*' 'refs/heads/*:refs/heads/*' fatal: Refusing to fetch into current branch refs/heads/fc/fast-export/cleanup of non-bare repository

Re: [RFC/PATCH 2/3] pull: trivial cleanups

2013-05-16 Thread Felipe Contreras
On Thu, May 16, 2013 at 6:54 AM, Antoine Pelisse apeli...@gmail.com wrote: On Thu, May 16, 2013 at 11:36 AM, Felipe Contreras felipe.contre...@gmail.com wrote: Actually trying that command: % git fetch origin 'refs/tags/*:refs/tags/*' 'refs/heads/*:refs/heads/*' fatal: Refusing to fetch into