Re: [PATCH v2] git-p4: Fix git-p4.mapUser on Windows
Luke Diamand writes: > On 27 January 2017 at 17:33, Junio C Hamano wrote: >> >> Luke, Lars, this version seems to be in line with the conclusion of >> your earlier reviews, e.g. >> >> >> >> Even though it looks OK to my eyes, I'll wait for Acks or further >> refinement suggestions from either of you two before acting on this >> patch. > > It looks good to me. The tests all pass, and the change looks correct. Thanks, queued.
Re: [PATCH v2] git-p4: Fix git-p4.mapUser on Windows
On 27 January 2017 at 17:33, Junio C Hamano wrote: > George Vanburgh writes: > >> From: George Vanburgh >> >> When running git-p4 on Windows, with multiple git-p4.mapUser entries in >> git config - no user mappings are applied to the generated repository. >> ... >> Using splitlines solves this issue, by splitting config on all >> typical delimiters ('\n', '\r\n' etc.) > > Luke, Lars, this version seems to be in line with the conclusion of > your earlier reviews, e.g. > > > > Even though it looks OK to my eyes, I'll wait for Acks or further > refinement suggestions from either of you two before acting on this > patch. It looks good to me. The tests all pass, and the change looks correct. Ack. Luke > > Thanks. > >> Signed-off-by: George Vanburgh >> --- >> git-p4.py | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/git-p4.py b/git-p4.py >> index f427bf6..b66f68b 100755 >> --- a/git-p4.py >> +++ b/git-p4.py >> @@ -656,7 +656,7 @@ def gitConfigInt(key): >> def gitConfigList(key): >> if not _gitConfig.has_key(key): >> s = read_pipe(["git", "config", "--get-all", key], >> ignore_error=True) >> -_gitConfig[key] = s.strip().split(os.linesep) >> +_gitConfig[key] = s.strip().splitlines() >> if _gitConfig[key] == ['']: >> _gitConfig[key] = [] >> return _gitConfig[key] >> >> -- >> https://github.com/git/git/pull/319
Re: [PATCH v2] git-p4: Fix git-p4.mapUser on Windows
George Vanburgh writes: > From: George Vanburgh > > When running git-p4 on Windows, with multiple git-p4.mapUser entries in > git config - no user mappings are applied to the generated repository. > ... > Using splitlines solves this issue, by splitting config on all > typical delimiters ('\n', '\r\n' etc.) Luke, Lars, this version seems to be in line with the conclusion of your earlier reviews, e.g. Even though it looks OK to my eyes, I'll wait for Acks or further refinement suggestions from either of you two before acting on this patch. Thanks. > Signed-off-by: George Vanburgh > --- > git-p4.py | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/git-p4.py b/git-p4.py > index f427bf6..b66f68b 100755 > --- a/git-p4.py > +++ b/git-p4.py > @@ -656,7 +656,7 @@ def gitConfigInt(key): > def gitConfigList(key): > if not _gitConfig.has_key(key): > s = read_pipe(["git", "config", "--get-all", key], ignore_error=True) > -_gitConfig[key] = s.strip().split(os.linesep) > +_gitConfig[key] = s.strip().splitlines() > if _gitConfig[key] == ['']: > _gitConfig[key] = [] > return _gitConfig[key] > > -- > https://github.com/git/git/pull/319