Skip to content

Commit 9fd268a

Browse files
authored
Merge pull request #144 from fly-apps/libyaml-no-longer-required
Docker image change was reverted - libyaml is back in
2 parents d035c64 + 333d7f1 commit 9fd268a

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

lib/generators/dockerfile_generator.rb

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -410,6 +410,11 @@ def generate_app
410410
missing = Set.new(base_packages + build_packages) -
411411
Set.new(dockerfile.scan(/[-\w]+/))
412412

413+
# https://github.com/docker-library/ruby/pull/497
414+
# https://github.com/rails/rails/pull/54237
415+
missing.delete("libyaml-dev")
416+
missing.delete("yaml-dev")
417+
413418
unless missing.empty?
414419
message = "The following packages are missing from the Dockerfile: #{missing.to_a.join(", ")}"
415420
STDERR.puts "\n" + shell.set_color(message, Thor::Shell::Color::RED, Thor::Shell::Color::BOLD)

0 commit comments

Comments
 (0)