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

add integrately android support #894

Closed
wants to merge 1 commit into from
Closed

Conversation

BDZNH
Copy link
Contributor

@BDZNH BDZNH commented May 9, 2022

支持在安卓源码内编译。
我把tutorial的几个bin编译出来看能正常跑的。我这里是Android 10的环境,不过Android.bp从Android 8的源码环境就可以识别到了,应该是可以用的。

@Barenboim
Copy link
Contributor

Barenboim commented May 9, 2022

这个你可能需要参考我们最新的bazel BUILD。kafka有两个文件,需要加上no-rtti。命名什么也可以参考一下BUILD

@BDZNH
Copy link
Contributor Author

BDZNH commented May 10, 2022

这个你可能需要参考我们最新的bazel BUILD。kafka有两个文件,需要加上no-rtti。命名什么也可以参考一下BUILD

你说的是这个修改吗 optimize bazel ,如何验证是否生效呢

@Barenboim
Copy link
Contributor

Barenboim commented May 11, 2022

kafka部分文件no-rtti,是为了避免最新版snappy不支持rtti的问题。如果不加no-rtti,会编译错误。
不过,我不太懂安卓源码内编译的作用,在根目录上加了个Android.bp文件挺奇怪的,而且下次我们增加文件,还得多改一份代码。我们也不倾向于在根路径放太多文件。
要不你在 #773 这个issue里补充一下安卓源码内编译的方法?给大家一个路径下载build文件?谢谢。

@BDZNH
Copy link
Contributor Author

BDZNH commented May 15, 2022

kafka部分文件no-rtti,是为了避免最新版snappy不支持rtti的问题。如果不加no-rtti,会编译错误。 不过,我不太懂安卓源码内编译的作用,在根目录上加了个Android.bp文件挺奇怪的,而且下次我们增加文件,还得多改一份代码。我们也不倾向于在根路径放太多文件。 要不你在 #773 这个issue里补充一下安卓源码内编译的方法?给大家一个路径下载build文件?谢谢。

issue 内我更新过了,应该直接复制issue的内容就可以

@BDZNH BDZNH closed this May 15, 2022
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