search

nicolasbock / ebuildtester

A dockerized approach to test a Gentoo package within a clean stage3
http://ebuildtester.readthedocs.io/
BSD 3-Clause "New" or "Revised" License
33 stars 16 forks source link

Add ccache support #184

Closed stkw0 closed 2 years ago

stkw0 commented 2 years ago

Closes: #163

stkw0 commented 2 years ago

This looks great @stkw0 !

Just one comment: Don't you have to also emerge ccache in the container to be able to use this feature?

Yes, I forgot it, I still didn't had time to test it with properly. Also I may add support for CCACHE_SIZE although I am not sure how this will work in a shared CCACHE

nicolasbock commented 2 years ago

Ok, I see. Also, according to [1] you also should set

FEATURES="ccache"
CCACHE_DIR="/var/cache/ccache"

i.e. CCACHE_DIR in make.conf.

[1] https://wiki.gentoo.org/wiki/Ccache

stkw0 commented 2 years ago

I used the default value for CCACHE_DIR according to man make.conf. I will check which is the real default value when I get home

El lun., 20 jun. 2022 19:01, Nicolas Bock @.***> escribió:

Ok, I see. Also, according to [1] you also should set

FEATURES="ccache" CCACHE_DIR="/var/cache/ccache"

i.e. CCACHE_DIR in make.conf.

[1] https://wiki.gentoo.org/wiki/Ccache

— Reply to this email directly, view it on GitHub https://github.com/nicolasbock/ebuildtester/pull/184#issuecomment-1160670762, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABFD7GQ4FBOISHJFMIY6KCLVQCPYLANCNFSM5ZJEQK5A . You are receiving this because you were mentioned.Message ID: @.***>

nicolasbock commented 2 years ago

Ah ok, that makes sense. Thanks!

stkw0 commented 2 years ago

Now it's working. Instead of adding another use flag for CCACHE_SIZE I created the configuration file for ccache which in my opinion is better as you don't have to remember to pass any extra flag and it applies everywhere