fly-apps / dockerfile-rails

Provides a Rails generator to produce Dockerfiles and related files.
MIT License
455 stars 38 forks source link

Add supporting freetds lib for required tiny_tds driver with SQL Server #64

Closed ttilberg closed 9 months ago

ttilberg commented 9 months ago

Adds required TDS drivers when sqlserver detected.

Continues: https://github.com/fly-apps/dockerfile-rails/commit/40a1f9ba39682f1c6763fff9983acd03fbfcff30 Finishes #61

ttilberg commented 9 months ago

The other DB drivers appeared to add the supporting gem in some cases, I wasn’t sure what that was for, as I’d think the Gemfile would need to have whatever db drivers required already. I didn’t add addition for sql server because I didn’t understand the intent. Feel free to add it to this PR if it makes sense. I figured this was a very usable minimum for support though and wanted to keep it simple without more info.