I do not understand what the intended side-effect of running this command is supposed to be.
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Feed All Stories
All Stories
All Stories
Yesterday
Yesterday
histedit: add missing b prefix to a string
debugcommands: fix typo in debuguigetpass
yuja committed rHG67f757ed86e0: dagop: fix subsetparentswalker to set p1/p2 chains at merge revision.
dagop: fix subsetparentswalker to set p1/p2 chains at merge revision
dagop: simplify dict/set reuse condition in subsetparentswalker
indygreg committed rHG843418dc0b1b: extensions: refactor function for obtaining disabled extension help.
extensions: refactor function for obtaining disabled extension help
tests: perform grep manually in test-doctest.py
tests: prevent printing \r to stdout
tests: force \n newlines when writing to sys.stdout
dispatch: force \n for newlines on sys.std* streams (BC)
hook: move stdio redirection to context manager
D8243: copies: stop recording buggy file merge when new file overwrite an old one now requires changes to proceed.
Per irc checkin, we expect changes on this before review.
pycompat: change argv conversion semantics
Heads up that we ran into a fun bug that was probably caused by this patch. I think what happened was this:
mharbison72 committed rHG99fa161a883c: phabricator: extract the logic to amend diff properties to a function.
phabricator: extract the logic to amend diff properties to a function
mharbison72 committed rHG949a87145336: phabricator: extract logic to print the status when posting a commit.
phabricator: extract logic to print the status when posting a commit
mharbison72 committed rHG5f9c917e3b50: phabricator: teach `getoldnodedrevmap()` to handle folded reviews.
phabricator: teach `getoldnodedrevmap()` to handle folded reviews
mharbison72 committed rHG419fec8237b7: phabricator: teach createdifferentialrevision() to allow a folded commit range.
phabricator: teach createdifferentialrevision() to allow a folded commit range
mharbison72 committed rHG0437959de6f5: phabricator: record all local commits used to create a Differential revision.
phabricator: record all local commits used to create a Differential revision
mharbison72 committed rHGdbe9182c90f5: phabricator: combine commit messages into the review when folding commits.
phabricator: combine commit messages into the review when folding commits
tests: use `f --hexdump` to print file content
url: pass str to pathname2url
tests: pass str to matchoutput()
hgcli: customize for Mercurial
hgcli: add stub PyOxidizer project
hgcli: remove legacy project
marmoute committed rHGc70bcaf7927b: nodemap: automatically "vacuum" the persistent nodemap when too sparse.
nodemap: automatically "vacuum" the persistent nodemap when too sparse
nodemap: display percentage of unused in `hg debugnodemap`
marmoute committed rHG01b0805534bb: nodemap: make sure on disk change get rolled back with the transaction.
nodemap: make sure on disk change get rolled back with the transaction
marmoute committed rHGdd5b47fb0860: nodemap: test that concurrent process don't see the pending transaction.
nodemap: test that concurrent process don't see the pending transaction
marmoute committed rHG82543879b48e: testlib: adjust wait-on-file timeout according to the global test timeout.
testlib: adjust wait-on-file timeout according to the global test timeout
marmoute committed rHG1ed6293fc31b: testlib: add a small scrip to help process to synchronise using file.
testlib: add a small scrip to help process to synchronise using file
setup: work around old versions of distutils breaking setup.py
chgserver: update the umask cache before each run
sure
D8352: extensions: don't crash if __file__ not defined now requires changes to proceed.
Seems overall good. I added a request for comments.
Tue, Mar 31
Tue, Mar 31
mharbison72 added inline comments to D8314: phabricator: add an option to fold several commits into one review (issue6244).
pulkit added a comment to D8189: testlib: add a small scrip to help process to synchronise using file.
I wanted to help with things here but unfortunately I have ~0 experience with shell scripts and the kind of process testing going in next few patches.
> On Windows, my assumption was os.fsencode() == .encode("mbcs") if > sys._enablelegacywindowsfsencoding(). So this looks good to me. > Perhaps, the "ignore" error mode would match the legacy Windows behavior. Does this mean it could be a problem running from source on Windows? For example, `hg version` (as opposed to `hg.exe version`) seems to be equivalent to `python hg`, which obviously doesn't have the proper environment variable or C API option to enable legacy mode. Should there be code early on that detects this and warns/aborts?
durin42 committed rHG4dacd0cef146: tests: handle new error string from FreeBSD for dns entry not resolving.
tests: handle new error string from FreeBSD for dns entry not resolving
notify: optional mail threading based on obsmarker
D8335: url: pass str to pathname2url is now accepted and ready to land.
D8334: tests: pass str to matchoutput() is now accepted and ready to land.
D8333: tests: handle new error string from FreeBSD for dns entry not resolving is now accepted and ready to land.
D8314: phabricator: add an option to fold several commits into one review (issue6244) now requires changes to proceed.
Mon, Mar 30
Mon, Mar 30
mharbison72 added inline comments to D8314: phabricator: add an option to fold several commits into one review (issue6244).
mharbison72 added a comment to D8344: extensions: refactor function for obtaining disabled extension help.
I noticed inconsistencies with this too. I was wondering if we should populate __index__ with the full help text to avoid that. (The other inconsistency that comes to mind is printing out the disabled extension list in hg help extensions.) While it is apparently only used for py2exe based stuff, it might be handy with PyOxidizer too, until that gains the ability to enumerate and read embedded *.py files and directories as resources.
I suspect the Windows process code in Python doesn't accept bytes and the POSIX code does?
In D8337#124542, @yuja wrote:On Windows, my assumption was os.fsencode() == .encode("mbcs") if
sys._enablelegacywindowsfsencoding(). So this looks good to me.
Perhaps, the "ignore" error mode would match the legacy Windows behavior.
D8172: notify: optional mail threading based on obsmarker is now accepted and ready to land.
This looks like it also fixes the phabricator test on Windows, which diverged by creating a different hash for the create alpha for phabricator test € commit.
The test harness *should* match existing \n output as a fallback, which got me to wondering if it was the (esc) at the end screwing it up. I tried this patch:
Sun, Mar 29
Sun, Mar 29
This is a dupe of D8339.