Merge pull request #2973 from balena-io/use-bash

Use bash instead of sh for running the elevated process on Linux and Mac
This commit is contained in:
Alexis Svinartchouk 2019-11-27 15:09:43 +01:00 committed by GitHub
commit 7050111bf4
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -148,7 +148,7 @@ const elevateScriptWindows = async (path) => {
} }
const elevateScriptUnix = async (path, name) => { const elevateScriptUnix = async (path, name) => {
const cmd = [ 'sh', escapeSh(path) ].join(' ') const cmd = [ 'bash', escapeSh(path) ].join(' ')
const [ , stderr ] = await sudoPrompt.execAsync(cmd, { name }) const [ , stderr ] = await sudoPrompt.execAsync(cmd, { name })
if (!_.isEmpty(stderr)) { if (!_.isEmpty(stderr)) {
throw errors.createError({ title: stderr }) throw errors.createError({ title: stderr })
@ -157,7 +157,7 @@ const elevateScriptUnix = async (path, name) => {
} }
const elevateScriptCatalina = async (path) => { const elevateScriptCatalina = async (path) => {
const cmd = [ 'sh', escapeSh(path) ].join(' ') const cmd = [ 'bash', escapeSh(path) ].join(' ')
try { try {
const { cancelled } = await catalinaSudo(cmd) const { cancelled } = await catalinaSudo(cmd)
return { cancelled } return { cancelled }