Re: [PATCH] mdiff: split on unicode character boundaries when shortening function name

2018-02-22 Thread Denis Laxalde
Josef 'Jeff' Sipek wrote: > # HG changeset patch > # User Josef 'Jeff' Sipek > # Date 1519251311 18000 > # Wed Feb 21 17:15:11 2018 -0500 > # Node ID b99df94fdd4813e0ce538a8caa682802da4a6cb2 > # Parent 106872aa15af9919220705ed72c78459774e1575 > mdiff: split on unicode

D2377: util: add a file object proxy that can read at most N bytes

2018-02-22 Thread lothiraldan (Boris Feld)
lothiraldan accepted this revision. lothiraldan added a comment. LGTM REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D2377 To: indygreg, #hg-reviewers, lothiraldan Cc: lothiraldan, mercurial-devel ___ Mercurial-devel

Re: [PATCH STABLE] graphlog: document what "_" and "*" mean

2018-02-22 Thread Feld Boris
LGTM On 22/02/2018 08:34, Anton Shestakov wrote: # HG changeset patch # User Anton Shestakov # Date 1519283924 -28800 # Thu Feb 22 15:18:44 2018 +0800 # Branch stable # Node ID a1b8de1983b1c5421f7c7195e01b20df5306500a # Parent c19e66dacaa184feba31136c18a369ba995ddfe4

[PATCH V2] patches: release the GIL while applying the patch

2018-02-22 Thread Boris Feld
# HG changeset patch # User Boris Feld # Date 1517839431 -3600 # Mon Feb 05 15:03:51 2018 +0100 # Node ID 585005c9c4901f4f94847e8637fbc58cc5b29c56 # Parent 0c34cb461a1ea5d3f8e1300e0b8bc16ed8fa8802 # EXP-Topic parallel-patching # Available At

D2090: fancyopts: add support for custom multi-arg opts in fancyopts.py

2018-02-22 Thread yuja (Yuya Nishihara)
yuja added inline comments. INLINE COMMENTS > fancyopts.py:258 > +elif isinstance(default, list): > +return _listopt(default[:]) > +elif type(default) is type(1): Perhaps it's safer to make `defaultvalue()` a function returning a copy of default, instead of passing a copy to

mercurial@36352: 15 new changesets

2018-02-22 Thread Mercurial Commits
15 new changesets in mercurial: https://www.mercurial-scm.org/repo/hg/rev/ddd9474d2e08 changeset: 36338:ddd9474d2e08 user:Martin von Zweigbergk date:Tue Feb 06 08:48:05 2018 -0800 summary: walkrepos: don't reimplement any()

Re: [PATCH STABLE] graphlog: document what "_" and "*" mean

2018-02-22 Thread Yuya Nishihara
On Thu, 22 Feb 2018 15:34:03 +0800, Anton Shestakov wrote: > # HG changeset patch > # User Anton Shestakov > # Date 1519283924 -28800 > # Thu Feb 22 15:18:44 2018 +0800 > # Branch stable > # Node ID a1b8de1983b1c5421f7c7195e01b20df5306500a > # Parent

Re: [PATCH V2] patches: release the GIL while applying the patch

2018-02-22 Thread Yuya Nishihara
On Thu, 22 Feb 2018 12:10:12 +0100, Boris Feld wrote: > # HG changeset patch > # User Boris Feld > # Date 1517839431 -3600 > # Mon Feb 05 15:03:51 2018 +0100 > # Node ID 585005c9c4901f4f94847e8637fbc58cc5b29c56 > # Parent 0c34cb461a1ea5d3f8e1300e0b8bc16ed8fa8802 > #

Re: [PATCH] help: fix wording describing SSH requirements

2018-02-22 Thread Yuya Nishihara
On Wed, 21 Feb 2018 17:51:08 -0500, Josef 'Jeff' Sipek wrote: > # HG changeset patch > # User Josef 'Jeff' Sipek > # Date 1519249869 18000 > # Wed Feb 21 16:51:09 2018 -0500 > # Node ID 106872aa15af9919220705ed72c78459774e1575 > # Parent

Re: [PATCH] mdiff: split on unicode character boundaries when shortening function name

2018-02-22 Thread Yuya Nishihara
On Thu, 22 Feb 2018 09:01:14 +0100, Denis Laxalde wrote: > Josef 'Jeff' Sipek wrote: > > # HG changeset patch > > # User Josef 'Jeff' Sipek > > # Date 1519251311 18000 > > # Wed Feb 21 17:15:11 2018 -0500 > > # Node ID b99df94fdd4813e0ce538a8caa682802da4a6cb2 > > #

