Differential D1434

Correctly expand shell arguments
ClosedPublic
Actions

Authored by dereckson on Mar 16 2018, 03:15.

Tags

None

Referenced Files

	F7979073: D1434.id3663.diff
	Tue, May 6, 05:09

	F7979055: D1434.id3664.diff
	Tue, May 6, 05:09

	Unknown Object (File)
	Sun, May 4, 16:14

	Unknown Object (File)
	Mon, Apr 28, 10:26

	Unknown Object (File)
	Sun, Apr 27, 14:25

	Unknown Object (File)
	Thu, Apr 24, 01:19

	Unknown Object (File)
	Wed, Apr 23, 04:28

	Unknown Object (File)
	Tue, Apr 22, 20:14

Subscribers

None

Details

Reviewers

Commits

rOPSbf659c5728cb: Correctly expand shell arguments

Summary

$* → "$@"

Reference: https://github.com/koalaman/shellcheck/wiki/SC2048

Test Plan

Tested on Equatower

Diff Detail

Repository

rOPS Nasqueron Operations

Lint

Lint Passed

Severity	Location	Code	Message
Advice	roles/phabricator/arcanist/files/arc.sh:67	SC2086	SHELLCHECK
Advice	roles/phabricator/arcanist/files/arc.sh:67	SC2086	SHELLCHECK
Advice	roles/phabricator/arcanist/files/arc.sh:69	SC2086	SHELLCHECK
Advice	roles/phabricator/arcanist/files/arc.sh:69	SC2086	SHELLCHECK

Unit

No Test Coverage

Branch

star-returns (branched from master)

Build Status

Buildable 2253
Build 2501: arc lint + arc unit

Event Timeline

dereckson requested review of this revision.Mar 16 2018, 03:15

dereckson created this revision.

Harbormaster completed remote builds in B2252: Diff 3662.Mar 16 2018, 03:15

arc too

Harbormaster completed remote builds in B2253: Diff 3663.Mar 16 2018, 03:16

Advices are for other arguments (yes this script should be checked)

This revision is now accepted and ready to land.Mar 16 2018, 03:18

Closed by commit rOPSbf659c5728cb: Correctly expand shell arguments (authored by dereckson). · Explain WhyMar 16 2018, 03:19

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

roles/

paas-docker/

wrappers/

files/

2 lines

phabricator/

arcanist/

files/

4 lines

Diff 3663

roles/paas-docker/wrappers/files/certbot.sh

Loading...

roles/phabricator/arcanist/files/arc.sh

Loading...