mharbison72 added a comment.
Any movement on this? There's a lot of test spew on Windows that this would
cut out.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D2988
To: hooper, #hg-reviewers
Cc: durin42, mharbison72, mercurial-devel
__
# HG changeset patch
# User Matt Harbison
# Date 1523415475 14400
# Tue Apr 10 22:57:55 2018 -0400
# Node ID 16cd8fe997579a6d4aecfe15b54daee3081610e7
# Parent 50ea07f04fd451faf27e14d914baf67eb51f15d5
tests: stabilize test-pull-bundle.t for Windows
See 594dd384803c.
diff --git a/tests/test-
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
We don't need to be advertising client protocol parameters as part
of the capabilities request during the handshake because nothing in
version 1 of the wire pro
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
Previously, we constructed an httppeer then always ran _fetchcaps()
to issue the capabilities command.
We want to issue the capabilities command before const
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
This allows the common redirect detection, content type
validation, and decompression wrapping to be usable outside of
httppeer instances.
REPOSITORY
rHG Mer
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
When clients connect to repositories over HTTP, they issue a request
to the well-known URL "?cmd=capabilities" to fetch the repository
capabilities. This is the
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
Before, we manually updated the Vary header value for each
header contributing to it.
All X-Hg* headers are reserved for the Mercurial protocol and
could h
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
Upcoming commits will move the initial ?cmd=capabilities handshake
request out of httppeer so the handshake can be performed before a
peer instance is construct
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
This allows us to use makepeer() in `hg debugwireproto`.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D3238
AFFECTED FILES
mercu
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
This is common for all HTTP requests. It should be part of
sendrequest().
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D3235
AFF
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
With the new handshake defined and in place on the server, we can
now implement it on the client.
The HTTP handshake mechanism has been taught to add headers
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
Versions of Mercurial older than 1.0 emitted the text/plain
and application/hg-changegroup media types in response to wire
protocol commands.
Way back in
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
This is generic and doesn't need to live as a method of httppeer.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D3233
AFFECTED FILE
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
wireprotoserver has generic and version 1 server code. The wireproto
module also has both version 1 and version 2 command implementations.
Upcoming work I want
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
This is relatively straightforward.
As part of this, we introduced a local @wireprotocommand that
wraps the main one and defines a v2 only policy by default.
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
Some of this feels awkward, such as having to pass in a function
to evaluate a capability. And this code is generally pretty difficult
to read. I didn't want to
indygreg abandoned this revision.
indygreg added a comment.
I fired this off too soon :(
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D3225
To: indygreg, #hg-reviewers
Cc: mercurial-devel
___
Mercurial-devel mailing l
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
It is complaining about unused imports.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D3227
AFFECTED FILES
tests/test-check-pyfla
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
I was lazy when I put these in test-wireproto-serverreactor.py. Let's
do it properly.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.or
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
We have a nice state machine of sorts for reacting to server-side
events. Now it is time to implement the client equivalent.
We introduce a "clientreactor."
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
We can now feed received frames into the client reactor and it will
validate their sanity, dispatch them appropriately.
The hacky HTTP peer has been updated
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
Python 3 has a concurrent.futures package in the standard library
for representing futures. The "futures" package on PyPI is a backport
of this package to work
indygreg updated this revision to Diff 7939.
REPOSITORY
rHG Mercurial
CHANGES SINCE LAST UPDATE
https://phab.mercurial-scm.org/D3208?vs=7912&id=7939
REVISION DETAIL
https://phab.mercurial-scm.org/D3208
AFFECTED FILES
mercurial/wireproto.py
CHANGE DETAILS
diff --git a/mercurial/wirepro
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
This is used so mutliple directories can provide a package. We don't
need it when vendoring.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
We conceivably don't need to distribute this package on Python 3
since we will use the version in the standard library. However,
we want installs to be usable o
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
This makes the package conform with our importing policy,
silencing a number of warnings. It also makes the package usable
when it isn't named "concurrent.futur
indygreg updated this revision to Diff 7938.
REPOSITORY
rHG Mercurial
CHANGES SINCE LAST UPDATE
https://phab.mercurial-scm.org/D3206?vs=7910&id=7938
REVISION DETAIL
https://phab.mercurial-scm.org/D3206
AFFECTED FILES
mercurial/wireproto.py
tests/test-http-api-httpv2.t
tests/test-wir
quark added inline comments.
INLINE COMMENTS
> spectral wrote in color.py:94
> I understand the reasoning for wanting `diff.inserted` and
> `diff.inserted.changed` to be the same color, but unfortunately I think it
> might not be super feasible in upstream.
>
> 'dim' support in my terminfo dat
# HG changeset patch
# User Matt Harbison
# Date 1523154140 14400
# Sat Apr 07 22:22:20 2018 -0400
# Node ID b784de3b414876f25964778b02e684f9b0a3787f
# Parent 473a8954c957945d1642d08eeefc8fe706597b59
lfs: infer the blob store URL from paths.default
If `lfs.url` is specified, it takes preced
# HG changeset patch
# User Matt Harbison
# Date 1523211732 14400
# Sun Apr 08 14:22:12 2018 -0400
# Node ID 9dea6544c2d20e5bab1561266d13124bfd536e07
# Parent b0ae606b4a10738c97d791ac6867cd100eb2ea9d
lfs: infer the blob store URL from an explicit push dest or default-push
Unlike pull, the b
# HG changeset patch
# User Matt Harbison
# Date 1523165019 14400
# Sun Apr 08 01:23:39 2018 -0400
# Node ID b0ae606b4a10738c97d791ac6867cd100eb2ea9d
# Parent b784de3b414876f25964778b02e684f9b0a3787f
lfs: infer the blob store URL from an explicit pull source
I don't see any easier way to do
# HG changeset patch
# User Matt Harbison
# Date 1523155211 14400
# Sat Apr 07 22:40:11 2018 -0400
# Node ID 473a8954c957945d1642d08eeefc8fe706597b59
# Parent 49a8c2cc7978cdd73155fc354aae8851a79c58d8
lfs: add the ability to disable the usercache
While the usercache is important for real wor
spectral added subscribers: dhduvall, indygreg.
spectral added inline comments.
INLINE COMMENTS
> quark wrote in color.py:94
> As I mentioned in the summary, I believe `diff.inserted` and
> `diff.inserted.changed` should have a same color. And `diff.inserted`
> probably shouldn't be bold.
>
>
quark added a comment.
That's ideal. But a lot of code in this area expects "line" to not contain
"\n". So the change won't be as easy as it looks.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D3210
To: quark, #hg-reviewers
Cc: yuja, mercurial-devel
__
quark added a comment.
Git first had a `contrib/diff-highlight/diff-highlight` script which inverts
foreground/background for hunks with len(deleted_lines) = len(inserted_lines).
Then the latest version shows diff inline. That is:
common words [+inserted words with green color][-de
This revision was automatically updated to reflect the committed changes.
Closed by commit rHG9b16a67cef56: eol: look up partial nodeid as partial nodeid
(authored by martinvonz, committed by ).
REPOSITORY
rHG Mercurial
CHANGES SINCE LAST UPDATE
https://phab.mercurial-scm.org/D3159?vs=7881&i
This revision was automatically updated to reflect the committed changes.
Closed by commit rHGc4131138eadb: histedit: look up partial nodeid as partial
nodeid (authored by martinvonz, committed by ).
REPOSITORY
rHG Mercurial
CHANGES SINCE LAST UPDATE
https://phab.mercurial-scm.org/D3158?vs=7
This revision was automatically updated to reflect the committed changes.
Closed by commit rHG966061b8826d: histedit: drop unnecessary check for
"self.node is not None" (authored by martinvonz, committed by ).
REPOSITORY
rHG Mercurial
CHANGES SINCE LAST UPDATE
https://phab.mercurial-scm.org/
This revision was automatically updated to reflect the committed changes.
Closed by commit rHG901e749ca0e1: context: extract partial nodeid lookup method
to scmutil (authored by martinvonz, committed by ).
CHANGED PRIOR TO COMMIT
https://phab.mercurial-scm.org/D3189?vs=7878&id=7934#toc
REPOSIT
yuja added a comment.
My bad. Added `if node is None: return` and pushed, thanks.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D3189
To: martinvonz, #hg-reviewers, yuja
Cc: yuja, mercurial-devel
___
Mercurial-devel ma
# HG changeset patch
# User Yuya Nishihara
# Date 1522590109 -32400
# Sun Apr 01 22:41:49 2018 +0900
# Node ID a4d06c685ee182279e91c2856d8a69e809c6e317
# Parent d9f6292315ef2ee3553a6dc10c20eefd
hgweb: wrap {archives} with mappinglist
No bare list of mappings should be put in a templ
# HG changeset patch
# User Yuya Nishihara
# Date 1522590015 -32400
# Sun Apr 01 22:40:15 2018 +0900
# Node ID d9f6292315ef2ee3553a6dc10c20eefd
# Parent f698f65fbaf88305353eb769e0130299fc747f08
hgweb: forward archivelist() of hgweb to webutil
self.configlist() is ui.configlist(untru
# HG changeset patch
# User Yuya Nishihara
# Date 1522588844 -32400
# Sun Apr 01 22:20:44 2018 +0900
# Node ID 411dfde85bce520eb637ea52bce93e336b9e9a30
# Parent 42132043359a74ab3d703f1791644a39b8b6e8dc
hgweb: wrap {labels} by hybridlist()
This one is a simple list of strings, which can supp
# HG changeset patch
# User Yuya Nishihara
# Date 1522588476 -32400
# Sun Apr 01 22:14:36 2018 +0900
# Node ID 42132043359a74ab3d703f1791644a39b8b6e8dc
# Parent 4dcd5d6cbfd17115e4a2359c124ec8887bb7986c
hgweb: wrap {pathdef} with mappinglist
No bare list of mappings should be put in a templa
# HG changeset patch
# User Yuya Nishihara
# Date 1522589635 -32400
# Sun Apr 01 22:33:55 2018 +0900
# Node ID cf2d909fdec997a9bb39573ded50ae362a667221
# Parent 1dd6d286577ee108d2b643f320bf22e72823b47d
hgweb: drop archivespecs from requestcontext
It's a constant.
diff --git a/mercurial/hgw
# HG changeset patch
# User Yuya Nishihara
# Date 1522589823 -32400
# Sun Apr 01 22:37:03 2018 +0900
# Node ID f698f65fbaf88305353eb769e0130299fc747f08
# Parent cf2d909fdec997a9bb39573ded50ae362a667221
hgweb: move archivelist() of hgwebdir to webutil
diff --git a/mercurial/hgweb/hgwebdir_mo
# HG changeset patch
# User Yuya Nishihara
# Date 1522589554 -32400
# Sun Apr 01 22:32:34 2018 +0900
# Node ID 1dd6d286577ee108d2b643f320bf22e72823b47d
# Parent 411dfde85bce520eb637ea52bce93e336b9e9a30
hgweb: move archivespecs to webutil
Move one step further from e38e7ea21987. Since 'archi
# HG changeset patch
# User Yuya Nishihara
# Date 1522588318 -32400
# Sun Apr 01 22:11:58 2018 +0900
# Node ID 4dcd5d6cbfd17115e4a2359c124ec8887bb7986c
# Parent 479fbecf9cccd27040b88e713956d15286b6d2b0
hgwebdir: wrap {entries} with mappinggenerator
No bare generator should be put in a templ
yuja added a comment.
BTW, what's the default of git?
The comment in color.py is a bit scary, which says "most terminals don't
support dim." If that's true, we shouldn't use "dim" by default.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D3212
To: quark,
martinvonz added inline comments.
INLINE COMMENTS
> yuja wrote in scmutil.py:439
> I could add it if you're happy with the following change.
>
> node = repo.unfiltered().changelog._partialmatch(prefix)
> repo.changelog.rev(node) # make sure node isn't filtered
> return node
LGTM. Thank y
> On Apr 10, 2018, at 9:34 AM, Yuya Nishihara wrote:
>
>> On Mon, 9 Apr 2018 17:43:47 -0400, Matt Harbison wrote:
>>
On Apr 9, 2018, at 9:43 AM, Yuya Nishihara wrote:
On Mon, 09 Apr 2018 00:26:45 -0400, Matt Harbison wrote:
# HG changeset patch
# User Matt Harbison
>
yuja requested changes to this revision.
yuja added a comment.
This revision now requires changes to proceed.
`print(nonascii)` may fail depending on the locale.
We have to use `getattr(sys.stdout, 'buffer', sys.stdout)` or
`pycompat.stdout`.
REPOSITORY
rHG Mercurial
REVISION DETAIL
ht
yuja requested changes to this revision.
yuja added inline comments.
This revision now requires changes to proceed.
INLINE COMMENTS
> mq.py:670
> +return True, pycompat.bytestr(repr(exactpos[0]))
> +return False, ' '.join([pycompat.bytestr(repr(p)) for p in pos])
>
https://bz.mercurial-scm.org/show_bug.cgi?id=5836
Bug ID: 5836
Summary: import of a git patch containing 'From' in the commit
message confuses Mercurial
Product: Mercurial
Version: 4.5
Hardware: PC
OS: Linux
yuja added inline comments.
INLINE COMMENTS
> martinvonz wrote in scmutil.py:439
> Makes sense. Okay if I do that in a follow-up so the rest of the stack is not
> blocked by this (I will be away until Friday)?
I could add it if you're happy with the following change.
node = repo.unfiltered()
yuja requested changes to this revision.
yuja added a comment.
This revision now requires changes to proceed.
I have no opinion about the "dim" thingy, but the series generally looks
good to me.
Thanks for tackling on the painfully slow `SequenceMatcher.ratio()` issue.
INLINE COMMENTS
yuja requested changes to this revision.
yuja added inline comments.
This revision now requires changes to proceed.
INLINE COMMENTS
> patch.py:2484
> +stripline = chompline.rstrip()
> +if line[0] == '-':
> +label = 'diff.deleted'
Don't use `bytes[n]` since it returns
pulkit created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
1. skip-blame because just b'' prefixes
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D3221
AFFECTED FILES
tests/test-ui-config.py
yuja added a comment.
Perhaps, it's simpler to use `mdiff.splitnewlines(chunk)` instead of
`chunk.split('\n')`
because otherwise the next patch has to reconstruct `line + '\n'`.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D3210
To: quark, #hg-reviewers
Cc:
pulkit created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
1. skip-blame because just b'' prefix
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D3220
AFFECTED FILES
tests/test-mq.t
CHANGE DE
pulkit created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
Comparing None with integers was fine in Python 2 but returns error in Python
3
which is nice. This patch replaces None with -1 where sorting is done and some
re
pulkit created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D3218
AFFECTED FILES
tests/test-mq-qpush-fail.t
CHANGE DETAILS
diff --git a/tests/test-mq-qpush-fail.t
pulkit created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D3217
AFFECTED FILES
tests/test-mq-qimport.t
CHANGE DETAILS
diff --git a/tests/test-mq-qimport.t b/test
pulkit created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D3216
AFFECTED FILES
hgext/mq.py
CHANGE DETAILS
diff --git a/hgext/mq.py b/hgext/mq.py
--- a/hgext/mq.p
pulkit created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
We deal with bytes internally and things should be bytes
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D3215
AFFECTED FILES
hgext/m
pulkit created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D3214
AFFECTED FILES
hgext/mq.py
CHANGE DETAILS
diff --git a/hgext/mq.py b/hgext/mq.py
--- a/hgext/mq.p
martinvonz added inline comments.
INLINE COMMENTS
> yuja wrote in scmutil.py:439
> We'll need to verify that the returned node is NOT hidden by e.g.
> calling `changelog.rev(node)`.
>
> It's unlikely that callers expect this function may return a hidden
> node.
Makes sense. Okay if I do that in
On Mon, 9 Apr 2018 17:43:47 -0400, Matt Harbison wrote:
>
> > On Apr 9, 2018, at 9:43 AM, Yuya Nishihara wrote:
> >
> >> On Mon, 09 Apr 2018 00:26:45 -0400, Matt Harbison wrote:
> >> # HG changeset patch
> >> # User Matt Harbison
> >> # Date 1523211732 14400
> >> # Sun Apr 08 14:22:12 2018
khanchi97 added a comment.
@pulkit Can you please review it?
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D3187
To: khanchi97, pulkit, #hg-reviewers
Cc: av6, mercurial-devel
___
Mercurial-devel mailing list
Mercurial-
69 matches
Mail list logo