Paths

Table of Contentst

Parameterize `@{upstream}` in arc to fix Windows
ClosedPublic
Actions

Authored by epriestley on Jul 9 2014, 10:05 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, May 3, 7:17 AM

	Unknown Object (File)
	Mon, Apr 29, 12:09 PM

	Unknown Object (File)
	Mon, Apr 29, 12:09 PM

	Unknown Object (File)
	Mon, Apr 29, 12:05 PM

	Unknown Object (File)
	Mon, Apr 29, 11:59 AM

	Unknown Object (File)
	Wed, Apr 24, 11:00 PM

	Unknown Object (File)
	Wed, Apr 17, 3:20 PM

	Unknown Object (File)
	Apr 2 2024, 3:29 PM

View All 90 Files

Subscribers

epriestley

Details

Reviewers

btrahan
chad

Commits

rARC0be983a7a334: Parameterize `@{upstream}` in arc to fix Windows

Summary

We manually quote this in a couple of places. That works fine on Lunix, but does not work on Windows. Instead, explicitly parameterize the command so the correct quoting rules are applied for the OS.

Test Plan

See IRC; windows user had issues fixed by this. arc:upstream still works locally.

Diff Detail

Repository

rARC Arcanist

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley updated this revision to Diff 23674.Jul 9 2014, 10:05 PM

epriestley retitled this revision from to Parameterize `@{upstream}` in arc to fix Windows.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added reviewers: chad, btrahan.

Herald added a subscriber: epriestley. · View Herald TranscriptJul 9 2014, 10:05 PM

Harbormaster completed remote builds in B1597: Diff 23674.Jul 9 2014, 10:06 PM

chad accepted this revision.Jul 9 2014, 11:05 PM

chad edited edge metadata.

This revision is now accepted and ready to land.Jul 9 2014, 11:05 PM

Closed by commit rARC0be983a7a334 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

repository/

api/

ArcanistGitAPI.php

6 lines

Diff 23702

View Options

src/repository/api/ArcanistGitAPI.php

Parameterize `@{upstream}` in arc to fix WindowsClosedPublicActions