Re: Odd httpheader=1024 required for Phabricator

2020-02-11 Thread Scott Palmer
> On Feb 11, 2020, at 4:49 PM, Emile Snyder wrote: > >  > >> On Tue, Feb 11, 2020 at 1:34 PM Makarius wrote: >> ... >> This patch is BC, but SSH clients shouldn't be using the removed >> capabilities so there should be no impact. >> >> >> What means "BC"? > > I suspect "backwards

Re: Odd httpheader=1024 required for Phabricator

2020-02-11 Thread Makarius
On 12/02/2020 07:50, Augie Fackler wrote: >> >> A bisection over the hg repository yields the following relevant changeset: >> >> changeset: 30563:e118233172fe >> user:Gregory Szorc >> date:Mon Nov 28 20:46:42 2016 -0800 >> files: mercurial/wireproto.py

Re: Odd httpheader=1024 required for Phabricator

2020-02-11 Thread Augie Fackler
> On Feb 11, 2020, at 16:32, Makarius wrote: > > On 11/02/2020 03:20, Augie Fackler wrote: >> I guess I'm not sure what's going on here. >> https://www.mercurial-scm.org/repo/hg/rev/5cda0ce05c42 is the revision that >> introduced that, but I'm not sure why you need to do anything /to >>

D6846: packaging: script the building of a MacOS installer using a custom python

