Differential D15693

Remove Windows workarounds from ArcanistGitAPI
AbandonedPublic
Actions

Authored by BYK on Apr 13 2016, 6:11 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Jun 2, 5:12 AM

	Unknown Object (File)
	Sat, May 31, 5:36 AM

	Unknown Object (File)
	Thu, May 29, 4:13 AM

	Unknown Object (File)
	Tue, May 27, 11:01 AM

	Unknown Object (File)
	Tue, May 27, 9:03 AM

	Unknown Object (File)
	Tue, May 27, 3:39 AM

	Unknown Object (File)
	Sun, May 25, 2:41 AM

	Unknown Object (File)
	Sat, May 24, 11:42 AM

Subscribers

Details

Reviewers

Group Reviewers

Blessed Reviewers

Summary

D15675 fixes rPHU so we don't need the workarounds for Windows in the Git API anymore. In fact they are broken with D15675 since they are either incorrectly escaped or double-escaped.

Depends on D15675.

Test Plan

manually verified.

Diff Detail

Repository

Branch

windows-escapes

Lint

Lint Passed

Unit

Test Failures

Build Status

Buildable 13997
Build 18143: arc lint + arc unit

Unit TestsFailed
View All

	Time	Test
	2,492 ms	ArcanistRepositoryAPIStateTestCase::Unknown Unit Message ("") EXCEPTION (CommandException): Command failed with error #2! COMMAND tar -C C:\Users\BYK\AppData\Local\Temp\21m4t71d5flwgs84/ -xzvvf C:\Users\BYK\AppData\Roaming\PortableApps\Arcanist\arcanist\src\repository\api\__tests__\state\git_basic.git.tgz
	209 ms	ArcanistLibraryTestCase::Unknown Unit Message ("") 1 assertion passed.
	59 ms	ArcanistLibraryTestCase::Unknown Unit Message ("") 1 assertion passed.
	54 ms	ArcanistLibraryTestCase::Unknown Unit Message ("") 1 assertion passed.
	2 ms	ArcanistRepositoryAPIMiscTestCase::Unknown Unit Message ("") 1 assertion passed.
		View Full Test Results (1 Failed · 4 Passed · 2 Skipped)

Event Timeline

BYK updated this revision to Diff 37814.Apr 13 2016, 6:11 AM

BYK retitled this revision from to Remove Windows workarounds from ArcanistGitAPI.

BYK updated this object.

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

BYK added a reviewer: epriestley.

BYK added a parent revision: D15675: Fix Windows escaping.

BYK added a subscriber: hach-que.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptApr 13 2016, 6:11 AM

Herald added a subscriber: Korvin. · View Herald Transcript

BYK mentioned this in D15675: Fix Windows escaping.Apr 15 2016, 9:09 AM

rebase

BYK mentioned this in T8298: Fix Windows Escaping.Jul 25 2016, 5:10 PM

rebase

rebase

• liugev6 added a commit: rP88a966993be1: Ajax commenting on lightbox images.Nov 22 2016, 8:51 AM

epriestley removed a commit: rP88a966993be1: Ajax commenting on lightbox images.Nov 22 2016, 12:09 PM

rebase

• TweikerManager added a commit: rP2604c5af55f6: (stable) Promote 2017 Week 4.Jan 29 2017, 10:50 PM

epriestley removed a commit: rP2604c5af55f6: (stable) Promote 2017 Week 4.Jan 29 2017, 10:52 PM

nickhutchinson added a subscriber: nickhutchinson.Mar 7 2017, 9:37 AM

jlbrown added a subscriber: jlbrown.Mar 29 2017, 2:08 PM

BYK abandoned this revision.Jan 8 2018, 3:03 PM

Revision Contents
Changeset List

Path

Size

src/

repository/

api/

ArcanistGitAPI.php

28 lines

Diff 40107

src/repository/api/ArcanistGitAPI.php

Loading...