D2438: util: use util.shellquote() instead of repr() in date parse abort

2018-02-26 Thread yuja (Yuya Nishihara)
yuja added a comment. I think `shellquote()` is okay, but simpler workaround is to use `bytestr(date)` to drop `b`. REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D2438 To: durin42, #hg-reviewers Cc: yuja, mercurial-devel

D2438: util: use util.shellquote() instead of repr() in date parse abort

2018-02-25 Thread durin42 (Augie Fackler)
durin42 updated this revision to Diff 6099. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D2438?vs=6086=6099 REVISION DETAIL https://phab.mercurial-scm.org/D2438 AFFECTED FILES mercurial/util.py tests/test-commit.t tests/test-revset.t CHANGE

D2438: util: use util.shellquote() instead of repr() in date parse abort

2018-02-25 Thread durin42 (Augie Fackler)
durin42 updated this revision to Diff 6086. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D2438?vs=6083=6086 REVISION DETAIL https://phab.mercurial-scm.org/D2438 AFFECTED FILES mercurial/util.py tests/test-commit.t tests/test-revset.t CHANGE

D2438: util: use util.shellquote() instead of repr() in date parse abort

2018-02-25 Thread durin42 (Augie Fackler)
durin42 created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY This results in slight output changes. I'm not thrilled with seeing a literal tab instead of \t in the output, but this is, to my memory, consistent with how