[question] ccache in pre-release

[tylerjw]

In our testing it seemed like pre-release wasn't using ccache but then I read the code for the pre-release test and I found these lines:

industrial_ci/industrial_ci/src/tests/ros_prerelease.sh

Lines 31 to 33 in e51bbf4

	if ! [ -d "$WORKSPACE/home/.ccache" ]; then
	ici_asroot mkdir -p "$WORKSPACE/home/.ccache"
	fi

These don't reference the CCACHE_DIR but instead check for the existance of a specific directory. Does this work if I have the ccache directory somewhere else?

[mathias-luedtke]

This was only a work-around for ros-infrastructure/ros_buildfarm#695 and might not be needed anymore.

Does this work if I have the ccache directory somewhere else?

I am not sure if ros_buildfarm supports this (ros-infrastructure/ros_buildfarm#582 wasn't merged yet).

[mathias-luedtke]

At the moment $WORKSPACE is a temporary directroy, but we could put it into $BASEDIR as well.
Then you could cache it from there..

[tylerjw]

Maybe WORKSPACE could be set to be BASEDIR if it was defined and then we could cache the ccache directory in there?

[mathias-luedtke]

Actually, CCACHE_DIR should work..

industrial_ci/industrial_ci/src/tests/ros_prerelease.sh

Lines 93 to 96 in bfc68c8

	if [ -n "${CCACHE_DIR}" ]; then
	ici_forward_mount CCACHE_DIR rw "$WORKSPACE/home/.ccache"
	CCACHE_DIR= # prevent cachedir from beeing added twice
	fi

[mathias-luedtke]

The ccache is forwarded properly, but it looks like ros_buildfarm does not use it anymore (ros-infrastructure/ros_buildfarm#844).

[mathias-luedtke]

See ros-infrastructure/ros_buildfarm#876 as well

[mathias-luedtke]

@tylerjw: please test #670. It should work with CCACHE_DIR. BASEDIR is now supported as well.