sofastack / sofa-rpc

SOFARPC is a high-performance, high-extensibility, production-level Java RPC framework.
https://www.sofastack.tech/sofa-rpc/docs/Home
Apache License 2.0
3.81k stars 1.17k forks source link

Fix mac aarch compilation #1303

Closed khotyn closed 11 months ago

khotyn commented 1 year ago

Fix #1302

Dependency upgrade to compatible with aarch and IPv6:

FIx test cases:

khotyn commented 1 year ago

可能还需要考虑下 gRPC 的版本的变化可能会导致的兼容性问题。

JervyShi commented 1 year ago

看起来 Nacos 还没升级支持 m1/m2 的 jraft 版本 https://github.com/alibaba/nacos/issues/9015

khotyn commented 1 year ago

@JervyShi 发布版本还没有,master 的代码已经升级了。

JervyShi commented 1 year ago

https://github.com/alibaba/nacos/pull/9790 看到有人已经提交了代码,等 Nacos 发版了我发个 nacos-embedded 版本

JervyShi commented 1 year ago

alibaba/nacos#9790 看到有人已经提交了代码,等 Nacos 发版了我发个 nacos-embedded 版本

https://github.com/JervyShi/nacos-embedded/releases/tag/v0.3.0 已经 Release,Maven 中央仓库已上传,但需要一些时间才能查到

khotyn commented 1 year ago

@JervyShi Thank you~~

stale[bot] commented 1 year ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

khotyn commented 1 year ago

Still waiting https://github.com/sofastack/sofa-registry/issues/327 to complete so we can have a sofa registry-test version that is compatible with aarch.

codecov[bot] commented 11 months ago

Codecov Report

Merging #1303 (f3b9034) into master (eb7d33f) will decrease coverage by 0.07%. The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1303      +/-   ##
============================================
- Coverage     72.09%   72.02%   -0.07%     
+ Complexity      784      783       -1     
============================================
  Files           416      416              
  Lines         17661    17661              
  Branches       2752     2752              
============================================
- Hits          12732    12721      -11     
- Misses         3525     3537      +12     
+ Partials       1404     1403       -1     

see 7 files with indirect coverage changes

khotyn commented 11 months ago

All the test case passes! @OrezzerO @EvenLjj @Lo1nt Pls help review~~~, thanks