Re: [libvirt-jenkins-ci PATCH 7/7] lcitool: Create and expose ccache wrappers

On Fri, Mar 27, 2020 at 08:34:59PM +0100, Andrea Bolognani wrote: > + commands.extend([ > + "mkdir -p /usr/local/share/ccache-wrappers", > + ]) > + > + if cross_arch: > + commands.extend([ > + "ln -s {ccache} /usr/local/share/ccache-wrappers/{cross_abi}-cc", > + "ln -s {ccache} /usr/local/share/ccache-wrappers/{cross_abi}-$(basename {cc})", > + ]) > + else: > + commands.extend([ > + "ln -s {ccache} /usr/local/share/ccache-wrappers/cc", > + "ln -s {ccache} /usr/local/share/ccache-wrappers/$(basename {cc})", > + ]) > + > script = "\nRUN " + (" && \\\n ".join(commands)) + "\n" > sys.stdout.write(script.format(**varmap)) I've just realized that this addition has prevented the caching and reuse the base layer across the cross images. The first "RUN" cmmand was supposed to have stuff that is common across cross images, but we've accidentally included the "cross_abi" in the path for ccache here. We need to put the ccache setup in the second RUN command, for cross containers. Only native images can have it in the first RUN command.