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

googletestとcompiletestsを2回目のビルドに対応させる #1436

Merged
merged 1 commit into from
Oct 21, 2020

Conversation

sanomari
Copy link
Contributor

@sanomari sanomari commented Oct 20, 2020

PR の目的

googletestとcompiletestsをリビルドに対応させて、
テストコードの変更時に毎回googletestが再コンパイルされてしまうビルド環境の不備を修正します。

カテゴリ

  • ビルド関連
    • ローカルビルド

PR の背景

誰も興味ないということでしたので自分で対応してみました。

PR のメリット

自明だと思います。

PR のデメリット (トレードオフとかあれば)

特にないと思います。

仕様・動作説明

googletestとcompiletestのビルド条件を変更します。

  • googletest
    こちらが修正のメインです。
    • 変更前 毎回フルビルド。(cmake configure してビルド、cmake install)
    • 変更後 libが未作成の場合にフルビルド。
  • compiletests
    サブの変更です。googletestのビルド設定に依存する書き方になってたので独立させました。
    • 変更前 毎回実行。(cmake configure のみですが configure が重いです。)
    • 変更後 未configureの場合に実行。

PR の影響範囲

googletestのビルドに影響する変更です。

テスト内容

  • クリーン状態からビルド、 gtestがビルドされること
  • ビルド状態からビルド、 gtestがビルドされないこと(vsの機能)
  • テストのファイル(code-main.cpp)のタイムスタンプを更新してビルド、gtestがビルドされないこと。
  • ビルド状態からクリーン、gtestの中間生成物がすべて削除されること。

関連 issue, PR

resolves #1433

参考資料

@AppVeyorBot
Copy link

Build sakura 1.0.3177 completed (commit ece95fa016 by @sanomari)

Copy link
Contributor

@berryzplus berryzplus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

問題なさそうに見えます。ぼくよりずっと詳しいかな?

よくわかってそうなのであえてツッコミ入れるんですけど、
コミットメッセージも修正しとくのがベターだと思います。

googletestとcompiletestsをリビルドに対応させる

修正するのはリビルドじゃなくて2回目のビルドですよね?

@AppVeyorBot
Copy link

Build sakura 1.0.3178 completed (commit 1aad4f9278 by @sanomari)

@berryzplus
Copy link
Contributor

マージしときます。

@berryzplus berryzplus merged commit 75e977b into sakura-editor:master Oct 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

テストをビルドするときに毎回googletestがビルドされる。
4 participants