Should be basically backwards compatible (they both join on spaces) but
has the following advantages:
- Objects are printed with util.inspect rather than toString, which
means no more [object Object]. (We shouldn't be passing objects to
end-user messages, but they can be useful in Console.debug.)
- The first string can optionally be a printf-style format string, if
that's a better fit for your message.
unnecessary: because dontRunConstraintSolver now means we don't need to
be in a buildmessage.capture
dangerous: because if you don't check the result of
buildmessage.capture, you leak errors.
- Errors in merging the cordova branch against 273b70bea4 left
in an unnecessary getLocalPackages function
- LocalCatalog.getLocalPackageNames is unnecessary (it's the same as
getAllPackageNames!) CompleteCatalog.getLocalPackageNames is OK I
guess.
- getPackagesForTest returned some extra data that was only used to call
the nonexistent catalog.removeLocalPackage method. Meh, we don't
clean up the catalog when the command ends now (which doesn't happens
unless you're using --once anyway). We'll probably fix this by making
catalog.complete less global soon anyway (ie, it should be build-specific).
- Fix a duplicate test runner app cp_r
Bug introduced in 0.9.4. This only seems to occur in some contexts; a
way to consistently reproduce is:
$ rm -rf packages/test-in-browser/.build.test-in-browser && curmeteor test-packages
[[[[[ Tests ]]]]]
=> Started proxy.
=> Started MongoDB.
=> Errors prevented startup:
While Building package `test-in-browser`:
error: package not available: bootstrap
- Simplify internal data structure to just have one map of name ->
various data instead of a whole bunch of different unsynchronized data
structures (most of which were used with frequent O(n) operations).
- Move the getLoadPathForPackage logic which combines local and remote
packages into LayeredCatalog from LocalCatalog, and delete the
unnecessary copy in BootstrapCatalogCheckout.
- Rename a bunch of fields to make it explicit which ones contain
package objects, which ones contain package directories, and which
ones contain package *search* directories
- Replace random version IDs and a long comment about why random is safe
with sequential. (I don't think these version IDs are ever used
anyway.)
- Drop unused "initializing" option to refresh
- Drop redundant call to _recomputeEffectiveLocalPackages in
addLocalPackage (it is called immediately below by refresh).