Re: [PATCH V2] patches: release the GIL while applying the patch

2018-02-22 Thread Yuya Nishihara
On Thu, 22 Feb 2018 12:10:12 +0100, Boris Feld wrote: > # HG changeset patch > # User Boris Feld > # Date 1517839431 -3600 > # Mon Feb 05 15:03:51 2018 +0100 > # Node ID 585005c9c4901f4f94847e8637fbc58cc5b29c56 > # Parent 0c34cb461a1ea5d3f8e1300e0b8bc16ed8fa8802 > #

Re: [PATCH 2 of 2] perfbranchmap: allow to select the filter to benchmark

2018-02-22 Thread Yuya Nishihara
On Wed, 21 Feb 2018 15:38:12 +0100, Boris Feld wrote: > # HG changeset patch > # User Boris Feld > # Date 1519209792 -3600 > # Wed Feb 21 11:43:12 2018 +0100 > # Node ID ddab5cae846a3e05ec4b951472e562fce06301ea > # Parent f40e09835c645bbc0b535228ac743d84e952f866 > #

Re: D2297: py3: backout 7c54917b31f6 to make sure second argument of open() is str

2018-02-22 Thread Yuya Nishihara
"There are around 100 occurences of open() call, it's better if we can prevent replacing all these calls "In future, when we will dropping all the compatibility code, we don't have to take care of changing pycompat.open back to open Yeah, that's the point why we have `from pycompat import open`

D2297: py3: backout 7c54917b31f6 to make sure second argument of open() is str

2018-02-22 Thread yuja (Yuya Nishihara)
yuja added a comment. "There are around 100 occurences of open() call, it's better if we can prevent replacing all these calls "In future, when we will dropping all the compatibility code, we don't have to take care of changing pycompat.open back to open Yeah, that's the point why

Re: [PATCH] mdiff: split on unicode character boundaries when shortening function name

2018-02-22 Thread Yuya Nishihara
On Thu, 22 Feb 2018 10:01:00 -0500, Josef 'Jeff' Sipek wrote: > On Thu, Feb 22, 2018 at 21:01:44 +0900, Yuya Nishihara wrote: > > On Thu, 22 Feb 2018 09:01:14 +0100, Denis Laxalde wrote: > > > Josef 'Jeff' Sipek wrote: > > > > # HG changeset patch > > > > # User Josef 'Jeff' Sipek

[Bug 5801] New: pure-python manifest code produces corrupt unsorted manifests

2018-02-22 Thread mercurial-bugs
https://bz.mercurial-scm.org/show_bug.cgi?id=5801 Bug ID: 5801 Summary: pure-python manifest code produces corrupt unsorted manifests Product: Mercurial Version: default branch Hardware: PC OS: Mac OS

Re: [PATCH] mdiff: split on unicode character boundaries when shortening function name

2018-02-22 Thread Josef 'Jeff' Sipek
On Fri, Feb 23, 2018 at 01:06:28 +0900, Yuya Nishihara wrote: > On Thu, 22 Feb 2018 10:01:00 -0500, Josef 'Jeff' Sipek wrote: ... > > Yeah... I thought that might be an issue. The code in the 'except' is meant > > as best-effort - > > Ok, I didn't notice that. It's indeed better to catch the

[PATCH] namespace: fastpath name lookup on invalid name

2018-02-22 Thread Boris Feld
# HG changeset patch # User Boris Feld # Date 1519313522 -3600 # Thu Feb 22 16:32:02 2018 +0100 # Node ID b65a85952c09cf4c71a1458fbc4ec77c49683314 # Parent 428de1a59f2df3d6d07ff1d7164c8ee56cbb7825 # EXP-Topic noname # Available At

Re: Diff at end of commit message

2018-02-22 Thread Uwe Brauer
>>> "Stefan" == Stefan Schwarzer writes: > When I commit a change, I edit the commit message in the > editor. Inside the editor, I like a diff of the changes > that are about to be committed. > I found out that I can set a template for the commit >

D2392: debugcommands: add debugwireproto command

2018-02-22 Thread indygreg (Gregory Szorc)
indygreg added a subscriber: sid0. indygreg added a comment. @sid0: I reckon we could find a way to take what I did in this commit and turn it into a wire protocol conformance tester. i.e. if we could refactor the test so it instantiates the server via alternate mechanisms, we could point

D2393: cleanup: say goodbye to manifestv2 format

2018-02-22 Thread durin42 (Augie Fackler)
durin42 created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY This experiment was a bust: we'd hoped for smaller repository sizes, but things got larger. Google ended up rolling out tree manifests in a format that's

D2391: sshpeer: factor out code for creating peers from pipes

