From 2a5bb9d56e785e8b8f70c24621d469c473c9061a Mon Sep 17 00:00:00 2001 From: Max Holder Date: Fri, 20 Mar 2015 14:00:17 -0400 Subject: Print `bundle install` output in `rails new` as soon as it's available Previously, running `rails new` would not print any of the output from `bundle install` until all the gems had finished installing. This made it look like the generator was hanging at the `bundle install` step. This commit switches to using `system` so that the bundle command can output as it needs to. This has the added benefit of including output bundler produces on standard error, which the previous code ignored since backticks only capture standard out. This is not a big deal right now since bundler does not currently print errors to standard error, but that may change in the future (see: bundler/bundler/issues/3353). --- railties/CHANGELOG.md | 7 +++++++ 1 file changed, 7 insertions(+) (limited to 'railties/CHANGELOG.md') diff --git a/railties/CHANGELOG.md b/railties/CHANGELOG.md index 595a50dbdc..b4ad3be8f9 100644 --- a/railties/CHANGELOG.md +++ b/railties/CHANGELOG.md @@ -1,3 +1,10 @@ +* Print `bundle install` output in `rails new` as soon as it's available + + Running `rails new` will now print the output of `bundle install` as + it is available, instead of waiting until all gems finish installing. + + *Max Holder* + * Add a new-line to the end of route method generated code. We need to add a `\n`, because we cannot have two routes -- cgit v1.2.3