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

Prevent the error in "cargo install rustbook" on Circle CI #200

Merged
merged 5 commits into from
Aug 12, 2016

Conversation

tatsuya6502
Copy link
Member

Circle CI で rustbook に関連する以下の問題を解決します。

  1. cargo install rustbook で malformed URL エラーが起こらないように $HOME/.gitconfig を削除する。
    • エラーの内容

      $ cargo install --root $RUST_HOME --git $RUSTBOOK_GIT_URL --branch $RUSTBOOK_GIT_BRANCH || true
         Updating git repository `https://github.com/tatsuya6502/rustbook.git`
      warning: spurious network error (2 tries remaining): [12/-12] Malformed URL 'ssh://[email protected]:/tatsuya6502/rustbook.git'
      
    • 根本的な原因は Cargo が使用している libgit2 が、ssh://[email protected]:/ という URL に対応していないためと思われる。この URL は、最近 Circle CI の box に追加された $HOME/.gitconfig の URL リライトルールで生成されている。

      [url "ssh://[email protected]:"]
         insteadOf = https://github.com
    • 暫定的な対応として、$HOME/.gitconfig を削除する。

    • 関連する issue コメント:
      the-rust-programming-language-ja/pull/198,
      rust-by-example-ja/issues/38

  2. rustbook のリポジトリ URL をアップデート
  3. 常に最新版の rustbook が使われるよう、cargo install rustbook--force オプションを追加。

@tatsuya6502
Copy link
Member Author

@KeenS レビューをお願いします。

- `|| true` was there to avoid the CI to fail when rustbook
  has been already installed. (In such case, `cargo install` will fail.)
  Now `cargo install` has `--force` option, it should not fail anymore.
@KeenS
Copy link
Member

KeenS commented Aug 12, 2016

ありがとうございます。一応rust by exampleのレポジトリの方のやりとりも追っているので.gitconfig諸々については了解しています。

マージします。

@KeenS KeenS merged commit c372c49 into rust-lang-ja:master Aug 12, 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.

2 participants