2018-02-22 Thread dhduvall (Danek Duvall)
dhduvall added inline comments. INLINE COMMENTS > sshpeer.py:537 > + > +``path`` and ``path`` are stored on the eventual peer instance and may > +not be used for anything meaningful. That second `path` should be `proc`? REPOSITORY rHG Mercurial REVISION DETAIL

D2393: cleanup: say goodbye to manifestv2 format

2018-02-22 Thread martinvonz (Martin von Zweigbergk)
martinvonz added a comment. LGTM. Last time I was going to remove it, I left it in because Durham felt that it was useful for forcing him to not specialize the manifest API too much for how v1 works. I believe he's pretty much done with that now, so that should no longer be a reason to keep

Re: [PATCH 2 of 2 RFC] dispatch: look up command by [:] syntax (PoC)

2018-02-22 Thread Anton Shestakov
On Thu, 22 Feb 2018 23:54:47 +0900 Yuya Nishihara wrote: > # HG changeset patch > # User Yuya Nishihara > # Date 1516798904 -32400 > # Wed Jan 24 22:01:44 2018 +0900 > # Node ID cb9e1cad42a9a13f17c4c75d350cd509b08f4a21 > # Parent

D2392: debugcommands: add debugwireproto command

2018-02-22 Thread indygreg (Gregory Szorc)
indygreg created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY We currently don't have a low-level mechanism for sending arbitrary wire protocol commands. Having a generic and robust mechanism for sending wire protocol

D2391: sshpeer: factor out code for creating peers from pipes

2018-02-22 Thread indygreg (Gregory Szorc)
indygreg created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY An upcoming commit will want to instantiate an SSH peer via an alternate mechanism that doesn't require running a new SSH command. To facilitate that, we

D2394: histedit: make histedit's commands accept revsets (issue5746)

2018-02-22 Thread sangeet259 (Sangeet Kumar Mishra)
sangeet259 updated this revision to Diff 6008. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D2394?vs=6007=6008 REVISION DETAIL https://phab.mercurial-scm.org/D2394 AFFECTED FILES hgext/histedit.py CHANGE DETAILS diff --git a/hgext/histedit.py

D2394: histedit: make histedit's commands accept revsets (issue5746)

2018-02-22 Thread sangeet259 (Sangeet Kumar Mishra)
sangeet259 added a comment. **What I did**: @durin42 suggested in the issue page that the error was in the `verify` function of the `histedit.py` file. But when I ran through debugger using a revision number, the code was throwing the

Re: [PATCH 1 of 6 remotenames-ext] Back out "remotenames: push anonymous head if --force is given"

2018-02-22 Thread Sean Farley
Ryan McElroy writes: > On 2/20/18 11:44 PM, Sean Farley wrote: >> Ryan McElroy writes: >> >>> # HG changeset patch >>> # User Ryan McElroy >>> # Date 1519040197 28800 >>> # Mon Feb 19 03:36:37 2018 -0800 >>> # Node ID

D2394: histedit: make histedit's commands accept revsets (issue5746)

2018-02-22 Thread rishabhmadan96 (Rishabh Madan)
rishabhmadan96 added a comment. It'd be better if you could update the commit message with the necessary details that you mentioned in your previous comment. REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D2394 To: sangeet259, durin42, #hg-reviewers Cc:

[Bug 5802] New: old pager.attend setting breaks hg import

2018-02-22 Thread mercurial-bugs
https://bz.mercurial-scm.org/show_bug.cgi?id=5802 Bug ID: 5802 Summary: old pager.attend setting breaks hg import Product: Mercurial Version: stable branch Hardware: PC OS: Linux Status: UNCONFIRMED

D2394: histedit: make histedit's commands accept revsets (issue5746)

2018-02-22 Thread sangeet259 (Sangeet Kumar Mishra)
sangeet259 added a comment. Oh, I forgot adding tests as @durin42 told. Doing that right away! REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D2394 To: sangeet259, durin42, #hg-reviewers Cc: mercurial-devel ___

D2394: histedit: make histedit's commands accept revsets (issue5746)

2018-02-22 Thread sangeet259 (Sangeet Kumar Mishra)
sangeet259 created this revision. Herald added a reviewer: durin42. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D2394 AFFECTED FILES hgext/histedit.py CHANGE DETAILS diff --git

[PATCH 2 of 2 RFC] dispatch: look up command by [:] syntax (PoC)

2018-02-22 Thread Yuya Nishihara
# HG changeset patch # User Yuya Nishihara # Date 1516798904 -32400 # Wed Jan 24 22:01:44 2018 +0900 # Node ID cb9e1cad42a9a13f17c4c75d350cd509b08f4a21 # Parent e2030eaec92b1ed12577cbe48cd0495d106818a9 dispatch: look up command by [:] syntax (PoC) This allows us to run the

