Re: [PATCH] Fix git-p4 submit in non --prepare-p4-only mode

2014-06-11 Thread Pete Wyckoff
frrr...@gmail.com wrote on Wed, 11 Jun 2014 14:06 +0100: > On Tue, Jun 10, 2014 at 06:39:58PM -0400, Pete Wyckoff wrote: > > frrr...@gmail.com wrote on Tue, 10 Jun 2014 13:14 +0100: > > > b4073bb387ef303c9ac3c044f46d6a8ae6e190f0 broke git p4 submit, here > > > is a proper fix, including proper hand

[PATCH] Fix git-p4 submit in non --prepare-p4-only mode

2014-06-11 Thread Maxime Coste
b4073bb387ef303c9ac3c044f46d6a8ae6e190f0 broke git p4 submit, here is a proper fix, including proper handling for windows end of lines. Signed-off-by: Maxime Coste Acked-by: Pete Wyckoff --- git-p4.py | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/git-p4.py b/

Re: [PATCH] Fix git-p4 submit in non --prepare-p4-only mode

2014-06-11 Thread Maxime Coste
On Tue, Jun 10, 2014 at 06:39:58PM -0400, Pete Wyckoff wrote: > frrr...@gmail.com wrote on Tue, 10 Jun 2014 13:14 +0100: > > b4073bb387ef303c9ac3c044f46d6a8ae6e190f0 broke git p4 submit, here > > is a proper fix, including proper handling for windows end of lines. > > I guess we don't have test co

Re: [PATCH] Fix git-p4 submit in non --prepare-p4-only mode

2014-06-10 Thread Pete Wyckoff
frrr...@gmail.com wrote on Tue, 10 Jun 2014 13:14 +0100: > b4073bb387ef303c9ac3c044f46d6a8ae6e190f0 broke git p4 submit, here > is a proper fix, including proper handling for windows end of lines. I guess we don't have test coverage for these cases? Is this something that should get put into a ma

[PATCH] Fix git-p4 submit in non --prepare-p4-only mode

2014-06-10 Thread Maxime Coste
b4073bb387ef303c9ac3c044f46d6a8ae6e190f0 broke git p4 submit, here is a proper fix, including proper handling for windows end of lines. Signed-off-by: Maxime Coste --- git-p4.py | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/git-p4.py b/git-p4.py index 7bb0f73..