Skip to content

rustbook の "update-playpenjs" ブランチを使うよう circle.yml を変更する #159

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

Merged
merged 1 commit into from
Jun 17, 2016

Conversation

tatsuya6502
Copy link
Member

#158 (Rust コードに「Run」ボタンが表示されない)の原因となった rustbook crate の問題を rustbook 側で修正しました。修正版を使うために、circle.yml 内で指定している rustbook git レポジトリのブランチ名を修正します。

なお、rustbook を再インストールするために、CircleCI のキャッシュをクリアする必要があります。本 PR のマージ後に、Circle CI の UI からキャッシュのクリアを行います(参考

- This will fix issue rust-lang-ja#158 once CircleCI's cache is cleared.
@tatsuya6502 tatsuya6502 added this to the 1.9 milestone Jun 17, 2016
@tatsuya6502 tatsuya6502 self-assigned this Jun 17, 2016
@tatsuya6502
Copy link
Member Author

なお、rustbook を再インストールするために、CircleCI のキャッシュをクリアする必要があります。

自分の CircleCI 環境で試したところ、キャッシュクリア後に rustbook が再インストールされ、その後生成された HTML で、Run ボタンが使えるようになったことを確認できました。

この内容でマージします。

参考:これ以降は、自分の環境 での結果です。

キャッシュクリア前

$ cargo install --root $RUST_HOME --git $RUSTBOOK_GIT_URL --branch $RUSTBOOK_GIT_BRANCH || true
    Updating git repository `https://github.com/tatsuya6502/rustbook.git`
error: binary `rustbook` already exists in destination as part of `rustbook v0.4.0 (https://github.com/tatsuya6502/rustbook.git?branch=rust-1.11.0-nightly#638aae71)`
Add --force to overwrite

キャッシュクリア後

$ cargo install --root $RUST_HOME --git $RUSTBOOK_GIT_URL --branch $RUSTBOOK_GIT_BRANCH || true
    Updating git repository `https://github.com/tatsuya6502/rustbook.git`
   Compiling rustbook v0.4.0 (https://github.com/tatsuya6502/rustbook.git?branch=update-playpenjs#2209bd35)
  Installing /home/ubuntu/rust/nightly-2016-06-01/bin/rustbook

playpen.js は最新版に戻り、また、生成されたページ上の Rust ブロックで「Run」ボタンが使えることを確認しました()。

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.

1 participant