Patch Set 4: > I think for this kind of change it would be worth discussing it in > the eclipse titan forum. Maybe they have some less > intrusive/hackish ideas about how to use ccache with TITAN? If > there are no flags yet to disable the timestamps, maybe the related > discussin will prompt them to add this to future TITAN releases.
Indeed. As noted in the patch itself, I believe we would need an upstream solution in the long term if we decided to adopt use of ccache. This patch set is not intended as a permanent solution. It is a proof of concept which allows us to evaluate advantages ccache puts on our table, before investing time with getting upstream involved. I think we should at least investigate how ccache affects our jenkins builds jobs before talking to upstream. That should show whether additional effort would be well spent. I invested a signifcant amount time (4 hours) to figure out why ccache wasn't working out of the box. This patch set saves other people the trouble of trying to get it to work. -- To view, visit https://gerrit.osmocom.org/7601 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ibb538f602206535c06980f88191c1dabe3c4cd82 Gerrit-PatchSet: 4 Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Owner: Stefan Sperling <[email protected]> Gerrit-Reviewer: Harald Welte <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: Neels Hofmeyr <[email protected]> Gerrit-Reviewer: Stefan Sperling <[email protected]> Gerrit-HasComments: No
