Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

'rails db:migrate' is recommended on Rails 5 #203

Merged
merged 1 commit into from
Jul 17, 2016

Conversation

kei-s
Copy link
Contributor

@kei-s kei-s commented Jul 10, 2016

No description provided.

@kei-s
Copy link
Contributor Author

kei-s commented Jul 10, 2016

Windows でも rails db:migrate で行けると思うのですが、確認できてません。その場合 os-specific を外して良さそうです

@kei-s kei-s force-pushed the rails_db_migrate branch from b1355de to 2fed50c Compare July 10, 2016 07:00
@hsbt
Copy link
Member

hsbt commented Jul 10, 2016

どなたか Windows + RailsInstaller で確認できます? @yuumi3 さんあたりご存じないかな...

@igaiga
Copy link
Contributor

igaiga commented Jul 10, 2016

Windows、Rails5を試そうと思ったのですが、インストールでつまづいて途方にくれてます・・・。

  • Rails5にはRuby2.2.2+が必要
  • RailsInstallerのガイド該当バージョン(3.1.0)はRuby2.1系
  • RailsInstallerのRuby2.2系(バージョン3.2.0)はインストーラにバグがある

@yuumi3
Copy link
Contributor

yuumi3 commented Jul 10, 2016

最近Windowsでという案件が減っているので、フォローしていませんでした。 RubyInstallerを使えば行けるかな・・・ 後で試してみます。

Windowsの人は4.2.xで良いのではないでしょうか?

@yuumi3
Copy link
Contributor

yuumi3 commented Jul 10, 2016

Railsinstallerのruby 2.2.4 + Rails5.0 動きました! ruby 2.3.0 はsqlite3が対応出来てないようです。 手順はまた後で

@yuumi3
Copy link
Contributor

yuumi3 commented Jul 11, 2016

手順は https://redmine.ey-office.net/projects/rails/wiki/InstallRailsWindows を参考にしてください。

Rails Girls では git をインストールしないといけないですよね。

@amatsuda
Copy link
Member

インストーラにバグがある

の詳細がわからなすぎて建設的な話にならない流れで残念だし判断材料に欠ける感じなんですが、どういう問題なんでしょうか?
五十嵐さんと思しき人物がなんかコメントしてるからこれですか?
railsinstaller/railsinstaller-windows#76

@igaiga
Copy link
Contributor

igaiga commented Jul 13, 2016

それです。 railsinstaller/railsinstaller-windows#76
RailsInstallerでインストールするとrails.bat などのバッチファイルが入るのですが、そのパスの部分が変で動かないという問題です。batファイルをエディタで開いてパスを修正していけば直るとのこと。(前に直してみて動きそうな感じでしたが、ちゃんと動作確認まではしてないです)

@igaiga
Copy link
Contributor

igaiga commented Jul 13, 2016

WinでRails5を使う方法まとめ

案a:クラウド環境(Cloud9 or Nitrousなど)を使う
案b:RubyInstallerでinstall
案c:RailsInstallerをBugFixしてRails5へ(動作未確認)

コーチ素振り会で話して今回の東京開催ではWinは案aのクラウド環境を動作確認して使うことにしました。動作確認して必要であればガイドのクラウドのところに修正のPRを別途投げます。

このPRはマージして、WinはRails5の環境構築を書けるようになったときに一緒に変更するのでどうでしょう?

@yuumi3
Copy link
Contributor

yuumi3 commented Jul 15, 2016

Rails installer試してみました。 railsinstaller/railsinstaller-windows#76 に書かれているように

> cd  C:\RailsInstaller\Ruby2.2.0\bin
> copy /h rake.bat rails.bat
> copy /h rake.bat bundle.bat

を行えば、普通に使えるようです。(Scaffoldで簡単なアプリを作りました)

ただし、Windows10 では出来たアプリをアクセスすると オブジェクトでサポートされていないプロパティまたはメソッドです エラーが発生しました。

coffee-script-sourceを1.8.0にダウングレードすると良いという情報があり、試したところ動きました
http://stackoverflow.com/questions/28421547/rails-execjsprogramerror-in-pageshome

だれか、この件わかりますか?

@hsbt
Copy link
Member

hsbt commented Jul 15, 2016

おそらくですが coffeescript と Windows の javascript エンジン(ブラウザは何だろう)の食い合わせが悪いだと思います。 coffee のバグっぽい雰囲気。

@yuumi3
Copy link
Contributor

yuumi3 commented Jul 16, 2016

ブラウザは IE で試してましたが、Chromeでもダメでした。

Stackoverflowにあった。 node.jsをインストールし gem 'execs'を追加するのも動きますね。

@igaiga
Copy link
Contributor

igaiga commented Jul 16, 2016

私もyuumi3の手順でやって /app ページのアプリを作るところまで動作OKでした。(Win10)

RailsInstaller3.2.0(Ruby2.2.4) でインストールして、実行したコマンドは以下。これでRails5環境ができました。(yuumi3の書いている /h オプションがついているとシンタックスエラーで怒られたのでオプションなしでやりました。)

cd  C:\RailsInstaller\Ruby2.2.0\bin
copy rake.bat rails.bat
copy rake.bat bundle.bat
gem update rails --no-document

yuumi3が書いている「オブジェクトでサポートされていないプロパティまたはメソッドですエラー」は踏まなくて、調べたらOSに事前にnode.jsがインストールされていました。execjs gemが自動的にGemfile.lockに入ってるので、それで回避できてるぽいです。

ガイドの記述だとWin 4. Install node でnode.jsをいれることになっているので、ガイドに従って進めていると踏まないですみそうな感じがします。念のため、「出るかもしれないエラー」欄に「起きたらnode.js いれてね」と書いておいたら良さそうです。

@igaiga
Copy link
Contributor

igaiga commented Jul 16, 2016

WinのRailsInstaller+Rails5環境、こんな感じで大丈夫そうですかね? > @yuumi3

大丈夫そうならば、Win版ガイドはRails5版に更新(RailsInstaller3.2.0に更新+上記のバグ修正方法を書く)するPRで一緒に rake db:migrate → rails db:migrte に更新するようにして、このPRは先にマージするのでいかがでしょう?

@yuumi3
Copy link
Contributor

yuumi3 commented Jul 16, 2016

お願いします > @igaiga

@igaiga
Copy link
Contributor

igaiga commented Jul 17, 2016

kei-sさん、みなさんありがとうございました。マージします。 🌟

@igaiga igaiga merged commit 000cee0 into railsgirls-jp:master Jul 17, 2016
@igaiga igaiga mentioned this pull request Jul 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants