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

travis-CIが通るようにする #42

Merged
merged 2 commits into from
Oct 30, 2017
Merged

travis-CIが通るようにする #42

merged 2 commits into from
Oct 30, 2017

Conversation

karubabu
Copy link
Contributor

バージョン情報を指定した場合何かのAPIしかインストールされないようなのでバージョン情報を指定しないようにします。

バージョン情報が無いのは問題かもしれませんがCIが通るようにはなります。

以下エラー文
E: Package 'valac-0.24' has no installation candidate
The command "if [ "$VALAC" = "valac-0.24" ]; then sudo add-apt-repository -y ppa:vala-team && sudo apt-get update -qq && sudo apt-get install -qq autotools-dev gnome-common gobject-introspection intltool libgtk-3-dev libgee-0.8-dev libgirepository1.0-dev libjson-glib-dev "$VALAC"; fi" failed and exited with 100 during .

バージョン情報が無いのは問題かもしれない
@ueno
Copy link
Owner

ueno commented Oct 28, 2017

I would rather suggest to use a newer system image through Docker, like this:
ueno/libkkc#4
https://docs.travis-ci.com/user/docker/

PRのコメント: ueno/libkkc@e5adcf4
を参考にして必要の無さそうなパッケージを削って作成しました
この状態ではcoverallsがレポジトリを見つけられないため、オーナーによって
```
gem install travis
cd /srcdir
travis encrypt COVERALLS_REPO_TOKEN="coveralls.io libskk_repo_token" --add
```
する必要があると思っています

私はBUILD_OPTS環境変数の意味を殆ど理解していないためこの部分を触っていません
@karubabu
Copy link
Contributor Author

fedoraのdocker imageを使用するように変更しました。
私が不要だと考えた(削除した後travis-ciを実行して問題なくビルドが通った物)パッケージやcoverallsのために必要なパッケージを追加しています。
configureやautogen.shの引数やBUILD_OPTS環境変数については私は殆ど分かっていないためそのままになっています。
coverallsはrepo_tokenを必要としているはずなので現状は実行できていません。
実行できた場合はこのようになることを確認しています。
https://travis-ci.org/karubabu/libskk/jobs/294833448
実行にはオーナーによって

gem install travis
cd /srcdir
travis encrypt COVERALLS_REPO_TOKEN="coveralls.io libskk_repo_token" --add

される必要があります。

@ueno ueno merged commit fb7162f into ueno:master Oct 30, 2017
@ueno
Copy link
Owner

ueno commented Oct 30, 2017

Cool, thanks!

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