Gemfile'a ruby versiyonu ekle #219
Conversation
ecylmz
requested review from
dilara,
ecmelkytz,
huseyin,
isubas and
sinansh
as code owners
| @@ -1,6 +1,7 @@ | |||
| # frozen_string_literal: true | |||
|
|
|||
| source 'https://rubygems.org' | |||
| ruby ENV['CUSTOM_RUBY_VERSION'] || '2.5.1' | |||
There was a problem hiding this comment.
Bu düzenleme patchlevel'ları yönetiyor mu? Örneğin şu an benim makinede durum:
$ ruby -v
ruby 2.5.1p57omu imajlarında da aynı sürüm ve patchlevel (p57) kurulu. Bunu deneyebildin mi? Patchlevel'dan rahatsız olursa bir versiyon operatörü kullanmamız gerekecek, o yüzden soruyorum.
There was a problem hiding this comment.
Bu düzenleme patchlevel'ları yönetemiyor. Gemfile'da ruby versiyonu aşağıdaki formatta veriliyormuş:
ruby 'RUBY_VERSION', :engine => 'ENGINE', :engine_version => 'ENGINE_VERSION',
:patchlevel => 'RUBY_PATCHLEVEL'There was a problem hiding this comment.
:patchlevel seçeneği verilirse ortamdan gelen versiyonla karışacak bu sefer. Aşağıdaki düzenleme çalışacaktır sanırım: 2.5.1 sürümüne eşit veya büyük
ruby ENV['CUSTOM_RUBY_VERSION'] || '~> 2.5.1'There was a problem hiding this comment.
omu imajlarında da aynı sürüm ve patchlevel (p57) kurulu. Bunu deneyebildin mi?
buildpack de p57 kuruyor. Heroku tarafından(yani heroku buildpack'ler tarafından) desteklenen versiyonlar: https://devcenter.heroku.com/articles/ruby-support#supported-runtimes
There was a problem hiding this comment.
Yani yönetiyor (hata vermiyor)? Yukarıda yönetemiyor dediğinden yazdım. :-) Desteklenen sürümleri biliyorum, tek emin olamadığım Gemfile'daki ruby '2.5.1' versiyon bildirimi ruby 2.5.1p57 sürümlü klasik dev ortamında ve Dokku'da şikayetsiz çalışıyor mu?
Edit: s/veya/ve/
There was a problem hiding this comment.
Vagrant'ta deneyerek, sorumun cevabını aldım. ruby '2.5.1' versiyon bildirimi patchlevel'ları da kapsıyor. Örneğin bundle 2.5.1p57'yi 2.5.1.57 gibi görüyor ve dördüncü seviyeyi (patch level) dikkate almıyor. Geçmişte bir ara bundle'dan beklemediğim bir hata almıştım o yüzden kuşkulandım.
Yönetme konusunda derdimi tam anlatamadım, kusura bakma. Bu durumda sorun kalmadı. PR'ı alabiliriz.
There was a problem hiding this comment.
Yani yönetiyor (hata vermiyor)? Yukarıda yönetemiyor dediğinden yazdım. :-)
Aslında şu yazımı yönetemiyor demek istemiştim, ruby '2.5.1-p57' ama açık bir şekilde belirtmemişim :(
ruby '2.5.1' versiyon bildirimi, hem dev ortamında hem de dokku'da sorunsuz çalışıyor, her iki ortamda da p57 geliyor.
|
@ecylmz @roktas Merge ettikten sonra bir şey farkettim, Ruby'yi rbenv ile birlikte kullanıyorum, rbenv ise her bir projenin sürüm bilgisini anlamak için rbenv local 2.5.1dediğimde gidip kendisi bir .ruby-version dosyası oluşturuyor. Aslında proje kökünde @ecylmz Hem .ruby-version dosyasını tutsak, hem de Gemfile değiştirdiğin şekilde kalsa ruby buildpack'i nasıl davranıyor? |
|
@msdundar iki yerde versiyon bilgisinin tutulması doğru değil, otoriter tek bir yer olmalı. Bu konuda şu düşüncelere katılırım: heroku/heroku-buildpack-ruby#63 (comment) @ecylmz: Tooling'i dikkate almıştık ama rbenv, rvm vb araçları (hiç birisini sevmiyorum :-P) atlamışız. Bu durumda diğer planı uygulayacağız ( |
Bu PR'in yaptığı işi/değişikliği ve bu işi/değişikliği neden yaptığını açıklayın
Ruby buildpack'i, sistemde kullanılacak ruby versiyonunu Gemfile'dan okumaktadır. Ön tanımlı ruby versiyonu eski bir ruby versiyonu(2.3.1) olduğundan, bize ise daha yeni bir ruby versiyonu(2.5.1) gerektiğinden Gemfile'a ruby versiyonu eklenmelidir.
İlgili iş kayıtları
#216
Kapatılacak iş kayıtları
fixes #216
Veritabanı
Etkilenmeyecektir.
Sistem/Ops
İşlem gerekmemektedir.