From 2b55d332b435c5710d9068e8d15e8f8751e756c3 Mon Sep 17 00:00:00 2001 From: Oliver Salzburg Date: Wed, 19 Dec 2018 12:39:09 +0100 Subject: [PATCH] emit xcodebuild & other spawned CLI commands (#479) --- bin/templates/scripts/cordova/lib/Podfile.js | 1 + bin/templates/scripts/cordova/lib/build.js | 4 ++-- bin/templates/scripts/cordova/lib/clean.js | 4 ++-- bin/templates/scripts/cordova/lib/run.js | 8 ++++---- 4 files changed, 9 insertions(+), 8 deletions(-) diff --git a/bin/templates/scripts/cordova/lib/Podfile.js b/bin/templates/scripts/cordova/lib/Podfile.js index 2e946f7a5..5cfdbb448 100644 --- a/bin/templates/scripts/cordova/lib/Podfile.js +++ b/bin/templates/scripts/cordova/lib/Podfile.js @@ -385,6 +385,7 @@ Podfile.prototype.install = function (requirementsCheckerFunction) { var opts = {}; opts.cwd = path.join(this.path, '..'); // parent path of this Podfile opts.stdio = 'pipe'; + opts.printCommand = true; var first = true; var self = this; diff --git a/bin/templates/scripts/cordova/lib/build.js b/bin/templates/scripts/cordova/lib/build.js index 4f28a262b..e63d57510 100644 --- a/bin/templates/scripts/cordova/lib/build.js +++ b/bin/templates/scripts/cordova/lib/build.js @@ -171,7 +171,7 @@ module.exports.run = function (buildOpts) { shell.rm('-rf', buildOutputDir); var xcodebuildArgs = getXcodeBuildArgs(projectName, projectPath, configuration, buildOpts.device, buildOpts.buildFlag, emulatorTarget, buildOpts.automaticProvisioning); - return superspawn.spawn('xcodebuild', xcodebuildArgs, { cwd: projectPath }); + return superspawn.spawn('xcodebuild', xcodebuildArgs, { cwd: projectPath, printCommand: true }); }).then(function () { if (!buildOpts.device || buildOpts.noSign) { @@ -224,7 +224,7 @@ module.exports.run = function (buildOpts) { function packageArchive () { var xcodearchiveArgs = getXcodeArchiveArgs(projectName, projectPath, buildOutputDir, exportOptionsPath, buildOpts.automaticProvisioning); - return superspawn.spawn('xcodebuild', xcodearchiveArgs, { cwd: projectPath }); + return superspawn.spawn('xcodebuild', xcodearchiveArgs, { cwd: projectPath, printCommand: true }); } return Q.nfcall(fs.writeFile, exportOptionsPath, exportOptionsPlist, 'utf-8') diff --git a/bin/templates/scripts/cordova/lib/clean.js b/bin/templates/scripts/cordova/lib/clean.js index 3e4d4140e..cbac5c120 100644 --- a/bin/templates/scripts/cordova/lib/clean.js +++ b/bin/templates/scripts/cordova/lib/clean.js @@ -33,9 +33,9 @@ module.exports.run = function () { return Q.reject('No Xcode project found in ' + projectPath); } - return superspawn.spawn('xcodebuild', ['-project', projectName, '-configuration', 'Debug', '-alltargets', 'clean'], { cwd: projectPath }) + return superspawn.spawn('xcodebuild', ['-project', projectName, '-configuration', 'Debug', '-alltargets', 'clean'], { cwd: projectPath, printCommand: true }) .then(function () { - return superspawn.spawn('xcodebuild', ['-project', projectName, '-configuration', 'Release', '-alltargets', 'clean'], { cwd: projectPath }); + return superspawn.spawn('xcodebuild', ['-project', projectName, '-configuration', 'Release', '-alltargets', 'clean'], { cwd: projectPath, printCommand: true }); }).then(function () { return shell.rm('-rf', path.join(projectPath, 'build')); }); diff --git a/bin/templates/scripts/cordova/lib/run.js b/bin/templates/scripts/cordova/lib/run.js index 8b8477759..28a2b7af4 100644 --- a/bin/templates/scripts/cordova/lib/run.js +++ b/bin/templates/scripts/cordova/lib/run.js @@ -79,7 +79,7 @@ module.exports.run = function (runOptions) { var ipafile = path.join(buildOutputDir, projectName + '.ipa'); // unpack the existing platform/ios/build/device/appname.ipa (zipfile), will create a Payload folder - return superspawn.spawn('unzip', [ '-o', '-qq', ipafile ], { cwd: buildOutputDir }); + return superspawn.spawn('unzip', [ '-o', '-qq', ipafile ], { cwd: buildOutputDir, printCommand: true }); }) .then(function () { // Uncompress IPA (zip file) @@ -148,7 +148,7 @@ function filterSupportedArgs (args) { * @return {Promise} Fullfilled when any device is connected, rejected otherwise */ function checkDeviceConnected () { - return superspawn.spawn('ios-deploy', ['-c', '-t', '1']); + return superspawn.spawn('ios-deploy', ['-c', '-t', '1'], { printCommand: true }); } /** @@ -160,9 +160,9 @@ function checkDeviceConnected () { function deployToDevice (appPath, target, extraArgs) { // Deploying to device... if (target) { - return superspawn.spawn('ios-deploy', ['--justlaunch', '-d', '-b', appPath, '-i', target].concat(extraArgs)); + return superspawn.spawn('ios-deploy', ['--justlaunch', '-d', '-b', appPath, '-i', target].concat(extraArgs), { printCommand: true }); } else { - return superspawn.spawn('ios-deploy', ['--justlaunch', '--no-wifi', '-d', '-b', appPath].concat(extraArgs)); + return superspawn.spawn('ios-deploy', ['--justlaunch', '--no-wifi', '-d', '-b', appPath].concat(extraArgs), { printCommand: true }); } }