Page MenuHomePhabricator

[bazel] Be consistent and say we don't support libfpm everywhere
ClosedPublic

Authored by aeubanks on Thu, Sep 22, 6:44 PM.

Details

Summary

We're inconsistent about saying whether or not we support libfpm in the bazel build.

This should perhaps be configurable via a --config.

This allows the buildbots to see when llvm-exegesis-related breakages occur.

Diff Detail

Event Timeline

aeubanks created this revision.Thu, Sep 22, 6:44 PM
Herald added a reviewer: ctetreau. · View Herald Transcript
Herald added a project: Restricted Project. · View Herald Transcript
Herald added a subscriber: mstojanovic. · View Herald Transcript
aeubanks requested review of this revision.Thu, Sep 22, 6:44 PM
Herald added a project: Restricted Project. · View Herald TranscriptThu, Sep 22, 6:44 PM

if we really care about this in the bazel build then I can look into making things work on the bots

courbet accepted this revision.Fri, Sep 23, 12:37 AM

How do we handle other optional deps with bazel (e.g. z3 or things like this) ?

This revision is now accepted and ready to land.Fri, Sep 23, 12:37 AM

How do we handle other optional deps with bazel (e.g. z3 or things like this) ?

looks like we just disable them all