Page MenuHomePhabricator
Differential D21052

Update some ancient "set X=Y" environment code for new Windows execution without a shell
ClosedPublic
Actions

Authored by epriestley on Apr 2 2020, 8:43 PM.
Tags
None
Referenced Files
F18742086: D21052.id.diff
Thu, Oct 2, 5:19 PM
F18739741: D21052.diff
Wed, Oct 1, 10:42 PM
F18642604: D21052.diff
Fri, Sep 19, 12:18 AM
F18567705: D21052.id.diff
Sep 9 2025, 4:06 PM
F18464032: D21052.id50155.diff
Sep 2 2025, 3:10 AM
F18463760: D21052.id50155.diff
Sep 2 2025, 2:12 AM
F18463636: D21052.id50153.diff
Sep 2 2025, 1:41 AM
F18463631: D21052.id.diff
Sep 2 2025, 1:41 AM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13209: How To Properly Escape Commands on Windows (A Dark Tragedy)
T13504: "proc_open()" may fail to return a resource on Windows if the command isn't executable
Commits
rARC368aec16a1ee: Update some ancient "set X=Y" environment code for new Windows execution…
Summary

Depends on D21051. Ref T13504. Ref T13209. This very old Mercurial code uses "X=Y hg ..." on Linux and "set X=Y & hg ..." on Windows.

The latter construct no longer works because we bypass the shell. The former construct is obsolete.

Additionaly, delete some ancient "branch merge" code which has no callers.

Test Plan

Created a diff in a Mercurial repository on Linux. I minimally vetted this on Windows since I don't have a "hg + Windows" environment at the moment.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Apr 2 2020, 8:43 PM
Harbormaster completed remote builds in B23972: Diff 50153.Apr 2 2020, 8:44 PM
epriestley requested review of this revision.Apr 2 2020, 8:44 PM
This revision was not accepted when it landed; it landed in state Needs Review.Apr 2 2020, 8:51 PM
Closed by commit rARC368aec16a1ee: Update some ancient "set X=Y" environment code for new Windows execution… (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rARC368aec16a1ee: Update some ancient "set X=Y" environment code for new Windows execution….

Revision Contents
Changeset List

Diff 50155

View Options

src/repository/api/ArcanistGitAPI.php

Loading...
View Options

src/repository/api/ArcanistMercurialAPI.php

Loading...
View Options

src/repository/api/ArcanistRepositoryAPI.php

Loading...
View Options

src/repository/api/ArcanistSubversionAPI.php

Loading...
Phabricator · Built by Phacility