Skip to content

Commit a124668

Browse files
authored
Merge pull request rails#47513 from rails/redis-is-not-needed
Dockerfile don't install the redis package
2 parents 5d7b6d8 + 270cfb9 commit a124668

File tree

1 file changed

+0
-6
lines changed

1 file changed

+0
-6
lines changed

railties/lib/rails/generators/app_base.rb

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -506,9 +506,6 @@ def dockerfile_build_packages
506506
# add databases: sqlite3, postgres, mysql
507507
packages += %w(pkg-config libpq-dev default-libmysqlclient-dev)
508508

509-
# add redis in case Action Cable, caching, or sidekiq are added later
510-
packages << "redis"
511-
512509
# ActiveStorage preview support
513510
packages << "libvips" unless skip_active_storage?
514511

@@ -543,9 +540,6 @@ def dockerfile_deploy_packages
543540
# start with databases: sqlite3, postgres, mysql
544541
packages = %w(libsqlite3-0 postgresql-client default-mysql-client)
545542

546-
# add redis in case Action Cable, caching, or sidekiq are added later
547-
packages << "redis"
548-
549543
# ActiveStorage preview support
550544
packages << "libvips" unless skip_active_storage?
551545

0 commit comments

Comments
 (0)