Closed
Bug 1473276
Opened 7 years ago
Closed 7 years ago
Replace `which` with `command -v` for modern bash use
Categories
(Release Engineering :: Release Automation, enhancement)
Release Engineering
Release Automation
Tracking
(firefox-esr60 fixed, firefox63 fixed)
RESOLVED
FIXED
People
(Reporter: sfraser, Assigned: sfraser)
References
Details
Attachments
(1 file)
$ ./mach lint -l shellcheck
.../taskcluster/docker/funsize-update-generator/scripts/mbsdiff_hook.sh
23:19 warning which is non-standard. Use builtin 'command -v' instead. 2230 (shellcheck)
|
Assignee | |
Updated•7 years ago
|
Assignee: nobody → sfraser
|
|
Assignee | |
Comment 1•7 years ago
|
||
This will start to cause an error when a newer shellcheck is available in CI
|
||
Comment 2•7 years ago
|
||
Comment on attachment 8989709 [details]
Bug 1473276 'which' is non-standard, use 'command -v' in partials r=mtabara
Mihai Tabara [:mtabara]⌚️GMT has approved the revision.
https://phabricator.services.mozilla.com/D1940
Attachment #8989709 -
Flags: review+
Pushed by sfraser@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/1e689eca8d98
'which' is non-standard, use 'command -v' in partials r=mtabara
|
||
Comment 4•7 years ago
|
||
| bugherder | ||
|
|
||
Comment 5•7 years ago
|
||
| bugherder | ||
|
||
Comment 6•7 years ago
|
||
| bugherder uplift | ||
status-firefox-esr60:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•