Differential D15693

Remove Windows workarounds from ArcanistGitAPI
AbandonedPublic
Actions

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

Tags

None

Referenced Files

	F15420739: D15693.diff
	Fri, Mar 21, 4:39 PM

	F15411802: D15693.diff
	Wed, Mar 19, 9:51 AM

	F15395855: D15693.diff
	Sun, Mar 16, 10:10 AM

	F15381610: D15693.diff
	Fri, Mar 14, 8:35 AM

	F15381217: D15693.id38849.diff
	Fri, Mar 14, 6:32 AM

	F15373299: D15693.diff
	Wed, Mar 12, 3:17 PM

	F15344784: D15693.diff
	Mon, Mar 10, 6:42 AM

	F15330741: D15693.diff
	Fri, Mar 7, 7:59 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 14213
Build 18474: arc lint + arc unit

Unit TestsFailed
View All

	Time	Test
	649 ms	ArcanistRepositoryAPIStateTestCase::Unknown Unit Message ("") EXCEPTION (CommandException): Command failed with error #2! COMMAND tar -C C:\Users\BYK\AppData\Local\Temp\81z7ox87yn8kks84/ -xzvvf C:\Users\BYK\AppData\Roaming\PortableApps\Arcanist\arcanist\src\repository\api\__tests__\state\git_basic.git.tgz
	98 ms	ArcanistLibraryTestCase::Unknown Unit Message ("") 1 assertion passed.
	268 ms	ArcanistLibraryTestCase::Unknown Unit Message ("") 1 assertion passed.
	86 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 40321

src/repository/api/ArcanistGitAPI.php

Loading...