Merge pull request #132 from fly-apps/ffi

Search for ffi in the gemfile
fly-apps · Jan 14, 2025 · 96b88b6 · 96b88b6
2 parents ea8f996 + 7e3a8a1
commit 96b88b6
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/dockerfile-rails.gemspec b/dockerfile-rails.gemspec
@@ -2,7 +2,7 @@
 
 Gem::Specification.new do |spec|
   spec.name        = "dockerfile-rails"
-  spec.version     = "1.7.1"
+  spec.version     = "1.7.2"
   spec.authors     = [
     "Sam Ruby",
   ]

diff --git a/lib/generators/dockerfile_generator.rb b/lib/generators/dockerfile_generator.rb
@@ -741,7 +741,7 @@ def build_packages
     packages = %w(build-essential libyaml-dev)
     packages += @@packages["build"] if @@packages["build"]
     packages += %w(nodejs npm) if (node_version == "lts") && (not using_execjs?)
-    packages << "libffi-dev" if  @@packages["ffi"]
+    packages << "libffi-dev" if @gemfile.include?("ffi")
 
     # add databases: sqlite3, postgres, mysql
     packages << "pkg-config" if options.sqlite3? || @sqlite3