Paths

Table of Contentst

Modernize editor selector
ClosedPublic
Actions

Authored by avivey on Jun 19 2017, 10:53 PM.

Tags

None

Referenced Files

	F18778274: D18141.id43644.diff
	Sat, Oct 11, 8:46 AM

	F18769892: D18141.id43645.diff
	Wed, Oct 8, 9:46 AM

	F18733648: D18141.id.diff
	Tue, Sep 30, 10:39 PM

	F18620804: D18141.id.diff
	Sep 15 2025, 6:21 AM

	F18604322: D18141.id43645.diff
	Sep 13 2025, 6:31 PM

	F18600724: D18141.diff
	Sep 13 2025, 10:26 AM

	F18600275: D18141.diff
	Sep 13 2025, 9:24 AM

	F18590745: D18141.diff
	Sep 12 2025, 7:02 AM

View All Files

Subscribers

Korvin

Details

Reviewers

epriestley
amckinley
kdhp

Group Reviewers

Blessed Reviewers

Commits

rPHU900873b4a4c5: Modernize editor selector

Summary

FreeBSD doesn't have nano by default, so allow falling back to vi. Also added sensible-editor.
This mostly retains the order we select things, but reduces some lines.

Test Plan

arc diff

Diff Detail

Repository

rPHU libphutil

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

avivey created this revision.Jun 19 2017, 10:53 PM

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJun 19 2017, 10:53 PM

Herald added a subscriber: Korvin. · View Herald Transcript

Harbormaster completed remote builds in B17536: Diff 43644.Jun 19 2017, 10:53 PM

avivey mentioned this in Z1336: General Chat.Jun 19 2017, 10:59 PM

kdhp accepted this revision.Jun 19 2017, 11:10 PM

epriestley accepted this revision.Jun 20 2017, 12:06 AM

This revision is now accepted and ready to land.Jun 20 2017, 12:06 AM

Closed by commit rPHU900873b4a4c5: Modernize editor selector (authored by avivey, committed by avivey). · Explain WhyJun 20 2017, 12:18 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

console/

PhutilInteractiveEditor.php

14 lines

Diff 43645

View Options

src/console/PhutilInteractiveEditor.php

Modernize editor selectorClosedPublicActions