search

google / lyra

A Very Low-Bitrate Codec for Speech Compression
Apache License 2.0
3.84k stars 355 forks source link

Failed to build for linux #85

Closed Ryuk17 closed 2 years ago

Ryuk17 commented 2 years ago

I try to build lyra for linux with bazel build -c opt :encoder_main, while I met the following error, which seems that the Android NDK is necessary. 2022-08-09 12-51-24 的屏幕截图

If I just want to build and run lyra on linux, is it necessary to download Android NDK ?

aluebs commented 2 years ago

Unfortunately at this time, the Android NDK is necessary even to build Lyra for Linux. But a completely agree that it shouldn't. We'll be looking into it, but if you figure it out, patches are always welcome :)

Ryuk17 commented 2 years ago

Thanks for you reply, I am willing to find some methods to build Lyra for Linux without Android NDK.

aluebs commented 2 years ago

This will be fixed in the next release, but in the meantime, as a workaround, you can remove the android_sdk_repository and android_ndk_repository from the WORKSPACE.

aluebs commented 2 years ago

This was fixed with the Lyra 1.2.0 release.