This is an archive of the discontinued LLVM Phabricator instance.

Fix hidden-redecls.m test for some environments
ClosedPublic

Authored by kschwarz on Oct 15 2020, 3:04 AM.

Details

Summary

This test was failing in our CI environment, because Jenkins mounts the workspaces into Docker containers using their full path, i.e. /home/jenkins/workspaces/llvm-build.
We've seen permission denied errors because /home/jenkins is mounted with root permissions and the default cache directory under Linux is $HOME/.cache.

The fix is to explicitly provide the -fmodules-cache-path, which the other tests already seem to provide.

Diff Detail

Event Timeline

kschwarz created this revision.Oct 15 2020, 3:04 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 15 2020, 3:04 AM
kschwarz requested review of this revision.Oct 15 2020, 3:04 AM
akyrtzi accepted this revision.Oct 15 2020, 9:29 AM
This revision is now accepted and ready to land.Oct 15 2020, 9:29 AM

Thank you! Are you able to commit it by yourself?

Yes, I'll commit it tomorrow, thanks for the review @akyrtzi!

This revision was landed with ongoing or failed builds.Oct 16 2020, 12:51 AM
This revision was automatically updated to reflect the committed changes.