Paths

Table of Contentst

arcanist: arc should work with any POSIX shell
AbandonedPublic
Actions

Authored by rsmarples on Mar 30 2017, 11:55 AM.

Tags

None

Referenced Files

	F13204389: D17582.diff
	Wed, May 15, 12:32 AM

	F13186736: D17582.diff
	Sat, May 11, 3:54 AM

	Unknown Object (File)
	Tue, May 7, 7:05 AM

	Unknown Object (File)
	Mon, May 6, 6:25 PM

	Unknown Object (File)
	Fri, May 3, 4:28 AM

	Unknown Object (File)
	Mon, Apr 29, 4:34 PM

	Unknown Object (File)
	Thu, Apr 25, 12:34 AM

	Unknown Object (File)
	Mon, Apr 22, 9:38 PM

View All 85 Files

Subscribers

epriestley

Details

Reviewers

None

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T12477: arc wrapper should work with any POSIX shell

Summary

Currently arc requires bash for $BASH_SOURCE[0] and
substring range removal, both of which are bash specific features.

This patch gets the same result with $0 and string prefix removal,
both of are are POSIX features and as such all shells should support.

Fixes T12477.

Test Plan

Run bin/arc on systems with bash and other shells as /bin/sh

Diff Detail

Repository

rARC Arcanist

Branch

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 16223
Build 21546: arc lint + arc unit

Event Timeline

rsmarples created this revision.Mar 30 2017, 11:55 AM

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptMar 30 2017, 11:55 AM

Herald added a subscriber: epriestley. · View Herald Transcript

Abandoning this since T12477 wasn't accepted upstream.

Revision Contents
Changeset List

Path

Size

bin/

arc

10 lines

Diff 42297

View Options

arcanist: arc should work with any POSIX shellAbandonedPublicActions