2020-02-11 Thread indygreg (Gregory Szorc)
indygreg added inline comments. INLINE COMMENTS > downloads.py:29 > +'openssl': { > +'url': 'https://www.openssl.org/source/openssl-1.0.2t.tar.gz', > +'size': 5355422, 1.0.2u is out and should be used. > python.py:54 > + "darwin64-x86_64-cc", > +

D6846: packaging: script the building of a MacOS installer using a custom python

2020-02-11 Thread indygreg (Gregory Szorc)
indygreg added a comment. macOS supports a `@loader_path` and related magic tokens in rpath to load libraries relative to the current binary. See e.g. https://blogs.oracle.com/dipol/dynamic-libraries,-rpath,-and-mac-os and

D7930: rust-status: update rust-cpython bridge to account for the changes in core

2020-02-11 Thread Raphaël Gomès
Alphare updated this revision to Diff 20158. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D7930?vs=20049=20158 BRANCH default CHANGES SINCE LAST ACTION https://phab.mercurial-scm.org/D7930/new/ REVISION DETAIL https://phab.mercurial-scm.org/D7930

D7922: rust-matchers: add function to generate a regex matcher function

2020-02-11 Thread Raphaël Gomès
Alphare added inline comments. INLINE COMMENTS > martinvonz wrote in matchers.rs:224 > Hmm, I don't like to replicate this into Rust. I argued for a long time with > Boris over a year ago that we should see if we can remove it from Python. He > said they (Octobus, I think) would look into that

D7119: rust-dirstatemap: remove additional lookup in dirstate.matches

2020-02-11 Thread Raphaël Gomès
Alphare added a comment. In D7119#119792 , @marmoute wrote: > @Alphare so what should we do of this patch ? IMO it should still be valid, it's harmless at best. I don't remember having strong performance numbers. Now that most of the

D7922: rust-matchers: add function to generate a regex matcher function

2020-02-11 Thread Raphaël Gomès
Alphare updated this revision to Diff 20154. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D7922?vs=19401=20154 BRANCH default CHANGES SINCE LAST ACTION https://phab.mercurial-scm.org/D7922/new/ REVISION DETAIL https://phab.mercurial-scm.org/D7922

D7929: rust-status: add bare `hg status` support in hg-core

2020-02-11 Thread Raphaël Gomès
Alphare updated this revision to Diff 20157. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D7929?vs=20048=20157 BRANCH default CHANGES SINCE LAST ACTION https://phab.mercurial-scm.org/D7929/new/ REVISION DETAIL https://phab.mercurial-scm.org/D7929

D7922: rust-matchers: add function to generate a regex matcher function

2020-02-11 Thread Raphaël Gomès
Alphare added inline comments. INLINE COMMENTS > Alphare wrote in matchers.rs:224 > I am having a little trouble reading the patchwork thread, but I gather that > you want to get rid of the preventive splitting of patterns and just let the > regex engine handle it on its own? Was this measure

D7922: rust-matchers: add function to generate a regex matcher function

2020-02-11 Thread Raphaël Gomès
Alphare updated this revision to Diff 20160. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D7922?vs=20154=20160 BRANCH default CHANGES SINCE LAST ACTION https://phab.mercurial-scm.org/D7922/new/ REVISION DETAIL https://phab.mercurial-scm.org/D7922

D8086: rust-status: refactor options into a `StatusOptions` struct

2020-02-11 Thread Raphaël Gomès
Alphare updated this revision to Diff 20155. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D8086?vs=20044=20155 BRANCH default CHANGES SINCE LAST ACTION https://phab.mercurial-scm.org/D8086/new/ REVISION DETAIL https://phab.mercurial-scm.org/D8086

D7931: rust-status: use bare hg status fastpath from Python

2020-02-11 Thread Raphaël Gomès
Alphare updated this revision to Diff 20159. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D7931?vs=20041=20159 BRANCH default CHANGES SINCE LAST ACTION https://phab.mercurial-scm.org/D7931/new/ REVISION DETAIL https://phab.mercurial-scm.org/D7931

D7924: rust-matchers: add `build_regex_match` function

2020-02-11 Thread Raphaël Gomès
Alphare updated this revision to Diff 20162. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D7924?vs=20042=20162 BRANCH default CHANGES SINCE LAST ACTION https://phab.mercurial-scm.org/D7924/new/ REVISION DETAIL https://phab.mercurial-scm.org/D7924

D7923: rust-matchers: add functions to get roots, dirs and parents from patterns

2020-02-11 Thread Raphaël Gomès
Alphare updated this revision to Diff 20161. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D7923?vs=19939=20161 BRANCH default CHANGES SINCE LAST ACTION https://phab.mercurial-scm.org/D7923/new/ REVISION DETAIL https://phab.mercurial-scm.org/D7923

D8087: rust-status: rename `StatusResult` to `DirstateStatus`

2020-02-11 Thread Raphaël Gomès
Alphare updated this revision to Diff 20156. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D8087?vs=20045=20156 BRANCH default CHANGES SINCE LAST ACTION https://phab.mercurial-scm.org/D8087/new/ REVISION DETAIL https://phab.mercurial-scm.org/D8087

D7797: rust-nodemap: pure Rust example

2020-02-11 Thread marmoute (Pierre-Yves David)
marmoute updated this revision to Diff 20166. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D7797?vs=20026=20166 CHANGES SINCE LAST ACTION https://phab.mercurial-scm.org/D7797/new/ REVISION DETAIL https://phab.mercurial-scm.org/D7797 AFFECTED FILES

Mercurial API push bookmark

2020-02-11 Thread Pierre Augier
Hi, I try to write a small Mercurial extension. One of the command would have to do something like hg pull hg up default hg bookmark master hg sum hg push git+ssh://g...@github.com/fluiddyn/fluiddyn hg bookmark master -d These commands work fine (with hg-git). However, when I try to write

D7894: nodemap: introduce an option to use mmap to read the nodemap mapping

2020-02-11 Thread marmoute (Pierre-Yves David)
Herald added a reviewer: indygreg. marmoute updated this revision to Diff 20165. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D7894?vs=20018=20165 CHANGES SINCE LAST ACTION https://phab.mercurial-scm.org/D7894/new/ REVISION DETAIL

[Bug 6268] New: hgweb with py3 error when enabling demandimport

2020-02-11 Thread mercurial-bugs
https://bz.mercurial-scm.org/show_bug.cgi?id=6268 Bug ID: 6268 Summary: hgweb with py3 error when enabling demandimport Product: Mercurial Version: 5.3 Hardware: PC OS: Linux Status: UNCONFIRMED Severity:

D7925: rust-matchers: add `IgnoreMatcher`

2020-02-11 Thread Raphaël Gomès
Alphare updated this revision to Diff 20163. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D7925?vs=20043=20163 BRANCH default CHANGES SINCE LAST ACTION https://phab.mercurial-scm.org/D7925/new/ REVISION DETAIL https://phab.mercurial-scm.org/D7925

D8086: rust-status: refactor options into a `StatusOptions` struct

2020-02-11 Thread Raphaël Gomès
Alphare updated this revision to Diff 20164. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D8086?vs=20155=20164 BRANCH default CHANGES SINCE LAST ACTION https://phab.mercurial-scm.org/D8086/new/ REVISION DETAIL https://phab.mercurial-scm.org/D8086

[PATCH STABLE] chgserver: spawn new process if schemes change

2020-02-11 Thread Yuya Nishihara
# HG changeset patch # User Yuya Nishihara # Date 1581418436 -32400 # Tue Feb 11 19:53:56 2020 +0900 # Branch stable # Node ID 3dcf73d0733d87e7a231809d23da4883f4ab3711 # Parent 84d98fa814a8bab0dd94a133187dc9d42a5e433f chgserver: spawn new process if schemes change The schemes extension

D7931: rust-status: use bare hg status fastpath from Python

2020-02-11 Thread Raphaël Gomès
Alphare updated this revision to Diff 20167. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D7931?vs=20159=20167 BRANCH default CHANGES SINCE LAST ACTION https://phab.mercurial-scm.org/D7931/new/ REVISION DETAIL https://phab.mercurial-scm.org/D7931

D7972: recover: don't verify by default

2020-02-11 Thread valentin.gatienbaron (Valentin Gatien-Baron)
valentin.gatienbaron added a comment. > I _think_ it's just paranoia. As long as the bundle wasn't woefully corrupt, it shouldn't be a problem. I _think_ if we set some of the [server]-section bundle validation options (which should be cheap enough) we could ditch this completely safely.

D7631: absorb: allowing committed changes to be absorbed into their ancestors

2020-02-11 Thread martinvonz (Martin von Zweigbergk)
This revision now requires changes to proceed. martinvonz added a comment. martinvonz requested changes to this revision. Sorry about the delay in responding. Please remember to rebase the series to the latest @ on hg-committed (the release notes are otherwise likely to conflict). INLINE

D7630: absorb: make the absorbed changeset be automatically "evolved"

2020-02-11 Thread martinvonz (Martin von Zweigbergk)
martinvonz added a comment. In D7630#118997 , @rdamazio wrote: >> In D7630#117270 , @marmoute wrote: >> >>> In D7630#115320 , @pulkit wrote: >>>

Re: Odd httpheader=1024 required for Phabricator

2020-02-11 Thread Makarius
On 11/02/2020 03:20, Augie Fackler wrote: > I guess I'm not sure what's going on here. > https://www.mercurial-scm.org/repo/hg/rev/5cda0ce05c42 is the revision that > introduced that, but I'm not sure why you need to do anything /to > phabricator/ unless it's trying (poorly) to pretend to be an

Re: Odd httpheader=1024 required for Phabricator

2020-02-11 Thread Emile Snyder
On Tue, Feb 11, 2020 at 1:34 PM Makarius wrote: > ... > This patch is BC, but SSH clients shouldn't be using the removed > capabilities so there should be no impact. > > > What means "BC"? > I suspect "backwards compatible"? ___ Mercurial mailing list

D7630: absorb: make the absorbed changeset be automatically "evolved"

2020-02-11 Thread marmoute (Pierre-Yves David)
marmoute added a comment. It looks like this series is introducing UI change of the same kind as the one @martinvonz is looking into for `hg copy`. I'll try to have a look at both of them tomorrow. REPOSITORY rHG Mercurial CHANGES SINCE LAST ACTION