Skip to content

Commit

Permalink
meson: Update version requirement for GTest (>= 1.10.0) (#1131)
Browse files Browse the repository at this point in the history
GTestの依存関係を探す処理を更新してバージョン要件を1.10.0以上にします。
また、ライブラリ 'gtest_main' を使うようにmeson.buildを変更します。

変更前は dependency() の 'main' キーワード引数[1]を利用して
GTestが提供する main() とリンクしていました。
GTest 1.10.0のころから pkg-config のサポートが追加されているため
'main' キーワードを使わなくてもテストをビルドできるようになりました。

この修正でMeson実装の一つである muon[2] を使ってJDimのビルドやテストが
可能になります。dependency() の 'main' キーワードは今のところ
古いGTestのサポート以外では使われていないようです。

[1]: https://mesonbuild.com/Dependencies.html#gtest-and-gmock
[2]: https://sr.ht/~lattis/muon/
  • Loading branch information
ma8ma committed Mar 25, 2023
1 parent 98cd6ed commit 67596c6
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion INSTALL
Expand Up @@ -32,7 +32,7 @@
・alsa-lib (-Dalsa=enabled)
・openssl 1.1.1 以上 (-Dtls=openssl)
・migemo (-Dmigemo=enabled)
・googletest (`test/RADME.md`を参照)
・googletest 1.10.0 以上 (`test/RADME.md`を参照)

WebPやAVIF画像の表示に必要なパッケージ
インストールされていない環境では`.webp`や`.avif`で終わるURLは通常リンクになる。
Expand Down
2 changes: 1 addition & 1 deletion docs/manual/make.md
Expand Up @@ -44,7 +44,7 @@ layout: default
- alsa-lib (`-Dalsa=enabled`)
- openssl 1.1.1 以上 (`-Dtls=openssl`)
- migemo (`-Dmigemo=enabled`)
- googletest ([test/RADME.md][testreadme]を参照)
- googletest 1.10.0 以上 ([test/RADME.md][testreadme]を参照)

#### WebPやAVIF画像の表示に必要なパッケージ
インストールされていない環境では`.webp``.avif`で終わるURLは通常リンクになる。
Expand Down
4 changes: 2 additions & 2 deletions meson.build
Expand Up @@ -151,9 +151,9 @@ endif
build_tests_opt = get_option('build_tests')
if not build_tests_opt.disabled()
# ディストロのパッケージとmeson wrapに対応
gtest_main_dep = dependency('gtest',
main : true,
gtest_main_dep = dependency('gtest_main',
fallback : ['gtest', 'gtest_main_dep'],
version : '>= 1.10.0',
required : build_tests_opt)
build_tests = gtest_main_dep.found()
else
Expand Down

0 comments on commit 67596c6

Please sign in to comment.