Previously, all progress display was turned off after the app’s initial start, so no progress would be displayed on restart due to server code change.
Now, we enable/disable progress display from inside the loop that re-runs the app on change.
Remove the “suppressDisplay” approach to hiding the spinner when there’s an error building the app, because it didn’t work on rebuilds, only the first build. Better to just call enableProgressDisplay(false) and hope for the best like we do elsewhere.
Conflicts:
tools/progress.js
Previously, if there were two parallel jobs like “Starting your app” and “Bundling app,” we’d display just a spinner and no message. Now, we choose a message and display it.
(…due to a server-side code change)
Restarting the server can take a while (30 seconds for one particular app), so it’s nice to at least print when we start doing it. Soon there will be progress displays during it, too (when we add more of them to `meteor run`).
We lose the opportunity to coalesce server restarted messages, as in `Meteor server restarted (x2)`, but adding any sort of progress display also does that.
We intentionally consider a machine without HAXM to be "acceptable"
since HAXM can't be installed on VMs. But we should still let the users
know how to install if if that's what they're missing.
We don't have a way to HCP a changed set of plugins or platforms, so
just exit and ask the user to re-run. There are probably some cases
where we don't have to exit (e.g. if you add the android platform, we
probably don't need to exit out of 'meteor run ios'), but we exit in all
cases for simplicity.
Now we see something like:
$ curmeteor
Unable to refresh catalog (are you offline?)
Could not download package iron:dynamic-template@0.4.1: getaddrinfo ENOTFOUND
Could not download package iron:router@0.9.4: getaddrinfo ENOTFOUND
Could not download package iron:layout@0.4.1: getaddrinfo ENOTFOUND
Could not download package iron:core@0.3.4: getaddrinfo ENOTFOUND
Could not install all the requested packages.
The specific issue is the "Could not springboard to release..." message
showing up on the same line as a progress bar. Seems like a good idea to
use Console everywhere in this file anyway.
Instead of "Meteor METEOR@1.0", print "Meteor 1.0".
This change is somewhat untested because in order to test it we need
1. a built release, and 2. installed 'meteor' to point to that built
release.
