Skip to content
This repository was archived by the owner on May 1, 2020. It is now read-only.

"ionic run --device --livelreload -c" should fail if ios-deploy did not finish successfully #334

Open
bastibense opened this issue Nov 9, 2016 · 3 comments · May be fixed by andypopa/ionic-app-scripts#5
Assignees

Comments

@bastibense
Copy link

bastibense commented Nov 9, 2016

Short description of the problem:

ionic run --device --livereload -c still begins to serve and watch even if the previous call to ios-deploy did not return successfully. I attached a snippet of my terminal so you can see the problem.

What behavior are you expecting?

If ios-deploy does not exit with a return code of 0, ionic-run should abort at that point and return an error code itself.

Steps to reproduce:

  1. run project on device using ionic run --device --livereload -c (ios device)
[ 65%] InstallingEmbeddedProfile

[ 70%] VerifyingApplication

[ 75%] CreatingContainer

[ 80%] InstallingApplication

[ 85%] PostflightingApplication

[ 90%] SandboxingApplication

[ 95%] GeneratingApplicationMap

[100%] Installed package /Users/bbense/Dropbox/devel/p4-app/platforms/ios/build/device/Leafled.app

------ Debug phase ------

Starting debug of J81AP 'IPad Air2' (xxxx) connected through USB...

2016-11-09 06:42:28.996 ios-deploy[80612:5705731] [ !! ] Unable to locate DeviceSupport directory. This probably means you don't have Xcode installed, you will need to launch the app manually and logging output will not be shown!

Error: Error code 253 for command: ios-deploy with args: --justlaunch,--no-wifi,-d,-b,/Users/bbense/Dropbox/devel/p4-app/platforms/ios/build/device/Leafled.app


Ionic server commands, enter:
  restart or r to restart the client app from the root
  goto or g and a url to have the app navigate to the given url
  consolelogs or c to enable/disable console log output
  serverlogs or s to enable/disable server log output
  quit or q to shutdown the server and exit

CSS changed:  www/build/main.css
JS changed:   www/build/main.js

Which @ionic/app-scripts version are you using?
@ionic/[email protected]

@amymarsh4work
Copy link

I'm having the same problem.

@danbucholtz
Copy link
Contributor

danbucholtz commented Jan 28, 2017

@jthoms1,

Let's chat about this one. I'm not sure if this is CLI or app-scripts where we need to make the change.

Thanks,
Dan

@tuanbs
Copy link

tuanbs commented Feb 2, 2017

I got the same issue. Any workaround please?

---- Updated ---
Hey, I solved it by restarting my iPhone and run the cmd again. It works like a charm. Thanks to @shazron at this https://github.com/phonegap/ios-deploy/issues/171

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants