Page MenuHomePhabricator

[Wilds] Implement a Filesystem::concatenatePaths(...) method
ClosedPublic

Authored by epriestley on Sep 18 2018, 8:01 PM.
Tags
None
Referenced Files
F13162898: D19691.diff
Mon, May 6, 4:19 PM
F13162197: D19691.id47067.diff
Mon, May 6, 12:45 PM
Unknown Object (File)
Fri, May 3, 6:14 AM
Unknown Object (File)
Thu, Apr 25, 1:04 AM
Unknown Object (File)
Fri, Apr 19, 7:58 PM
Unknown Object (File)
Sun, Apr 14, 6:41 AM
Unknown Object (File)
Apr 2 2024, 5:18 PM
Unknown Object (File)
Mar 30 2024, 6:46 AM
Subscribers
None

Details

Summary

Ref T13098. This is mostly for getting sensible results under Windows, hopefully instead of weird mixed paths with both / and \, at least in more cases.

You can pass in several components like array('/path/to/something/', '/thing.c') and they'll be concatenated with exactly one correct separator.

Test Plan

Used this in the new WorkingCopy stuff, which doesn't run yet. 💁

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 20857
Build 28369: Run Core Tests

Event Timeline

epriestley created this revision.
This revision is now accepted and ready to land.Sep 19 2018, 11:18 PM
epriestley added inline comments.
src/filesystem/Filesystem.php
1057

("Hilariously", this is missing a third parameter and doesn't work at all, but got fixed later.)

This revision was automatically updated to reflect the committed changes.