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 で cppcheck を実行出来ないか調査する #432

Closed
m-tmatma opened this issue Sep 9, 2018 · 15 comments
Closed

travis ci で cppcheck を実行出来ないか調査する #432

m-tmatma opened this issue Sep 9, 2018 · 15 comments
Labels
CI appveyor など CI 関連 【ChangeLog除外】 cppcheck static analysis

Comments

@m-tmatma
Copy link
Member

m-tmatma commented Sep 9, 2018

travis ci で cppcheck を実行出来ないか調査する
#427

@beru beru added the CI appveyor など CI 関連 【ChangeLog除外】 label Sep 10, 2018
@beru beru added the cppcheck label Sep 18, 2018
@m-tmatma
Copy link
Member Author

sakura-editor の organization を travis-ci に関連づけてみました。
https://travis-ci.org/sakura-editor/sakura

@m-tmatma
Copy link
Member Author

いろいろ実験できるように
https://travis-ci.org/sakura-editor/sandbox
も関連づけました。

@m-tmatma
Copy link
Member Author

https://arne-mertz.de/2017/04/continuous-integration-travis-ci/
によると

install: 
  - sudo apt-get install -qq cppcheck

という処理で cppcheck をインストールするようだ。

@berryzplus
Copy link
Contributor

apt-getってやっぱりフリー版はlinuxのみですか...

@m-tmatma
Copy link
Member Author

apt-getってやっぱりフリー版はlinuxのみですか...

参考にしたサイトがそうなだけで windows もいけるみたいです。
https://docs.travis-ci.com/user/reference/windows/

ただ Early adopters of our Windows environment と書いてあるので、まだ実験的なものかもしれません。
https://docs.travis-ci.com/user/reference/windows/#support

@m-tmatma
Copy link
Member Author

↑ ただ cppcheck を実行するという目的なら、OS は linux でも問題ないと思います。

@k-takata
Copy link
Member

という処理で cppcheck をインストールするようだ。

travisでは普通は、

addons:
  apt:
    packages:
      - cppcheck

のようにsudoを使わずにパッケージをインストールします。sudoを使わない方が速いのです。ただ、インストールできるパッケージは許可されているものだけなので、もしcppcheckが許可されていないのであれば、許可するようにお願いするか、sudoを使わざるを得ません。

travisでWindowsは最近使えるようになったばかりなので、Linuxの方が情報も多くていいんじゃないでしょうか。

@m-tmatma
Copy link
Member Author

travis で用意されている cppcheck は 1.61 だった。
https://travis-ci.org/m-tmatma/sandbox/builds/459304456

4年以上前 (ver 1.63 のリリースで 2014-01-04 ) のものを使っている。
https://sourceforge.net/p/cppcheck/news/

テストに使用した設定ファイルは
m-tmatma/sandbox@ab4ccc9

@berryzplus
Copy link
Contributor

↑ ただ cppcheck を実行するという目的なら、OS は linux でも問題ないと思います。

MinGW対応を引き継いだ理由の一つでもあるんですよね。
cross-GCC対応ならLinux環境しかないVMを使える可能性が残せるので。

@berryzplus
Copy link
Contributor

流してみてて申し訳ない

いろいろ実験できるように
https://travis-ci.org/sakura-editor/sandbox
も関連づけました。

これってもしかしてもう動きます?
(なんかトリガになるキーワードなりを付けてpushするとビルドが走る?)

@m-tmatma
Copy link
Member Author

これってもしかしてもう動きます?

はい。PR を投げました。
sakura-editor/sandbox#8

(なんかトリガになるキーワードなりを付けてpushするとビルドが走る?)

単に push する or PR を投げるだけです

該当のビルドです。
https://travis-ci.org/sakura-editor/sandbox/builds/459316224

@k-takata さんのビルドも、 sudo apt-get install -qq cppcheck も 試しましたが、
cppcheck のバージョンはどちらも 1.61 でした。

@m-tmatma
Copy link
Member Author

はい。PR を投げました。
sakura-editor/sandbox#8

PR の画面で Show all Checks をクリックすると appveyor と travis-ci の結果の
どちらを見るか選べます。

@berryzplus
Copy link
Contributor

バージョンの話は、もしかするとdistの選択で利用できるもののバージョンが決まるからなのかもしれない、と思いました。travis ciのdistがどういう構成か分かりませんがappveyorと似た感じにベースパッケージを選択させてるんではないかと思うので。

@m-tmatma
Copy link
Member Author

m-tmatma commented Apr 5, 2019

#833 を対応したので閉じます。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI appveyor など CI 関連 【ChangeLog除外】 cppcheck static analysis
Projects
None yet
Development

No branches or pull requests

4 participants