From ac1d344368ee532011dc0c2093ea455ad9aaa5a1 Mon Sep 17 00:00:00 2001
From: David Taylor <david@taylorhq.com>
Date: Wed, 5 May 2021 13:33:11 +0100
Subject: [PATCH] DEV: Update server startup warning to be more specific
 (#12949)

Using `bundle exec` will slow down server startup by at least 0.5s. `bin/unicorn` has built-in handling of bundler dependencies, so it is better to launch `bin/rails s` or `bin/unicorn` directly.
---
 bin/rails   | 11 -----------
 bin/unicorn |  6 ++++++
 2 files changed, 6 insertions(+), 11 deletions(-)

diff --git a/bin/rails b/bin/rails
index f95c5282d1..595d718419 100755
--- a/bin/rails
+++ b/bin/rails
@@ -10,17 +10,6 @@ if !ENV["RAILS_ENV"] && (ARGV[0] == "s" || ARGV[0] == "server")
 
   ENV["RAILS_LOGS_STDOUT"] ||= "1"
 
-  STDERR.puts <<~MESSAGE
-    --------
-    WARNING: Discourse uses `bin/unicorn` to start the web server.
-    For backwards compatibility, `rails s` will do this automatically.
-    For improved performance you should run `bin/unicorn` directly.
-
-    Running:
-    UNICORN_PORT=#{ENV["UNICORN_PORT"]} RAILS_LOGS_STDOUT=#{ENV["RAILS_LOGS_STDOUT"]} bin/unicorn
-    --------
-  MESSAGE
-
   exec File.expand_path("unicorn", __dir__)
 end
 
diff --git a/bin/unicorn b/bin/unicorn
index 6437aced5e..106649c75f 100755
--- a/bin/unicorn
+++ b/bin/unicorn
@@ -6,6 +6,12 @@ ENV['BUNDLE_GEMFILE'] ||= File.expand_path("../../Gemfile",
   Pathname.new(__FILE__).realpath)
 RAILS_ROOT = File.expand_path("../../", Pathname.new(__FILE__).realpath)
 
+if defined? Bundler
+  STDERR.puts <<~MESSAGE
+    WARNING: Using `bundle exec` to start the server is unnecessary, and will make startup slower. Use `bin/rails s` or `bin/unicorn`.
+  MESSAGE
+end
+
 require 'rubygems'
 require 'bundler/setup'
 require 'digest'