mirror of
https://github.com/clusterio/clusterio.git
synced 2026-11-12 07:04:52 +09:00
Remove unneeded comments
This commit is contained in:
2 files changed
-7
No files matched your search
@@ -118,10 +118,6 @@ async function safeOutputFile(file, data, options={}) {
|
||||
}
|
||||
|
||||
async function execFile(cmd, args) {
|
||||
// Arguments are escaped here and joined into the command rather than
|
||||
// passed to execFile. With shell: true it would only concatenate them by
|
||||
// space without escaping anything, which is what it is deprecated for
|
||||
// (DEP0190) as it gives the impression the arguments are made safe.
|
||||
const command = [cmd, ...args.map(escapeArg)].join(" ");
|
||||
logger.verbose(`executing ${command}`);
|
||||
return new Promise((resolve, reject) => {
|
||||
|
||||
@@ -8,9 +8,6 @@ const { escapeArg } = require("../../packages/create/escape_arg");
|
||||
const execFile = util.promisify(child_process.execFile);
|
||||
|
||||
async function exec(file, args) {
|
||||
// Escape the arguments and join them into the command the same way
|
||||
// create.js does, rather than passing them to execFile which would only
|
||||
// concatenate them by space without escaping (DEP0190).
|
||||
const command = [file, ...args.map(escapeArg)].join(" ");
|
||||
const { stdout, stderr } = await execFile(
|
||||
command,
|
||||
|
||||
Reference in new issue
Block a user