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

Support all compile options and unittest for Linux workflow #2018

Merged
merged 12 commits into from
Nov 27, 2022

Conversation

guodongxiaren
Copy link
Member

@guodongxiaren guodongxiaren commented Nov 26, 2022

What problem does this PR solve?

Issue Number: #1963

Problem Summary:

What is changed and the side effects?

Changed:

  • 支持所有编译参数的编译。比如with-thrift和with-glog
  • 新增actions目录和配置文件,抽象公共action,加强复用,减少冗余代码
  • 支持clang的unittest,gcc的unittest有失败case,暂不包含gcc的unittest
  • 针对make编译,额外调用一次protoc来处理mcpack2pb中的proto,因为make的编译不稳定,经常报错src/mcpack2pb/generator.cpp:30:10: fatal error: idl_options.pb.h: No such file or directory Makefile修复,避免mcpack编译失败

Side effects:


Check List:

  • Please make sure your changes are compilable(请确保你的更改可以通过编译).
  • When providing us with a new feature, it is best to add related tests(如果你向我们增加一个新的功能, 请添加相关测试).
  • Please follow Contributor Covenant Code of Conduct.(请遵循贡献者准则).

guodongxiaren and others added 6 commits November 26, 2022 15:16
* all opt

* Update ci_linux.yml

* Update ci_linux.yml

* Update ci_linux.yml

* Update ci_linux.yml

* Update ci_linux.yml

* action

* fix

* fix

* all-dep

* fix

* Update action.yml

* fix run

* all-opt

* core

* core

* init-make-config

* opt

* config brpc

* fix

* rm deafult
@guodongxiaren guodongxiaren changed the title Support all compile options for Linux workflow Support all compile options and unittest for Linux workflow Nov 26, 2022
@zyearn
Copy link
Member

zyearn commented Nov 26, 2022

针对make编译,额外调用一次protoc来处理mcpack2pb中的proto,因为make的编译不稳定,经常报错src/mcpack2pb/generator.cpp:30:10: fatal error: idl_options.pb.h: No such file or directory
Side effects:

我记得这个是并发编译的时候protoc还在执行,但引用idl_options.pb.h已经需要前一步的结果了,这里应该需要加一个编译顺序

@zyearn
Copy link
Member

zyearn commented Nov 26, 2022

install-all-dependences 和 install-dependences 的区别是all包含非必要的依赖,可以换个名字比如:

  • install-essential-dependences
  • install-all-dependences

然后all版本引用essential的文件而不是再写一份,不清楚这个在yml文件里能不能实现?

@guodongxiaren
Copy link
Member Author

@zyearn 修改了,有空看下

@zyearn
Copy link
Member

zyearn commented Nov 27, 2022

赞,之后提个PR把老的travis删掉应该就好了

@zyearn zyearn merged commit 1936db9 into apache:master Nov 27, 2022
@zyearn
Copy link
Member

zyearn commented Nov 27, 2022

以及github主页上的build link应该要改成指向Actions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants