microsoft / vsts-cordova-tasks

Streamline CI setup for your Apache Cordova, PhoneGap, Ionic, or Cordova CLI compatible app using a set of useful pre-defined build steps for VS Team Services or TFS
http://go.microsoft.com/fwlink/?LinkID=691188
Other
25 stars 27 forks source link

Warning reported when installing Cordova 5.4.1 or PhoneGap 5.3.10 causes build to fail incorrectly with the VSTS Hosted pool #17

Closed Chuxel closed 8 years ago

Chuxel commented 8 years ago

For some reason a warning reported out in the Hosted pool when installing PhoneGap CLI 5.3.10 causes the build to fail even though the command ultimately succeeds.

Set workingFolder to default: C:\LR\MMS\Services\Mms\TaskAgentProvisioner\Tools\agents\default\tasks\PhoneGapCommand\1.0.0
Prepare node module: vso-task-lib version: Latest Version for task script at workingFolder: C:\LR\MMS\Services\Mms\TaskAgentProvisioner\Tools\agents\default\tasks\PhoneGapCommand\1.0.0\node_modules\vso-task-lib.
Create junction point from C:\LR\MMS\Services\Mms\TaskAgentProvisioner\Tools\agents\default\tasks\PhoneGapCommand\1.0.0\node_modules\vso-task-lib to C:\LR\MMS\Services\Mms\TaskAgentProvisioner\Tools\agents\default\modules\vso-task-lib\0.4.5.
Module cache at C:\cordova\node_modules
Installing phonegap@5.3.10 to C:\cordova\node_modules\phonegap\5.3.10. (This may take a few minutes.)
Exec complete.

npm WARN deprecated npmconf@2.1.2: this package has been reintegrated into npm and is now out of date with respect to npm

npm WARN deprecated lodash@1.0.2: lodash@<2.0.0 is no longer maintained. Upgrade to lodash@^3.0.0

...

Γö£ΓöÇΓöÇ colors@0.6.0-1
Γö£ΓöÇΓöÇ semver@1.1.0
Γö£ΓöÇΓöÇ minimist@0.1.0
Γö£ΓöÇΓöÇ qrcode-terminal@0.9.4
Γö£ΓöÇΓöÇ shelljs@0.1.4
Γö£ΓöÇΓöÇ prompt@0.2.11 (revalidator@0.1.8, pkginfo@0.3.1, read@1.0.7, utile@0.2.1, winston@0.6.2)
Γö£ΓöÇΓöÇ phonegap-build@0.9.2 (colors@0.6.2, qrcode-terminal@0.8.0, shelljs@0.0.9, optimist@0.3.7, phonegap-build-api@0.3.3)
Γö£ΓöÇΓöÇ cordova@5.4.1 (ansi@0.3.0, underscore@1.7.0, q@1.0.1, nopt@3.0.1, update-notifier@0.5.0, cordova-lib@5.4.1)
ΓööΓöÇΓöÇ connect-phonegap@0.18.2 (home-dir@0.1.2, connect-inject@0.3.2, ip@0.3.1, request-progress@0.3.1, walkdir@0.0.8, http-proxy@1.8.1, adm-zip@0.4.7, shelljs@0.2.6, request@2.33.0, tar@0.1.19, gaze@0.4.3, node-static@0.7.0, localtunnel@1.3.0, useragent@2.0.8, archiver@0.14.3, connect@2.12.0, socket.io@1.0.4)

C:\cordova\node_modules\phonegap\5.3.10\node_modules\.bin\phonegap.cmd remote login -u chuck_lantz_dev@hotmail.com -p ********
[phonegap] logged in as chuck_lantz_dev@hotmail.com

******************************************************************************

Finishing task: PhoneGapCommand

******************************************************************************

Task PhoneGapCommand failed. This caused the job to fail. Look at the logs for the task for more details.

Using a cached version of PhoneGap Build does not cause the same issue to occur.

Chuxel commented 8 years ago

This appears to be due to VSTS switching from running PowerShell scripts to node Scripts in the Windows Hosted agent. Other warnings that are bubbled up do not cause a failure so it's unclear why this particular situation is causing a problem.

Chuxel commented 8 years ago

Fixed by PR 18