roc-streaming / roc-toolkit

Real-time audio streaming over the network.
https://roc-streaming.org
Mozilla Public License 2.0
1.02k stars 203 forks source link

Support using ccache when building gengetopt and ragel #639

Closed gavv closed 1 week ago

gavv commented 7 months ago

User can enable ccache using --compiler-launcher=ccache scons option.

When user specifies --build-3rdparty=<dependencies>, and ccache is enabled, we automatically use ccache for almost all dependencies. However, we don't enable ccache for gengetopt and ragel. See build-3rdparty.py script.

The reason is that gengetopt and ragel are tools needed during build. Unlike other dependencies, they're not cross-compiled. Instead, they are built natively and then scons is using them. Because of that, build-3rdparty.py doesn't provide CC and CXX vars when it builds gengetopt and ragel, and so there is no place where it could inject ccache.

So the fix is straightforward:

See also #638.

jeshwanthreddy13 commented 2 weeks ago

Hi @gavv , I'm interested in this project and would like to get involved. Could you assign this issue to me so I can start working on it?

gavv commented 2 weeks ago

Welcome, thanks!

gavv commented 1 week ago

Landed