Fri, 18 May 2018 19:54:50 -0400 patch: fix import-time syntax error in test-check-module-imports.t
Augie Fackler <augie@google.com> [Fri, 18 May 2018 19:54:50 -0400] rev 38052
patch: fix import-time syntax error in test-check-module-imports.t This is more correct, though we were getting lucky in practice with our rewriter saving us. # skip-blame just a b prefix Differential Revision: https://phab.mercurial-scm.org/D3584
Fri, 18 May 2018 19:52:35 -0400 py3: whitelist two more passing tests observed by buildbot
Augie Fackler <augie@google.com> [Fri, 18 May 2018 19:52:35 -0400] rev 38051
py3: whitelist two more passing tests observed by buildbot Differential Revision: https://phab.mercurial-scm.org/D3583
Sat, 19 May 2018 00:23:36 +0530 py3: use stringutil.pprint() to format a list to print
Pulkit Goyal <7895pulkit@gmail.com> [Sat, 19 May 2018 00:23:36 +0530] rev 38050
py3: use stringutil.pprint() to format a list to print Differential Revision: https://phab.mercurial-scm.org/D3582
Sat, 19 May 2018 00:21:59 +0530 py3: fix kwargs handling in qgurad in hgext/mq.py
Pulkit Goyal <7895pulkit@gmail.com> [Sat, 19 May 2018 00:21:59 +0530] rev 38049
py3: fix kwargs handling in qgurad in hgext/mq.py # skip-blame because just r'' prefixes Differential Revision: https://phab.mercurial-scm.org/D3581
Sat, 19 May 2018 00:19:56 +0530 py3: use .startswith() instead of bytes[0]
Pulkit Goyal <7895pulkit@gmail.com> [Sat, 19 May 2018 00:19:56 +0530] rev 38048
py3: use .startswith() instead of bytes[0] bytes[0] returns the ascii value of character at 0 index. Differential Revision: https://phab.mercurial-scm.org/D3580
Thu, 17 May 2018 23:11:24 -0700 crecord: fallback to text mode if diffs are too big for curses mode
Kyle Lippincott <spectral@google.com> [Thu, 17 May 2018 23:11:24 -0700] rev 38047
crecord: fallback to text mode if diffs are too big for curses mode crecord uses curses.newpad to create a region that we can then scroll around in by moving the main 'screen' as a veiwport into the (probably larger than the actual screen) pad. Internally, at least in ncurses, pads are implemented using windows, which have their dimensions limited to a certain size. Depending on compilation options for ncurses, this size might be pretty small: (signed) short, or it might be larger ((signed) int). crecord wants to have enough room to have all of the contents of the main area of the chunkselector in the pad; this means that the full size with everything expanded must be less than these (undocumented, afaict) limits. It's not easy to write tests for this because the limits are platform- and installation- dependent and undocumented / unqueryable, as far as I can tell. Differential Revision: https://phab.mercurial-scm.org/D3577
Thu, 17 May 2018 15:33:28 -0700 narrow: filter copies in core
Martin von Zweigbergk <martinvonz@google.com> [Thu, 17 May 2018 15:33:28 -0700] rev 38046
narrow: filter copies in core Differential Revision: https://phab.mercurial-scm.org/D3576
Thu, 17 May 2018 15:25:52 -0700 narrow: filter set of files to check for case-folding to core
Martin von Zweigbergk <martinvonz@google.com> [Thu, 17 May 2018 15:25:52 -0700] rev 38045
narrow: filter set of files to check for case-folding to core Differential Revision: https://phab.mercurial-scm.org/D3575
Thu, 17 May 2018 15:12:48 -0700 narrow: filter merge actions in core
Martin von Zweigbergk <martinvonz@google.com> [Thu, 17 May 2018 15:12:48 -0700] rev 38044
narrow: filter merge actions in core Part of the ongoing effort to move narrow into core. Differential Revision: https://phab.mercurial-scm.org/D3574
Thu, 17 May 2018 12:23:38 -0700 hgweb: extract code for emitting multiple changelist records
Gregory Szorc <gregory.szorc@gmail.com> [Thu, 17 May 2018 12:23:38 -0700] rev 38043
hgweb: extract code for emitting multiple changelist records changelistentry() exists so extensions can easily monkeypatch the function to add additional metadata. In at least one case at Mozilla, we have an extension doing this where performance would greatly benefit if we were able to "batch fetch" metadata for all revisions that will eventually have their data set. By extracting the logic for "obtain records for N revisions" into a standalone function, we make it possible for a monkeypatched function to more efficiently bulk fetch data for N revisions. Differential Revision: https://phab.mercurial-scm.org/D3573
Wed, 16 May 2018 22:44:27 -0400 phabricator: register config settings
Matt Harbison <matt_harbison@yahoo.com> [Wed, 16 May 2018 22:44:27 -0400] rev 38042
phabricator: register config settings I didn't bother registering the deprecated phabricator.auth.*, and I'm not sure if the two flagged as 'developer config' should be moved to [devel] (or why there's a distinction between `repophid` and `callsign`).
Wed, 16 May 2018 14:11:41 -0600 filemerge: support specifying a python function to custom merge-tools
hindlemail <tom_hindle@sil.org> [Wed, 16 May 2018 14:11:41 -0600] rev 38041
filemerge: support specifying a python function to custom merge-tools Eliminates the need to specify a python executable, which may not exist on system. Additionally launching script inprocess aids portability on systems that can't execute python via the shell. Example usage "merge-tools.myTool.executable=python:c:\myTool.py:mergefn" where myTool.py contains a function: "def mergefn(ui, repo, args, **kwargs):" where args is list of args passed to merge tool. (by default, expanded: $local $base $other) Invoking the specified python function was done by exposing and invoking (hook._pythonhook -> hook.pythonhook)
Thu, 24 May 2018 21:54:31 +0900 help: correct signature of separate() template function stable
Yuya Nishihara <yuya@tcha.org> [Thu, 24 May 2018 21:54:31 +0900] rev 38040
help: correct signature of separate() template function Without the dots, it looked as if separate() would take a list of arguments.
Fri, 18 May 2018 21:32:05 +0900 hgweb: do not try to replace signal handlers while locking stable
Yuya Nishihara <yuya@tcha.org> [Fri, 18 May 2018 21:32:05 +0900] rev 38039
hgweb: do not try to replace signal handlers while locking According to the issue 5889, mod_wsgi issues a warning on signal.signal() call, and we wouldn't want to see it in error log. The problem addressed by d77c3b023393 could potentially occur in web session, but that would be less likely than in user processes.
Fri, 18 May 2018 21:24:06 +0900 lock: add internal config to not replace signal handlers while locking stable
Yuya Nishihara <yuya@tcha.org> [Fri, 18 May 2018 21:24:06 +0900] rev 38038
lock: add internal config to not replace signal handlers while locking signal.signal() is blocked in some WSGI environments, and a horrible warning is sent to the server log. So we need a way to disable it, and I think abusing ui.config is the simplest workaround.
Tue, 22 May 2018 21:51:20 -0400 merge with i18n stable
Augie Fackler <augie@google.com> [Tue, 22 May 2018 21:51:20 -0400] rev 38037
merge with i18n
Fri, 04 May 2018 18:55:57 -0300 i18n-pt_BR: synchronized with 32a75a8a5b0f stable
Wagner Bruna <wbruna@softwareexpress.com.br> [Fri, 04 May 2018 18:55:57 -0300] rev 38036
i18n-pt_BR: synchronized with 32a75a8a5b0f
Fri, 04 May 2018 18:55:29 -0300 i18n-ja: fix block indentation stable
Wagner Bruna <wbruna@softwareexpress.com.br> [Fri, 04 May 2018 18:55:29 -0300] rev 38035
i18n-ja: fix block indentation
Tue, 01 May 2018 18:22:52 +0900 i18n-ja: synchronized with 32a75a8a5b0f stable
FUJIWARA Katsunori <foozy@lares.dti.ne.jp> [Tue, 01 May 2018 18:22:52 +0900] rev 38034
i18n-ja: synchronized with 32a75a8a5b0f
Mon, 21 May 2018 15:14:46 +0200 httppeer: declare 'dbg' at the function level stable
Boris Feld <boris.feld@octobus.net> [Mon, 21 May 2018 15:14:46 +0200] rev 38033
httppeer: declare 'dbg' at the function level As we just saw in the previous changeset, having the variable defined into a branch creates bug. This is a cheap to move it at the function level.
Fri, 04 May 2018 19:06:46 +0200 httppeer: properly gate debug usage behind debug flag check stable
Boris Feld <boris.feld@octobus.net> [Fri, 04 May 2018 19:06:46 +0200] rev 38032
httppeer: properly gate debug usage behind debug flag check The "dbg" local variable is only defined if the 'debugflag' is set to True. However, it was used indiscriminately later in the function. We hide its usage behind the 'debugflag' value to avoid raising a NameError.
Sun, 13 May 2018 11:09:53 +0900 test-http-branchmap: fix stdio mode on Windows
Yuya Nishihara <yuya@tcha.org> [Sun, 13 May 2018 11:09:53 +0900] rev 38031
test-http-branchmap: fix stdio mode on Windows
Sun, 13 May 2018 11:08:35 +0900 test-http-branchmap: fix encoding test to wrap the server stream
Yuya Nishihara <yuya@tcha.org> [Sun, 13 May 2018 11:08:35 +0900] rev 38030
test-http-branchmap: fix encoding test to wrap the server stream Wrapping sys.std* streams has no effect since 39d13b8c101d, "py3: bulk replace sys.stdin/out/err by util's."
Thu, 17 May 2018 21:55:00 +0900 test-merge-tools: create repo directory to free $TESTTMP for temporary files
Yuya Nishihara <yuya@tcha.org> [Thu, 17 May 2018 21:55:00 +0900] rev 38029
test-merge-tools: create repo directory to free $TESTTMP for temporary files
Sat, 12 May 2018 23:44:08 +0200 notify: add option to include function names in the diff output
Joerg Sonnenberger <joerg@bec.de> [Sat, 12 May 2018 23:44:08 +0200] rev 38028
notify: add option to include function names in the diff output This is a localized version of diff.showfunc. Differential Revision: https://phab.mercurial-scm.org/D3553
Wed, 16 May 2018 10:34:31 -0700 tests: update py3 test since json fields are now sorted
Martin von Zweigbergk <martinvonz@google.com> [Wed, 16 May 2018 10:34:31 -0700] rev 38027
tests: update py3 test since json fields are now sorted The fields have been sorted since 814151cd8c4a (logcmdutil: rewrite jsonchangeset printer to be backed by jsonformatter, 2018-04-13), and the test case has thus been failing since that commit. Differential Revision: https://phab.mercurial-scm.org/D3570
Wed, 16 May 2018 15:48:12 -0700 tests: test-parseindex.t works just fine with chg
Kyle Lippincott <spectral@google.com> [Wed, 16 May 2018 15:48:12 -0700] rev 38026
tests: test-parseindex.t works just fine with chg In 538e850ae737 I annotated tests that failed with chg with "#require no-chg", but did not investigate why the tests were failing. This test was failing for unrelated reasons (I forgot to rebuild hg) and so should not be marked as no-chg. Test Plan: make local pushd contrib/chg make clean; make popd cd tests for f in '' --with-chg=../contrib/chg/chg; do ./run-tests.py --local $f test-parseindex.t done Differential Revision: https://phab.mercurial-scm.org/D3571
Fri, 27 Apr 2018 21:55:02 +0530 py3: convert the report to bytes
Pulkit Goyal <7895pulkit@gmail.com> [Fri, 27 Apr 2018 21:55:02 +0530] rev 38025
py3: convert the report to bytes Differential Revision: https://phab.mercurial-scm.org/D3569
Fri, 11 May 2018 10:36:28 -0700 json: reject unicode on py2 as well
Martin von Zweigbergk <martinvonz@google.com> [Fri, 11 May 2018 10:36:28 -0700] rev 38024
json: reject unicode on py2 as well This makes it consistent with the behavior on py3. Differential Revision: https://phab.mercurial-scm.org/D3536
Wed, 16 May 2018 20:22:23 +0900 dispatch: mask negative exit code recorded in blackbox log
Yuya Nishihara <yuya@tcha.org> [Wed, 16 May 2018 20:22:23 +0900] rev 38023
dispatch: mask negative exit code recorded in blackbox log That's what we do for the exit code delivered to the environment.
(0) -30000 -10000 -3000 -1000 -300 -100 -50 -30 +30 +50 +100 +300 +1000 +3000 +10000 tip