Re: [PATCH v5 12/10] remote-bzr: support the new 'force' option

2013-11-11 Thread Felipe Contreras
On Mon, Nov 11, 2013 at 12:12 PM, Richard Hansen wrote: > On 2013-11-11 06:51, Felipe Contreras wrote: >> def do_option(parser): >> global force >> _, key, value = parser.line.split(' ') > > I'm surprised you prefer this over 'key, val = parser[1:3]' or even > '_, key, val = parse

Re: [PATCH v5 12/10] remote-bzr: support the new 'force' option

2013-11-11 Thread Richard Hansen
On 2013-11-11 06:51, Felipe Contreras wrote: > Richard Hansen wrote: >> Signed-off-by: Richard Hansen >> --- >> contrib/remote-helpers/git-remote-bzr | 34 >> +- >> contrib/remote-helpers/test-bzr.sh| 22 +- >> 2 files changed, 54 insertion

RE: [PATCH v5 12/10] remote-bzr: support the new 'force' option

2013-11-11 Thread Felipe Contreras
Richard Hansen wrote: > Signed-off-by: Richard Hansen > --- > contrib/remote-helpers/git-remote-bzr | 34 +- > contrib/remote-helpers/test-bzr.sh| 22 +- > 2 files changed, 54 insertions(+), 2 deletions(-) > > diff --git a/contrib/remote-he