This is an archive of the discontinued LLVM Phabricator instance.

Fix binaries directory for flang slaves gcc10 and clang10
ClosedPublic

Authored by CarolineConcatto on Aug 12 2020, 1:10 AM.

Event Timeline

CarolineConcatto requested review of this revision.Aug 12 2020, 1:10 AM
DavidTruby requested changes to this revision.Aug 12 2020, 5:02 AM
DavidTruby added inline comments.
buildbot/osuosl/master/config/builders.py
1562–1563

I think these should either both be clang-10 and clang++-10, or neither should have the suffix. I think they are installed without a suffix on the machine (so should be clang and clang++)

This revision now requires changes to proceed.Aug 12 2020, 5:02 AM
buildbot/osuosl/master/config/builders.py
1562–1563

Actually, both of them work, or looks like they work.
But maybe to keep it more clean having both equal is better

Keep consistency between the names for clang

This revision is now accepted and ready to land.Aug 12 2020, 6:01 AM