Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 21 Oct 2015 15:13:27 -0700
From:      Bryan Drewery <bdrewery@FreeBSD.org>
To:        =?UTF-8?Q?Juan_Ram=c3=b3n_Molina_Menor?= <listjm@club.fr>, freebsd-current@freebsd.org
Subject:   Re: [CFT] Buildworld ccache support
Message-ID:  <56280E07.50900@FreeBSD.org>
In-Reply-To: <5627F937.4060301@club.fr>
References:  <56264279.7020504@club.fr> <56279AD2.6080009@FreeBSD.org> <5627F937.4060301@club.fr>

next in thread | previous in thread | raw e-mail | index | archive | help
This is an OpenPGP/MIME signed message (RFC 4880 and 3156)
--3THOVHGqnkfjmnpj1T2gCeGBKNmlwNHoM
Content-Type: text/plain; charset=windows-1252
Content-Transfer-Encoding: quoted-printable

On 10/21/2015 1:44 PM, Juan Ram=F3n Molina Menor wrote:
> +.if ${CC:M/*} =3D=3D ""
> +# Can use PATH.
> +PATH:=3D ${CCACHE_WRAPPER_PATH}:${PATH}

These lines in particular did have problems that I fixed recently.

- It would potentially add /usr/local/libexec/ccache into PATH multiple
times in sub-makes (I think, it may have been my own PATH confusing me).
This would still use ccache.
- It would consider the '-isystem /usr/obj/...' used for LIB32MAKE to
match on '${CC:M/*}' which is intended to see if the compiler itself
starts with a /. In this case of building during build32 ccache was not
used.

I have since added 'env CCACHE=3D1' to CC and an .info ${PATH} near this
code to see if it is working and it sure seems to be working in all of
the build.  This does suggest that using the PATH method may not be good
as it leads to confusion about whether it is used or not.

I have seen ccache stats get really messed up before.  They seemed stuck
to me earlier.  I had updated ccache in ports recently and had been
building Poudriere using my global ccache dir.  Outside of the jail I
still had the older ccache.  This may have led to it messing up my cache
and stats.  Once I upgraded ccache in the host and cleaned the cache it
seemed to resume incrementing the counters.

I've also since removed the CCP handling from the patch as I found that
ccache just records a stat and bails out in that case.

--=20
Regards,
Bryan Drewery


--3THOVHGqnkfjmnpj1T2gCeGBKNmlwNHoM
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: OpenPGP digital signature
Content-Disposition: attachment; filename="signature.asc"

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iQEcBAEBAgAGBQJWKA4HAAoJEDXXcbtuRpfPEdkIAJsEF3yiDOeLFmpy8vXZ7gLO
OacBKUnDmyPlNWpidNeqR4cQ+d2nZX8I7RKV/sufM5KTjaqffoyIWXekffvOMqfm
F3cFirMg5/XCV5NLz4X6KLaxbUW4kokgraZK4+urK04xtSoxypfv4qiuLnoZ5rbP
jeQWELIaZnlNymMG+6HoHkljrNjbCHRmLywtKwmVjHNvY5XzkqIYqWM04EswVLmJ
tPL/CS8AICjvaEZ46g4tlKSTC5IPUFvvPFNLZmrqGgnj5DyoCYvn2oGfortKnnDp
P1LkwV4t6joflNymi2/eubqI+/OC9jCAMYfcGESLdE3aXuSjhHKocvwXTTVrQaA=
=XZ/U
-----END PGP SIGNATURE-----

--3THOVHGqnkfjmnpj1T2gCeGBKNmlwNHoM--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?56280E07.50900>