Page MenuHomePhabricator

Fix root path finding on Windows
AbandonedPublic

Authored by BYK on Apr 10 2016, 4:04 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Mar 26, 10:55 PM
Unknown Object (File)
Thu, Mar 21, 10:55 PM
Unknown Object (File)
Sun, Mar 17, 2:51 PM
Unknown Object (File)
Sun, Mar 17, 4:22 AM
Unknown Object (File)
Wed, Mar 13, 7:28 PM
Unknown Object (File)
Wed, Mar 13, 6:09 PM
Unknown Object (File)
Sat, Mar 9, 9:28 PM
Unknown Object (File)
Fri, Mar 8, 2:02 PM

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T8291: arc version fails on Windows
Summary

/ is not a root path on Windows and when using commands like arc version it resolves to the current working directory. This patch replaces the default / with %HOMEDRIVE% to fix the issue.

This patch fixes T8291 when combined with D15675.

Test Plan

Manual verification on Windows.

Diff Detail

Repository
rPHU libphutil
Branch
windows-root-path
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 15173
Build 19933: arc lint + arc unit

Event Timeline

BYK retitled this revision from to Fix root path finding on Windows.
BYK updated this object.
BYK edited the test plan for this revision. (Show Details)
BYK added a reviewer: epriestley.
eadler added inline comments.
src/filesystem/Filesystem.php
780–782

won't this fail if arc is run on a project checked out to a drive other than %HOMEDRIVE%?

src/filesystem/Filesystem.php
780–782

Possibly. That said there's not reliable equivalent of / on Windows AFAIK. If you think getting the install drive for arc is a better option, I can update accordingly.

Btw. this section is gonna change substantially since there are some other issues I realized when fixing tests. Updating coming soon.

BYK edited edge metadata.
  • Fix test failures

Anything I can do to move this one forward?