[PATCH 1 of 2 RFC] dispatch: remember command table per extension (PoC DO NOT PUSH)

2018-02-22 Thread Yuya Nishihara
# HG changeset patch # User Yuya Nishihara # Date 1516797703 -32400 # Wed Jan 24 21:41:43 2018 +0900 # Node ID e2030eaec92b1ed12577cbe48cd0495d106818a9 # Parent b8d0761a85c7421071750de23228415306852d69 dispatch: remember command table per extension (PoC DO NOT PUSH) The next

Re: [PATCH] mdiff: split on unicode character boundaries when shortening function name

2018-02-22 Thread Josef 'Jeff' Sipek
On Thu, Feb 22, 2018 at 21:01:44 +0900, Yuya Nishihara wrote: > On Thu, 22 Feb 2018 09:01:14 +0100, Denis Laxalde wrote: > > Josef 'Jeff' Sipek wrote: > > > # HG changeset patch > > > # User Josef 'Jeff' Sipek > > > # Date 1519251311 18000 > > > # Wed Feb 21 17:15:11

D2297: py3: backout 7c54917b31f6 to make sure second argument of open() is str

2018-02-22 Thread pulkit (Pulkit Goyal)
pulkit added a comment. In https://phab.mercurial-scm.org/D2297#37960, @yuja wrote: > Can you elaborate why you think it wasn't a good idea to replace open()? > > IMHO, it's as hackish as replacing '' with b''. Added couple of points. REPOSITORY rHG Mercurial REVISION

mercurial@36358: 6 new changesets

2018-02-22 Thread Mercurial Commits
6 new changesets in mercurial: https://www.mercurial-scm.org/repo/hg/rev/cc9d0763c8e9 changeset: 36353:cc9d0763c8e9 user:Daniel Ploch date:Wed Feb 21 20:05:29 2018 -0800 summary: fancyopts: add support for custom multi-arg opts in fancyopts.py

D2367: narrowcommands: add some missing strkwargs calls for py3

2018-02-22 Thread pulkit (Pulkit Goyal)
pulkit added a comment. This one got pushed with a wrong commit message. REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D2367 To: durin42, #hg-reviewers, indygreg Cc: indygreg, pulkit, mercurial-devel ___

D2383: sshpeer: defer pipe buffering and stderr sidechannel binding

2018-02-22 Thread indygreg (Gregory Szorc)
This revision was automatically updated to reflect the committed changes. Closed by commit rHG11ba1a96f946: sshpeer: defer pipe buffering and stderr sidechannel binding (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D2385: wireproto: document the wonky push protocol for SSH

2018-02-22 Thread indygreg (Gregory Szorc)
This revision was automatically updated to reflect the committed changes. Closed by commit rHGb8d0761a85c7: wireproto: document the wonky push protocol for SSH (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D2380: sshpeer: return framed file object when needed

2018-02-22 Thread indygreg (Gregory Szorc)
This revision was automatically updated to reflect the committed changes. Closed by commit rHG043e77f3be09: sshpeer: return framed file object when needed (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D2381: tests: store protocol payload in files

2018-02-22 Thread indygreg (Gregory Szorc)
This revision was automatically updated to reflect the committed changes. Closed by commit rHG02782e6e2c38: tests: store protocol payload in files (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D2381?vs=5967=5999

D2378: sshpeer: rename _recv and _send to _readframed and _writeframed

2018-02-22 Thread indygreg (Gregory Szorc)
This revision was automatically updated to reflect the committed changes. Closed by commit rHG7f8f74531b0b: sshpeer: rename _recv and _send to _readframed and _writeframed (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D2384: wireprototypes: move baseprotocolhandler from wireprotoserver

2018-02-22 Thread indygreg (Gregory Szorc)
This revision was automatically updated to reflect the committed changes. Closed by commit rHG0c231df1ffdc: wireprototypes: move baseprotocolhandler from wireprotoserver (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D2379: sshpeer: move logic for sending a request into a new function

2018-02-22 Thread indygreg (Gregory Szorc)
This revision was automatically updated to reflect the committed changes. Closed by commit rHGa34d5ef53c2e: sshpeer: move logic for sending a request into a new function (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D2382: sshpeer: make pipe polling code more explicit

2018-02-22 Thread indygreg (Gregory Szorc)
This revision was automatically updated to reflect the committed changes. Closed by commit rHG066e6a9d52bb: sshpeer: make pipe polling code more explicit (authored by indygreg, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE