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
F18381561: D21052.id50153.diff
Thu, Aug 28, 8:03 PM
F18380667: D21052.diff
Thu, Aug 28, 6:07 PM
F18380544: D21052.id.diff
Thu, Aug 28, 5:56 PM
F18265503: D21052.id50155.diff
Fri, Aug 22, 12:06 PM
F18234906: D21052.id50155.diff
Wed, Aug 20, 6:50 PM
F18221431: D21052.id50155.diff
Tue, Aug 19, 6:03 PM
F18217258: D21052.id50155.diff
Tue, Aug 19, 8:44 AM
F18207050: D21052.id50155.diff
Mon, Aug 18, 4:38 PM
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