xiaoyifang / goldendict-ng

The Next Generation GoldenDict
https://xiaoyifang.github.io/goldendict-ng/
Other
1.72k stars 95 forks source link

action: update brew before running macOS workflows #1960

Closed shenlebantongying closed 1 week ago

shenlebantongying commented 1 week ago

This is exactly the same as: https://github.com/xiaoyifang/goldendict-ng/blob/808d8576027641da897c43dd607f89f26619beab/.github/workflows/PR-check-cmake.yml#L20

Whatever in the GitHub's runner image is outdated compared to whatever in the remote server.

sonarcloud[bot] commented 1 week ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

xiaoyifang commented 1 week ago

from the size of the final macos package, it seems the icu-data may be still not got packaged.https://github.com/xiaoyifang/goldendict-ng/releases/tag/v24.11.0-alpha.652da8e1

shenlebantongying commented 1 week ago

libicui18n.dylib and libicuuc.dylib are both in the dmg files.

libzim only needs icu-i18n (and indirectly icu-uc). I don't think it requires icu-data.

https://github.com/openzim/libzim/blob/cdcbb6c14888593cbfaec83f078b8435b9c2a91c/meson.build#L79-L83

The FindICU macro provided by CMake is not what we needs.

There are Apple's ICU <-> homebrew's ICU. The FindICU from CMake will find the System's ICU and the libzim built by homebrew requires homebrew's ICU.

shenlebantongying commented 1 week ago

Ok, it appears things changed, now it also requires icudata

xiaoyifang commented 1 week ago

https://github.com/xiaoyifang/goldendict-ng/issues/1954#issuecomment-2479690304

icuuc->icudata